Optimisation d'une boucle

Version imprimable

Bonjour,

Je fait une page où j'affiche les données de la base de données.
J'aimerais ajouter une ligne à chaque entrée dans la base de données, c'est pour cela que j'ai fait une boucle

Code:

for

et pas faire un tableau en entier à chaque entrée.

Voici la requête SQL :

Code:

$query_selection_devoirs_eleves = mysql_query("SELECT matiere_cours_devoirs_eventuels,classe_cours_devoirs_eventuels,date_depot_cours_devoirs_eventuels,date_devoirs_eventuels,resume_cours_devoirs_eventuels,nom_auteur_devoirs_eventuels FROM test_site_questions,test_site_utilisateur WHERE pseudo_utilisateur='$pseudo_utilisateur' AND type_devoir='3'")or die ('Erreur lors de la requête SQL'.mysql_error());

Voici le code (boucle) que j'ai fait :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 echo '<table width="100%" border="1">'; echo '<tr><td colspan="2">Cahier de texte de la Classe :'.$donnees_devoirs_eleves['classe_cours_devoirs_eventuels'].'</td></tr>'; while ($donnees_devoirs_eleves = mysql_fetch_array($query_selection_devoirs_eleves)) { $nombre_de_devoirs = mysql_num_rows($query_selection_devoirs_eleves); for ($i = 1; $i < $nombre_de_devoirs; $i++) { echo '<tr><td width="60">Date de dépot'.$donnees_devoirs_eleves['date_depot_cours_devoirs_eventuels'].'<br />Date (pour quand)'.$donnees_devoirs_eleves['date_devoirs_eventuels'].'</td><td>Matière : '.$donnees_devoirs_eleves['matiere_cours_devoirs_eventuels'].html_entity_decode($donnees_devoirs_eleves['resume_cours_devoirs_eventuels']).$donnees_devoirs_eleves['nom_auteur_devoirs_eventuels'].' </td></tr> '; } } echo '</table>';
Ce code à l'air de fonctionner mais j'ai mis d'entrée

Code:

i=1

dans la boucle

Code:

for

.

Qu'est-ce que vous en pensez ?

A mon avis, il y a (peut-être) des améliorations à faire...

Cordialement

13/09/2013, 11h17
sabotage

Je ne vois pas à quoi sert ta boucle for.
13/09/2013, 11h31
vinceom92

Je t'explique ce que je voudrais faire avec ce code :

Je voudrais faire un code qui permettrait de rajouter les données issues de la base de données dans ce cahier de texte pour éviter de faire un tableau dès qu'il y a un nouveau enregistrement.

Je précise que ce code à l'air de marcher mais ce qui m'étonne, c'est que si je mets

Code:

for ($i = 1; $i < $nombre_de_devoirs; $i++)

, ça à l'air de fonctionner alors que si je mets

Code:

for ($i = 0; $i < $nombre_de_devoirs; $i++)

, ça m'affiche les données en double.

Pourquoi ?

Voilà, je me pose cette question !!!
13/09/2013, 11h55
sabotage

Je ne vois toujours la logique de ce que tu fais.

mysql_fetch_array renvoit 1 ligne de résultat.

ta boucle for à l'intérieur du while conduit forcemment à la répétition de cette ligne plusieurs fois.

je suppose que $nombre_de_devoirs = 1 donc quand $i = 1, ta boucle for ne fait rien
et quand $i = 0 ta boucle for tourne deux fois.
13/09/2013, 13h36
vinceom92

En fait, c'est un cahier de texte pour les élèves : quand un professeur met un devoir pour telle date, ça l'affiche sur cette page.
A chaque enregistrement d'un devoir, je n'insère pas un tableau mais j'ajoute des lignes pour une question de présentation.

Voilà.
13/09/2013, 13h49
sabotage

Donc c'est bien ce que je dis : ta boucle for ne sert à rien.
13/09/2013, 14h37
vinceom92

Qu'est-ce que tu proposes ?
13/09/2013, 14h46
sabotage

de la retirer
13/09/2013, 14h58
vinceom92

Peux-tu me donner ta version comme exemple STP ?

La même chose sans le for :

Code:

1
2
3
4
5
6
while ($row = mysql_fetch_array($query_selection_devoirs_eleves)) {
	echo '<tr>
			<td width="60">Date de dépot'.$row['date_depot_cours_devoirs_eventuels'].'<br />Date (pour quand)'.$row['date_devoirs_eventuels'].'</td>
			<td>Matière : '.$row['matiere_cours_devoirs_eventuels'].$row['resume_cours_devoirs_eventuels'].$row['nom_auteur_devoirs_eventuels'].'&nbsp;</td>
		</tr>';
};

Bonjour,

Merci de ta réponse.

Par contre, j'ai une autre question : s'il y a aucun enregistrement (au début), comment afficher un message (par exemple : Aucune donnée trouvée.) sachant que pour éviter que ça le fasse plusieurs fois, j'ai mis
Code:

1 2 3 4 5 6 echo '<table width="100%" border="1">'; echo '<tr><td colspan="2" class="titre">Cahier de texte de la Classe : '; while ($donnee_selection_classe_eleve = mysql_fetch_array($query_selection_classe_eleves)) { echo "<div class='classe'>".$donnee_selection_classe_eleve['classe_cours_devoirs_eventuels']."</div>"; } '</td></tr>';
avant mes deux boucles.

Actuellement même s'il y a aucune donnée trouvée, le début du tableau s'affiche sans le nom de la classe ce qui est logique.

En une question : comment ne pas afficher le début du tableau quand il y a aucun enregistrement trouvé et afficher un message du type Aucune donnée trouvée. ?

Cordialement

14/09/2013, 15h31
ypcman

Bonjour.
Il me semble que tu n'as qu'a coder ce que tu écris !
Un if sur la valeur du nombre d'enregistrement. Si c'est zéro, tu affiches ton message, sinon tu rentres dans le while
Tu as déjà presque tout codé :

Code:

$nombre_de_devoirs = mysql_num_rows($query_selection_devoirs_eleves);