Eviter d'exécuter 2 fois de suite la méthode PDO execute

Version imprimable

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:

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>";
	}

17/11/2014, 07h20
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.
17/11/2014, 07h57
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 ? :mrgreen:
17/11/2014, 09h21
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:

1 2 3 $sql="SELECT * FROM categories ORDER BY categorie"; $qid=$bdd->query($sql);
17/11/2014, 09h32
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.

Merci, voici ce que j'ai fait :

Code:

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 } ?>