Discussion :

[Herman]

Bonjour,

Ci dessous une requête SQL qui semble lourde et possible d'améliorer. Actuellement à chaque visite du site elle lit 209686 lignes pour en sortir 10.

Code :

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

1
2
3
4
5
6
7
8
$sql_query  = "SELECT news.*,categorie.icone,count(com_news.id_news),categorie.nom_jeu FROM `news` ";
$sql_query .= "LEFT JOIN affiche ON news.id = affiche.id_affiche_item ";
$sql_query .= "LEFT JOIN categorie ON categorie.id_categorie = news.categorie ";
$sql_query .= "LEFT JOIN com_news ON news.id = com_news.id_news ";
$sql_query .= "WHERE id_pageune='0' ";
$sql_query .= "AND (publier=1 OR publier=5)  ";
$sql_query .= "ORDER BY date  ";
$sql_query .= "DESC LIMIT 0,10  ";

N'ayant aucune idée de comment l'améiorer, je me tourne vers vous : ))

Merci d'avance !

[ced]

Bonjour,

Il va falloir nous donner quelques indications : la structure des tables, le but de cette requête et, éventuellement, quelques données exemple...
Le premier problème que je vois dans cette requête, c'est qu'elle a un COUNT et pourtant aucun GROUP BY... Si MySQL permet cette "entorse" à la norme SQL, il vaut mieux éviter de l'employer (même à bon escient, je pense), parce qu'il n'y a pas de garantie sur les autres informations associées au COUNT.

Bref, que doit faire cette requête ?

ced

[Herman]

Récupérer des news pour les afficher.

Pour le moment je m'en sort avec ça :

Code :

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

1
2
3
4
5
6
$sql_query  = "SELECT news.*,categorie.icone,categorie.nom_jeu FROM `news` ";
$sql_query .= "LEFT JOIN categorie ON categorie.id_categorie = news.categorie ";
$sql_query .= "WHERE id_pageune='0' ";
$sql_query .= "AND (publier=1 OR publier=5)  ";
$sql_query .= "ORDER BY date  ";
$sql_query .= "DESC LIMIT 0,10  ";

et le count je le fait plus tard puisque ça ajoutait une jointure, et que ça comptait les commentaires de TOUTE les news, pour au final n'en prendre que 10. Donc je prends les 10 avant de compter les commentaires maintenant.

Au final je ne pense pas qu'on puisse faire mieux non ?

[ced]

Si, on peut améliorer encore une condition :

Code :

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

AND (publier=1 OR publier=5)

A remplacer par :

Code :

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

AND publier IN (1, 5)

ced

[Herman]

ok nikel, merci