Discussion :

[link.80]

Bonjour,

J'ai une petite question à vous poser, je ne sais pas ce qu'il est mieux de faire d'un point de vue performances pour un système tout simple de news qui peuvent être rattachées à des images.

J'ai une table contenant mes news (ID, TITRE, TEXTE)
ainsi qu'une table contenant mes images (ID, URL, ID_NEWS)

Actuellement je fais comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	$retour_messages=mysql_query("SELECT * FROM news ");
	while($result=mysql_fetch_assoc($retour_messages)) {		
				$aff.='<article>';
				$aff.='<p>'.getdateSQL($result["PDATE"]).' - <strong>'.$result["TITRE"].'</strong></p>';
				$aff.=$result["TEXTE"];
					$r=mysql_query("SELECT * FROM news_images WHERE ID_GALERIE=".$result["ID"]."");
					while($img=mysql_fetch_assoc($r)) {
					  $aff.='<img src="upload/news/'.$img["URL"].'">';
					}
				$aff.='</article>';			
	}

D'un niveau performances, je me suis dit que je pourrai utiliser une jointure pour éviter la boucle dans la boucle…

Voici donc ma requête :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
$retour_messages=mysql_query("
	SELECT * FROM news JOIN 
	news_images ON news.ID = news_images.ID_GALERIE
	WHERE news.PUBLIE=1 
	GROUP BY news.ID");
 
	while($result=mysql_fetch_assoc($retour_messages)) {	
				$aff.='<article>';
				$aff.='<p>'.getdateSQL($result["PDATE"]).' - <strong>'.$result["TITRE"].'</strong></p>';
				$aff.=$result["TEXTE"];
				 $aff.='<img src="upload/news/'.$result["URL"].'">';
				$aff.='</article>';			
	}

Seulement avec ce code j'ai uniquement la première image qui s'affiche, il faudrait boucler sur les images en ayant le total du nombre de résultats trouvé ?

Quelle serait selon vous la meilleure solution d'un point de vue performances ?

ça serait intéressant d'avoir votre avis à ce sujet,

D'avance merci et salutations

[skuatamad]

C'est bien par une jointure qu'il faut passer, par contre le GROUP BY n'a rien à faire dans la requête.
Par ailleurs le * doit être remplacé par la liste des colonnes à sélectionner.

Je vous conseille la lecture des tutoriels du forum sur l'apprentissage du SQL :
http://sql.developpez.com/#apprendre-sql

[link.80]

Merci pour la réponse, mais si j'enlève le GROUP BY, ma boucle affiche une fois chaque photo en doublant les entrées de la table news (Une news peut contenir plusieurs photos).

[link.80]

personne pour m'aider ?

[punkoff]

Personnellement je penses que je garderai votre 1ere solution.

Vous avez une colonne de type "TEXT" dans votre 1ere table, et dupliquer le retour de cette colonne, à cause de la jointure, me semble pas tip top.


Je mettrai plutot une limitation sur le nombre de news affichable en l'état ... les chargerai au fur et a mesure que l'utilisateur clique un bouton "afficher plus" par exemple.