6

Então galera, eu não estou conseguindo resolver esse problema de uma query dentro de um foreach loop não executar. Primeiramente eu pensei que era algo errado com pdo->beginTransaction então eu comentei essa parte e deixei somente código puro.

    <?php
    if (isset($_POST["nome"])):
        try{
            // Começo da validação
            //$pdo->beginTransaction();
            // Validação do campo nome
            if ($_POST["nome"] != null):
                $procurarnome=$pdo->prepare("SELECT id FROM users WHERE username=$name");
                $procurarnome->execute();
            else:
                die("Houve um erro no sistema, contate um administrador!");
            endif;
            // Validação dos campos dinâmicos
            $cadastraresposta=$pdo->prepare("INSERT INTO form_respostas(perguntaid,username,reposta)VALUES(:perguntaid,$name,:resposta)");
            foreach ($listarpergunta as $pergunta) {
                if ($_POST["pergunta$pergunta->id"] != null):
                    $resposta=addslashes($_POST["pergunta$pergunta->id"]);
                    $cadastraresposta->bindParam(":perguntaid",$pergunta->id,PDO::PARAM_INT);
                    $cadastraresposta->bindParam(":resposta",$resposta);
                    $cadastraresposta->execute();
                    var_dump($cadastraresposta->execute());
                else:
                    $pdo->rollBack();
                    die("Preencha todos os campos corretamente!<br />");
                endif;
            }
            // Todos os arquivos foram preenchidos corretamente
            //$pdo->commit();
            echo "Obrigado!";
        }
        catch(PDOException $pe){
             //$pdo->rollback();
             die($pe->getMessage());
        }
    else:
?>

As querys de INSERT dentro do foreach não fazem nada no banco de dados e esse var_dump me retorna um valor "boolean false". O que devo fazer?

2
  • O PHP está configurado para exibir os erros? Commented 10/08/2014 às 2:11
  • 2
    Prepared Statements: voce está fazendo isso errado Commented 10/08/2014 às 5:58

3 Respostas 3

2

Acho que tem que acionar o bind fora do foreach, tenta assim:

$cadastraresposta=$pdo->prepare("INSERT INTO form_respostas(perguntaid,username,reposta)VALUES(:perguntaid,$name,:resposta)");
$cadastraresposta->bindParam(":perguntaid",$perguntaid,PDO::PARAM_INT);
$cadastraresposta->bindParam(":resposta",$resposta);
foreach ($listarpergunta as $pergunta) {
    if ($_POST["pergunta$pergunta->id"] != null):
        $perguntaid=$pergunta->id;
        $resposta=addslashes($_POST["pergunta$pergunta->id"]);
        $cadastraresposta->execute();
    else:
        $pdo->rollBack();
        die("Preencha todos os campos corretamente!<br />");
    endif;
}

Manual PHP:

http://php.net/manual/pt_BR/pdo.prepared-statements.php

http://php.net/manual/pt_BR/pdostatement.bindparam.php

4
  • Obrigado pela dica mas infelizmente não obtive sucesso. Preciso utilizar o foreach por se tratar de um formulário com adição de campos dinamicamente, cadastrados no banco de dados... Commented 10/08/2014 às 17:21
  • 1
    Eu não retirei o foreach, apenas retirei o bindparam de dentro dele, consulte o manual o bind não precisa repetir... Commented 10/08/2014 às 22:08
  • Mas o bind não teria que vir depois da variável? Lembrando que $resposta passa mais de um resultado por causa do post dinâmico... Commented 10/08/2014 às 22:30
  • 1
    Não, você especifica que quer escutar aquela variável para aquele parâmetro, então se ela mudar de valor, o comando $cadastraresposta->execute(); vai pegar o valor atual da variável, Commented 10/08/2014 às 22:41
1

Vejo alguns problemas no código.

  1. O Jader está correto, o bindParam deveria ser necessário somente uma vez, fora do foreach, do contrário, nem faria sentido ter uma variável extra e esta função não seria desta forma;

  2. Não acho muito seguro escrever "pergunta$pergunta->id", pode ser que o PHP use o $pergunta como variável e continue com o texto "->id".

  3. Se o índice de array não existe no array $_POST uma chamada retorna null?

Sugestões:

  1. Aceitar a sugestão do Jader;

  2. Use uma outra variável para o bind do id da pergunta. Pode não ser necessário, mas me parece mais simples;

  3. Reescreva os índices de $_POST como "pergunta${perguntaId}" ou algo parecido, que pode ser mais confiável;

  4. Nossa, nem sabia que o if podia ser escrito assim. O tradicional não é mais legível e com maior suporte dos editores de código?

  5. Eu acho que ao utilizar bind, as strings não precisam ser tratadas com addslashes, porém você deve testar esta parte pois não tenho certeza;

  6. Vou deixar o teste por null, mas pode ser necessário mudar, não lembro de como é de fato.

  7. Verifique a variável $name;

  8. Eu, particularmente, gosto de inicializar as variáveis antes do uso;

  9. Jader observou a falta de delimitadores de string na query, substituindo para 'name'.

A parte em questão do código ficaria assim:

// Validação dos campos dinâmicos
$cadastraresposta=$pdo->prepare("INSERT INTO form_respostas (perguntaid, username, reposta) VALUES (:perguntaid, '$name', :resposta)");
$perguntaId = 0;
$resposta = 0;
$cadastraresposta->bindParam(":perguntaid", $perguntaId, PDO::PARAM_INT);
$cadastraresposta->bindParam(":resposta", $resposta);
foreach ($listarpergunta as $pergunta) {
  $perguntaId = $pergunta->id;
  $resposta = $_POST["pergunta${perguntaId}"];
  if ($resposta != null) {
    $cadastraresposta->execute();
    if ($cadastraresposta->errorCode() != 0) {
      print_r($cadastraresposta->errorInfo());
      die("Erro ao executar Query!");
    }
  } else {
    $pdo->rollBack();
    die("Preencha todos os campos corretamente!<br />");
  }
}
// Todos os arquivos foram preenchidos corretamente
//$pdo->commit();
echo "Obrigado!";

Se não funcionar, tente ver o que tem de errado com o PDO::errorCode ou PDO::errorInfo.

6
  • 1
    sobre o #4, é uma forma muito usada em views, por ser mais fácil encontrar o inicio e fim do que {...}... #8, $perguntaId = 0 definir como zero não faz sentido algum Commented 10/08/2014 às 22:31
  • Como o Papa Charlie disse, também não entendi a definição da variável como 0, mesmo assim não funcionou. Dei um echo nas variáveis do post e elas retornam o valor corretamente, não sei porque não está inserindo... Quem sabe alguma forma melhor que o foreach para fazer isso? Commented 10/08/2014 às 23:48
  • Verifique na query onde você tem ,$name, acho que deve ser ,'$name', Commented 11/08/2014 às 2:01
  • 1
    Sua frase "gosto de inicializar" me lembrou quando programava em ASP. Não me referi exclusivamente ao bindParam, mas sim à declaração de quaisquer variaveis com atributo 0 ou null antes de se inserir um valor real. Commented 11/08/2014 às 4:56
  • 1
    @MateusMelo Verifique a observação do Jader sobre limitar a string $name no SQL, verifique também se ocorre erro na query, adicionei o código para verificar na resposta. Há alguma saída (qualquer) no html?
    – fbiazi
    Commented 11/08/2014 às 4:58
-2

O post é antigo mas deve ajudar no futuro; A solução é utilizar o bindValue ao invés do bindParam.

1
  • 3
    Olá user3474474, você poderia dar um exemplo concreto de código? Commented 9/09/2016 às 8:42

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 .