Insert "orientado"

[srmacedo 66]

Até então eu usava um comando para cada insert que eu fosse fazer no meu banco de dados. Visto que isso ocupa muito tempo, fiz esse simples algoritmo para auxiliar e deixar de forma automática.

 

function inserir($pCampos, $pTabela, $pValue) {
        $pCampos2 = array();
        
        for ($x = 0; $x < count($pCampos); $x++) {
            array_push($pCampos2, ":".$pCampos[$x]);
        }
        
        $sqlQuery = "INSERT INTO ".$pTabela." (".implode(", ", $pCampos).") values "
                . "(".implode(", ", $pCampos2).")";
        
        $this->setPdo($this->con->getConexao()->prepare($sqlQuery));
        
        for ($x = 0; $x < count($pCampos); $x++) {
        $this->getPdo()->bindValue($pCampos2[$x], $pValue[$pCampos[$x]]);
        }
        
        $this->getPdo()->execute();
    }

 

Gostaria de saber se existe uma forma melhor de se fazer isso, ou se está bom da forma que está.

Até mais.

[Williams Duarte 431]

Cheguei neste post através deste!
 

Seu código tem sérios problemas de design, dentre ele estes for()
 

for ($x = 0; $x < count($pCampos); $x++) {
            array_push($pCampos2, ":".$pCampos[$x]);
        }

for ($x = 0; $x < count($pCampos); $x++) {
        $this->getPdo()->bindValue($pCampos2[$x], $pValue[$pCampos[$x]]);
        }

 

 

e parâmetros invertidos

( $pCampos, $pTabela, $pValue )

E claro a falta de Doc.
 

Pra resolver o problema do for, use a função array_map repare que ele é uma função de callback.


Exemplo
 

$map = array_map(function($i){
    return ':'. $i;
}, $pCampos);

print_r($map);

só criar uma função separada para usar em outros lugares.

[srmacedo 66]

@Williams Duarte, obrigado pelo seu comentário e por sua dica com array_map.

Qual seria um padrão recomendado para uma situação dessa forma?

Perdão pela falta de doc.

[Williams Duarte 431]

Pesquise por Domain Service ou Application Service em modelagem DDD.

 

[Williams Duarte 431]

 ia me esquecendo hehehe

Ou também e de forma mais simplista o CRUD