Comparaison avec PDO

**Sltcmi** · 07/01/2016, 09h14

Bonjour,

Je ne suis pas certain de devoir poster cela ici ou dans la section PDO...

Je cherche à "contrôler" une insertion dans la base de donnée -> J'ai une table compétence_personnel dans laquelle j'insert l'id_Personnel, l'id_Competence, le niveau, et si le salarié en question est en formation ou non. Jusque là ok. - Le soucis c'est que j'aimerais qu'il ne soit pas possible d'insérer une deuxième fois la même compétence pour le même salarié. Exemple : J'insert l'id_Personnel 119, que j'associe à l'id_Competence 19, je refais la même manipulation, c'est-à-dire je ré-insert les mêmes données sauf que je souhaite retourner une erreur du style "compétence déjà acquise / pas possible". Je bloque à ce niveau, j'ai testé plusieurs choses différentes.

l'id_Competence ($idComp), et l'id_Personnel($idSal1) sont "générés" dans un input type="hidden" avec de l'ajax. Apparemment quand je var_dump($request['id_Competence']) et var_dump($request['Personnel']) ça me retourne "null".

Mon code atm :

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
 
if(isset($_POST['idSal1']) && isset($_POST['titreComp']) && isset($_POST['niveau']) && isset($_POST['formation']) && isset($_POST['nomSal']) && isset($_POST['idComp'])) {
 
    $idSal1 = htmlspecialchars($_POST['idSal1']);
    $idComp = htmlspecialchars($_POST['idComp']);
    $niveau = htmlspecialchars($_POST['niveau']);
    $formation = htmlspecialchars($_POST['formation']);
    $test = false;
    $maRequete = $db->query("SELECT * FROM competence_personnel WHERE id_Personnel = '".$idSal1."'");
 
    if(!empty($_POST['idSal1']) && !empty($_POST['idComp']) && $_POST['niveau'] != "" && $_POST['formation'] != "" ) {
 
        while($request = $maRequete->fetch() && $test == false) {
 
            if($request['id_Competence'] != $idComp && $request['id_Personnel'] != $idSal1) {
                var_dump($request['id_Competence']);
                var_dump($idComp);
                $test = true;
                echo '4';
            }
        }
 
        if($test == true) {
 
            $insert = $db->prepare('INSERT INTO competence_personnel (id_Personnel, id_Competence, niveau, formation) 
                                        VALUES(:id_Personnel, :id_Competence, :niveau, :formation)');
            $insert->execute(array(
                'id_Personnel'=>$idSal1,
                'id_Competence'=>$idComp,
                'niveau'=>$niveau,
                'formation'=>$formation
                ));
            echo "<script>alert(\"Vos données ont été enregistrées\")</script>";   
        }
 
        else {
 
            echo "<script>alert(\"Compétence déjà acquise\")</script>";    
        }
    }
 
    else {
 
        echo "<script>alert(\"Le formulaire n'est pas complet\")</script>";    
    }
}
 
 
?>

Si quelqu'un a une idée pour me mettre sur la voie :-)

Merci!

**sabotage** · 07/01/2016, 10h06

Il faut que tu conçoive tes traitement avec plus de simplicité
- tu cherches un couple personnel/competence alors pourquoi tu ne mets que personnel dans ta requête SELECT ?
- htmlspecialchars, comme son nom l'indique, traite de HTML, pas d'insertion dans une base de données
- il faut uniformiser tes nommages : pourquoi "id_Personnel" devient "idSal1" quand on est dans le formulaire

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
21
22
<?php
if (isset($_POST['idSal1'], $_POST['titreComp'], $_POST['niveau'], $_POST['formation'], $_POST['nomSal'], $_POST['idComp'])) {
 
    $sth = $db->prepare("SELECT COUNT(*) FROM competence_personnel WHERE id_Personnel = :id_Personnel AND id_Competence = :id_Competence");
	$sth->execute(array(':id_Personnel'=>$_POST['idSal1'], ':idCompetence'=>$_POST['idComp']));
	$row = $sth->fetch(PDO::FETCH_NUM);
	if ($row[0] === 0) {
		$insert = $db->prepare('INSERT INTO competence_personnel (id_Personnel, id_Competence, niveau, formation) 
                               VALUES(:id_Personnel, :id_Competence, :niveau, :formation)');
		$insert->execute(array(
                'id_Personnel'=>$_POST['idSal1'],
                'id_Competence'=>$_POST['idComp'],
                'niveau'=>$_POST['niveau'],
                'formation'=>$_POST['formation']
                ));
            echo "<script>alert(\"Vos données ont été enregistrées\")</script>";   
	} 
	else {
 
		echo "<script>alert(\"Compétence déjà acquise\")</script>";    
	}
}

**Sltcmi** · 07/01/2016, 10h23

Hello,

merci d'avoir pris le temps de répondre, j'ai essayé ton code et ça m'a donné l'erreur :

Warning: PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number:", manquait juste un "_" à idCompetence dans l'execute();

Tu peux m'expliquer le if($row[0] === 0) ? ça marche seulement si je change par if($row[0] == 0)

Merci :-)

Edit : Grâce à ton code j'ai remarqué plusieurs mauvaises habitudes, comme la répétition des isset(), etc, pareil pour les remarques. Merci ça m'évitera de répéter les mêmes erreurs :-)

**sabotage** · 07/01/2016, 10h33

=== est une comparaison valeur et type au lieu de valeur seule
Si ça ne marche pas c'est qu'a priori ton pilote renvoit "0" au lieu de 0.

J'ai oublié une précision : ta table doit contenir un index "UNIQUE" avec les deux colonnes id_Personnel et id_Competence. Ainsi il sera techniquement impossible d'avoir des doublons.