Discussion :

[flavors]

Bonjour.


J'essaie de faire un ajout dans une BDD avec une condition mais je rencontre un problème.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
 
$requeteElements = $pdo->prepare('INSERT INTO Elements (...) VALUES(:...)');
$requetVerifElements = $pdo->prepare('SELECT COUNT(...) FROM Elements WHERE nom=:nom');
 
$nbElements = count($listeElements);
for($i=0 ; $i<$nbElements ; $i++) {
    $cElem = $listeElements[$i][0];
 
     $requetVerifElements -> bindParam(':nom'=>$cElem["nom"]);
    $return = $requetVerifElements -> execute();
    $data = $requetVerifElements->fetch();
 
    if($return == FALSE)
        { break 2; }
    elseif($data == 0)
        { $return = $requeteElements -> execute(array(...)) or die("Erreur SQL!"); }
    else
        { $test = 1; }
}

Si j'ai le retour de mon comptage qui est faux, je sors.
Sinon, si l'élément n'existe pas déjà, je l'ajoute.
Sinon test =1.

Je ne comprends pas pourquoi ça fonctionne sans le else mais dés que je rajout cette condition, ça ne marche plus.
Je fais appel à ce scipt via une requête AJAX et je n'arrive pas à localiser le problème. Je sais par contre que mes requêtes SQL sont correctes.

Quelqu'un pourrait-il m'apporter son aide =)?

Merci.

[Kasko]

if($return == FALSE)
{ break 2; }
elseif($data == 0)
{ $return = $requeteElements -> execute(array(...)) or die("Erreur SQL!"); }
else
{ $test = 1; }

Selon ton premier test la variable $return est booléenne.Voilà ce que fait ton branchement conditionnel :

si $return = faux exécution du premier code sinon si $data = 0 exécution du 2ème code sinon tu initialises une variable à 1 or $data est de type différent que $return et surtout vu qu'une booléenne ne prenne que 2 valeur donc après le IF c'est le ELSE qui doit automatiquement suivre mais il ne peux pas avoir un 3ème test sur cette variable.donc si tu veux utiliser $data fais un autre IF à part.

[sabotage]

fetch retourne un tableau.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 $data = $requetVerifElements->fetch(PDO::FETCH_NUM);
if($data[0] == 0) {

Egalement ça

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$nbElements = count($listeElements);
for($i=0 ; $i<$nbElements ; $i++) {
    $cElem = $listeElements[$i][0];

s'écris proprement comme ça

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
foreach ($listeElements as $value) {
     $cElem = $value[0];

et ton test

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 if($return == FALSE)

signifierait que ta requête ne fonctionne pas.
ce n'est pas vraiment le sujet.

[flavors]

Merci pour vos réponses.

if($return == FALSE)
signifierait que ta requête ne fonctionne pas.
ce n'est pas vraiment le sujet.

Si on sait que notre requête SQL est correct, il est inutile de la tester en faisant ça?
Dans le cas d'un ajout on peut quand même mettre ce test? Par exemple pour pouvoir faire un retour à l'utilisateur si il essaie de rentrer un champ déjà existant en BDD?

Je rencontre le même probléme.
J'ai remplacé mon code avec ce que vous m'avez dis

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
foreach ($listeElements as $value) {
    $cElem = $value[0];
 
    $requetVerifElements -> bindParam(':nom'=>$cElem["nom"]);
    $return = $requetVerifElements -> execute();
    $data = $requetVerifElements->fetch();
 
    if($data[0] == 0)
        { $return = $requeteElements -> execute(array(...)) or die("Erreur SQL!"); }
    else
        { $test = 1; }
}

Encore une fois sans le else ça passe mais avec ça ne fonctionne pas.

Merci.

[sabotage]

Le symptome que tu décris n'est pas possible puisque ton else ne fait rien. Je pense que tu interpretes mal tes résultats.
De plus tu n'as pas mis le PDO::FETCH_NUM de mon code.

Un simple echo te montrera ce que fais ton traitement exactement :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
foreach ($listeElements as $value) {
 
    $requetVerifElements -> bindParam(':nom'=>$value[0]['nom']);
    $return = $requetVerifElements -> execute();
    $data = $requetVerifElements->fetch(PDO::FETCH_NUM);
 
    if($data[0] == 0)
        { 
            $return = $requeteElements -> execute(array(...)) or die("Erreur SQL!"); }
            echo 'nom  : ' . $value[0]['nom'] .  ' non trouvé ; count = ' . $data[0] . ' - Insertion <br/>';
    else
        { 
            $test = 1;
            echo 'nom  : ' . $value[0]['nom'] .  ' trouvé ; count = ' . $data[0] . '<br/>';    
    }
}

[flavors]

Je n'y comprends rien.

J'ai mis votre code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
if($data[0] == 0)
        { 
            echo 'nom  : ' . $value[0]['nom'] .  ' non trouvé ; count = ' . $data[0] . ' - Insertion <br/>';
        }
else
        { 
            echo 'nom  : ' . $value[0]['nom'] .  ' trouvé ; count = ' . $data[0] . '<br/>';    
        }
echo $value[0]['nom']."<br/>";
echo $data[0]."<br/>";

Donc quand je laisse le else ça ne marche pas (j'ai Status Code:500 Internal Server Error dans l'outil developper).
Par contre quand je ne l'ai pas , j'ai bien le nom et 1 ou 0 (de data[0]) qui s'affiche avec le if si la condition est remplie.

Pourtant le else est correct, je ne vois pas d'accolade manquante ou d'erreur de syntaxe...

PS : j'ai bien ajouté le fetch(PDO::FETCH_NUM)

[sabotage]

Consulte tes logs d'erreur PHP.
Erreur 500 c'est ce que donne un serveur qui n'affiche pas les erreurs.