Discussion :

[laurentSc]

Bonsoir,

mon code nécessite d'exécuter 2 fois la méthode execute sur la même requête SQL ; je pense que ce n'est pas top en matière de performance mais ne sais pas comment l'éviter. Quelqu'un pourrait-il me le dire ? Voici le code actuel :

Code php :

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
<?php
$sql="SELECT * FROM categories ORDER BY categorie";
$qid = $bdd->prepare($sql);
$qid->execute();	
 
 $nb_lignes=0;
 while ($ligne=$qid->fetch(PDO::FETCH_ASSOC))	
		$nb_lignes++;
  if ($nb_lignes > 0){
	?>
		<table>
		<tr>
		<td>Sélectionner la catégorie</td>
		<td>
    		<form action="index.php?page=knowledge/knowledge_accueil?okcat" method="POST">
			<select name="selectedcat">
			<?php
				$qid->execute();
				for ($i=0;$i<$nb_lignes;$i++) {		
					$ligne=$qid->fetch(PDO::FETCH_OBJ);	
					$t = $ligne->categorie;
			?>
					<option><?php echo $t; ?> </option>
			<?php } ?>
		        </select>
		</td>
		<td><input type="submit" value="Ok"></td>		
		</form>
		</td></tr></table>
<?php
	}
	else
	{
		echo "Aucune catégorie trouvée dans la base !<br>";
	}

[sabotage]

Stocke le resultat de la requête dans une variable PHP et réutilise ce résultat au lieu de re-executer la requête.

[Bovino]

Ou mieux : sépare la collecte des données de la gestion de l'affichage.
Sinon, tu pourrais expliquer l'utilité d'une requête préparée dans le cas que tu montres ?

[laurentSc]

Excuse-moi, mais qu'est-ce que ça signifie séparer la collecte des données et l'affichage ?

Sinon, ici, une requête préparée, vachement utile, c'est juste le résultat d'un copier-coller (en fait, chez moi, l'exécution d'une requête avec PDO est toujours préparée, mais c'est vrai que dans ce cas, il suffisait de faire :

Code php :

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

1
2
3
 
$sql="SELECT * FROM categories ORDER BY categorie";
$qid=$bdd->query($sql);

[Bovino]

C'est une question assez similaire à ce que l'on essaye de faire dans les parties statiques d'un site : séparer la structure de la page (HTML) de la présentation (CSS) et de l'interactivité (JavaScript).
Dans ton projet PHP, c'est pareil, il faut éviter d'avoir de l'affichage en plein milieu du traitement d'une requête.
Essaye plutôt de récupérer les données à afficher dans des variables lors du traitement et ensuite de gérer l'affichage en utilisant ces variables.

[laurentSc]

Merci, voici ce que j'ai fait :

Code php :

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
 
	$sql="SELECT * FROM categories ORDER BY categorie";
	$qid=$bdd->query($sql);
 
	$nb_lignes=0;
 
	 while ($ligne=$qid->fetch(PDO::FETCH_OBJ)){	
			$nb_lignes++;
			$t[]=$ligne->categorie;
			}
	 if ($nb_lignes > 0)
	{
	?>
		<table>
		<tr>
		<td>Sélectionner la catégorie</td>
		<td>
    		<form action="index.php?page=knowledge/knowledge_accueil?okcat" method="POST">
			<select name="selectedcat">
			<?php
				for ($i=0;$i<$nb_lignes;$i++) {		
			?>
					<option><?php echo $t[$i]; ?> </option>
			<?php } ?>