Discussion :

[Florian_mbf]

Bonjour à tous !!

Voici ma fonction php qui me permet de remplir une liste déroulante grâce à deux requetes :
La premiere permet de récupérer les identifiants des formateurs que je passe dans une liste afin de l'utiliser dans ma seconde requete.
Elle retourne ce que je veux ( j'ai déjà effectué des tests sur phpMyAdmin),
la deuxieme retourne également ce que je veux sur phpMyAdmin mais dans ma liste déroulante, seule une valeur est affectée et là j'avoue que je sèche totalement à une possible erreur ou incompréhension de ma part, c'est pour cela que je vous demande un petit coup de main pour me dire ce qui cloche dans cette function ou tout simplement chez moi ^^.
Thanks people !

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
<?php
include_once("connexion.php");
 
 
$conn = new PDO("mysql:host=$hostdb; dbname=$namedb", $userdb, $passdb, array(PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION));
$conn->exec("SET CHARACTER SET utf8");      // Sets encoding UTF-8
 
$id_sess=$_COOKIE['cookie'];
 
function remplirListFormateur(){
 
global $id_sess, $conn;
	try {
		// Define and perform the SQL SELECT query
		$reponse = $conn->query("SELECT sess_formateur, sess_formateur2, sess_formateur3, sess_formateur4, sess_formateur5 FROM sess WHERE sess_id = '".$id_sess."'");
 
		while ($donnees = $reponse->fetch()){
			$idFormateurs = $donnees['sess_formateur'].','.$donnees['sess_formateur2'].','.$donnees['sess_formateur3'].','.$donnees['sess_formateur4'];
		}
		$reponse->closeCursor();
 
		$sql="SELECT formateur_id, formateur_nom, formateur_prenom FROM formateur WHERE formateur_id IN ('".$idFormateurs."')";
		$result = $conn->query($sql);  
		?>
		<select name="list_formateur_cours">
		<option value="0">Sélectionner un formateur</option>
		<?php
 
		while ($row = $result->fetch(PDO::FETCH_ASSOC))
		{ ?>
		<option value="<?php echo $row['formateur_id']?>"><?php echo $row['formateur_prenom'].' '.$row['formateur_nom']; ?></option>    
 
 
		<?php}?>
		</select>
		<?php
		$result->closeCursor();
 
	} catch(PDOException $e) {
		exit('<b>Catched exception at line '. $e->getLine() .' :</b> '. $e->getMessage());
	}
 
}

[sabotage]

Commence par afficher ta requête pour voir ce que tu executes réellement.

[Florian_mbf]

J'y travaille, je mets une solution dès que je l'a trouve pour ceux ou celles qui en auraient besoin un jour, ça bloque au niveau de la condition IN ..

[Florian_mbf]

En faite, la requete merdoullait car lorsque qu'un formateur n'etant pas present les virgules dans la premiere requete était tout de meme inséré, de ce fait j'ai fait comme ceci :

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
function remplirListFormateur(){
	global $id_sess, $conn;
		try {
			// Define and perform the SQL SELECT query
			$reponse = $conn->query("SELECT sess_formateur, sess_formateur2, sess_formateur3, sess_formateur4, sess_formateur5 FROM sess WHERE sess_id = '".$id_sess."'");
 
			while ($donnees = $reponse->fetch()){
				$form1 = $donnees['sess_formateur'];
				$form2=$donnees['sess_formateur2'];
				$form3=$donnees['sess_formateur3'];
				$form4=$donnees['sess_formateur4'];
				$form5=$donnees['sess_formateur5'];
			}
			$list=$form1;
			if(!empty($form2)){ $list .=", ".$form2."";}
			if(!empty($form3)){ $list .=", ".$form3."";}
			if(!empty($form4)){ $list .=", ".$form4."";}
			if(!empty($form5)){ $list .=", ".$form5."";}
 
			$reponse->closeCursor();
 
			$sql="SELECT formateur_id, formateur_nom, formateur_prenom FROM formateur WHERE formateur_id IN (".$list.")";
			$result = $conn->query($sql);  
			while ($row = $result->fetch(PDO::FETCH_ASSOC))
			{ ?>
				<option value="<?php echo $row['formateur_id']?>"><?php echo $row['formateur_prenom'].' '.$row['formateur_nom']; ?></option>    
			<?php
			}
 
			$result->closeCursor();
 
		} catch(PDOException $e) {
			exit('<b>Catched exception at line '. $e->getLine() .' :</b> '. $e->getMessage());
		}
 
}

[rawsrc]

Salut,

tu peux grandement simplifier et sécuriser ton code :

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
<?php
 
function remplirListFormateur()
{
    global $id_sess, $conn;
    try
    {
        // Define and perform the SQL SELECT query
        $sql = 'SELECT sess_formateur, sess_formateur2, sess_formateur3, sess_formateur4, sess_formateur5 FROM sess WHERE sess_id = '.intval($id_sess);
        $qry = $conn->query($sql);
        // tu ne récupères qu'un seul enregistrement, le while est inutile
        // on supprime les données vides
        $rows = array_filter($qry->fetch());
        $list = implode(', ', $rows);
 
        $sql = "SELECT formateur_id, formateur_nom, formateur_prenom FROM formateur WHERE formateur_id IN ({$list})";
        $qry = $conn->query($sql);
 
        // fonction d'échappement à l'affichage
        $hsc = function($p) { return htmlspecialchars($p, ENT_QUOTES, 'utf-8'); };
 
        while ($row = $qry->fetch(PDO::FETCH_ASSOC))
        {
            echo
<<<HTML
<option value="{$hsc($row['formateur_id'])}">{$hsc($row['formateur_prenom'])} {$hsc($row['formateur_nom'])}</option>    
HTML;
        }
    }
    catch(PDOException $e)
    {
        exit('<b>Catched exception at line '. $e->getLine() .' :</b> '. $e->getMessage());
    }
}

Je pense que ta table est très mal conçue : 5 colonnes comme celles-là : sess_formateur, sess_formateur2, sess_formateur3, sess_formateur4, sess_formateur5 ! Ouch et tu vas modifier ta table à chaque fois que tu auras besoin d'un sess_formateurX ?
C'est ingérable. Tu devrais reprendre ça...