1

Eu tenho um formulário com dados pessoais do cliente e queria enviar esses dados para um class que possui um método que salva os dados no banco de dados, mas o formulário é muito grande e eu queria mandar um array ou o próprio POST, mas não queria ficar quebrando isso em variáveis, mandar o array inteiro para o banco.

$res = $cliente->cadastrar_cliente($_POST['name'],$_POST['cpf_cnpj'],
            $_POST['name_fan'],$_POST['zip_code'],
            $_POST['city'],$_POST['state'],$_POST['street'],
            $_POST['complement'],$_POST['district'],$_POST['district'],
            $_POST['email'],$_POST['password'],$_POST['telefone'],$_POST['celular'])

Estou mando desse jeito, isso ta na view, queria mandar algo menos "feio" para não deixar a view bagunçada.

function cadastrar_cliente(
            $nome_responsavel, $cpf_cnpj = null, $nome_fantasia, $cep, 
            $cidade,$uf,$rua, $complemento,$bairro,$numero,$email, $senha, $telefone = null, $celular)

        {
            $pdo = $this->conn->open_connect("db_forpaper");

            $sql = "INSERT INTO tbl_cliente (nome_responsavel,cpf_cnpj,nome_fantasia,cep, 
            cidade,uf,rua,complemento,bairro,numero,email,senha,telefone,celular) 
            VALUES ('$nome_responsavel', '$cpf_cnpj', '$nome_fantasia', '$cep', '$cidade', 
            '$uf','$rua','$complemento','$bairro','$numero','$email','$senha','$telefone','$celular')";

            $insert = $pdo->prepare($sql)->execute();
            if($insert){
                $res = "ok";
            }else{
                $res = "erro";
            }
            return $res;
        }

Assim eu pego na minha class com o método de cadastro.

function cadastrar_cliente($_POST)

        {
            $pdo = $this->conn->open_connect("db_forpaper");

            $sql = "INSERT INTO tbl_cliente (nome_responsavel,cpf_cnpj,nome_fantasia,cep, 
            cidade,uf,rua,complemento,bairro,numero,email,senha,telefone,celular) 
            VALUES ($_POST)";

            $insert = $pdo->prepare($sql)->execute();
            if($insert){
                $res = "ok";
            }else{
                $res = "erro";
            }
            return $res;
        }

Queria algo assim para evitar ficar criando um array enorme ou quebrando ele em variáveis locais. E eu queria uma opinião, este jeito que estou fazendo está muito errado? sou estagiário e esse é meu primeiro sistema que faço sozinho.

3
  • Não vou escrever como resposta pois o código só funciona com MySql(MariaDB não funciona) e a engine tem que ser InnoDB. Supondo as propriedades de $_POST estejam em mesma quantidade que as colunas de sua tabela e os nomes da propriedades sejam os mesmos das colunas de sua tabela você pode inserir como JSON: $sql = "INSERT INTO tbl_cliente VALUES ({json_encode ($_POST)})"; 13/07/2019 às 15:40
  • 1
    Se usasse Prepared Statment, que inclusive é mais seguro do que o que está fazendo atualmente, poderia simplesmente fazer algo como $stmt->bind_param("ssssssssssssss", ...$_POST).
    – Inkeliz
    13/07/2019 às 16:38
  • Sim sim, eu fiz desse jeito, muito obrigado. 13/07/2019 às 16:52

1 Resposta 1

0

Se você quer deixar isso menos feio, porque não utiliza o PDO com curingas? Isso deixa até sua aplicação mais segura. Você pode ler sobre isso aqui. E tem um exemplo bem legal de outra pessoa AQUI.

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .