exécution trop lente de requèteRésolu

Question

Bonjour, 
ma page de cesse de tourner quand j'exécute ma requête .
il y a t il une autre méthode ??
                $sql = "SELECT *            FROM victime V              ,prejudicemoraux PM              ,agent AG              ,tb_departement DPTD             ,tb_sous_prefecture SPD              ,tb_commune COMD             ,tb_localite LCD             ,prejudicemateriel PMAT              ,infogenerale IG                       ,tb_parent PWHERE $critere                        AND   V.matricule  = PM.id_prejM            AND   PM.id_prejM  = AG.id            AND   AG.id  = DPTD.id            AND   DPTD.id  = SPD.id            AND   SPD.id  = COMD.id            AND   COMD.id  = LCD.id            AND   LCD.id  = PMAT.id_prejMat            AND   PMAT.id_prejMat  = IG.id_infoG            AND   IG.id_infoG  = P.id ORDER BY V.nomV             ";

jordane45 · Answer

Bonjour,

Oui... en passant par des jointures INNER JOIN ou LEFT JOIN..
Et puis.. tout dépend aussi de ce que contient ta variable $criteres...

Par contre, es-tu sûr que le souci vienne de la requête ( tu l'as testé en direct dans ta BDD ?? ) ou de son traitement dans son code php ??

PS: Pourquoi.... alors que tu poses une question sur une REQUETE SQL ... tu poses ta question dans PHP ????
(ce n'est pas la première fois que je te fais cette remarque ! )

msi79 · Answer

en direct exécution de la requête donne rien

mpmp93 · Answer

Bonjour,

je connais ce genre de problème.....

D'une part, si la volumétrie de données est importante, il faut déja limiter le nombre de fiches. Donc faire un LIMIT, exemple:

LIMIT 0,20

à rajouter dans le code SQL va limiter à 20 fiches....
 

Ensuite, piège très grossier dans lequel beaucoup tombent, même des pros en BDD, c'est oublier d'indexe les champs sur lesquels on fait des WHERE.

Par exemple, dans votre code SQL, je vois ceci:

WHERE $critere  

Si un des critères est par exemple date_des_faits, il faut que ce champ soit indexé.

Enfin, je vois que vous utilisez des alias de noms de tables:

FROM victime V              ,prejudicemoraux PM              ,agent AG              ,tb_departement DPTD             ,tb_sous_prefecture SPD              ,tb_commune COMD             ,tb_localite LCD             ,prejudicemateriel PMAT              ,infogenerale IG                       ,tb_parent P


C'est très bien pour la lisibilité, mais il faudrait gérer les jointures entre ces tables avec INNER JOIN et les index primaires.

Attention, trop de jointures "flinguent" les performances, voire déconnectent des données.

Je serai à votre place, je ferai ceci:

SELECT *
            FROM victime V
WHERE $critere   
 ORDER BY V.nomV
LIMIT 0,20

afin de sortir 20 fiches respectant vos critères (champs indexés évidemment). Ensuite, vous affichez le minimum syndical à l'écran et ensuite, quand on veut voir une fiche, là vous allez piocher dans les différentes tables ce qui est nécessaire pour voir la fiche en détail.

Performance maximale assurée...

A+

jordane45 · Answer

@mpm :

Indexer les champs : Oui (j'avais oublié de lui dire).

Utiliser les Jointures : Oui (je lui ai dit)

Utiliser des "LIMITES" .. mouais... pourquoi pas... Mais dans ce cas.. il est plus propre d'utiliser de la pagination... mais là.. c'est un autre sujet.

Ensuite, vous affichez le minimum syndical à l'écran et ensuite, quand on veut voir une fiche, là vous allez piocher dans les différentes tables ce qui est nécessaire pour voir la fiche en détail. 
[...]
Performance maximale assurée... 
Pas nécessairement... tout dépend de ce qu'il veut avoir à l'écran....et comment il l'affiche.
Si il découpe son affichage en plusieurs "vues" du genre :
Un tableau contenant le strict minimum ... puis si on clique sur la ligne d'un ticket.. on affiche les infos de se ticket (et donc.. seulement à ce moment là on va requêter la BDD pour en récupérer le contenu...) là.. oui.....
Mais connaissant MSI et  sa façon de coder et la manière dont il présente le contenu de ses pages... à mon avis.. ça ne lui correspondra pas...


Sans oublier ...Pour gagner en performance.. il faut aussi limiter les  SELECT *  et ne faire un SELECT  que des champs nécessaires...

Et enfin... tout dépend également de ce qu'il a dans son WHERE.
Si le WHERE contient des conditions sur des champs TEXTE (ou varchar).. la requête sera forcément plus longue que sur des champs numériques....

Bref...

Déjà... il faut, pour s'assurer que le souci vient de la requête, et non pas de son traitement lors de l'affichage des données dans la page PHP.. qu'il teste sa requête en direct dans la BDD ...
( 1 - Faire un ECHO de la requête dans la page PHP
 2 - Prendre cette requête ainsi générée et la tester directement dans la BDD
 3 - Regarder combien de temps elle met....)
=> Perso.. si elle met plus de 2 voir 3 secondes (et encore.. je suis large)... c'est clair que la lenteur vient d'elle.....
=> Sinon... le souci vient du traitement dans la page !

msi79 · Answer

bonjour @mpmp93. merci de bien vouloir m'aider .
au fait au départ c'est ce que j'avais fait :
SELECT *            FROM victime VWHERE $critere    ORDER BY V.nomVLIMIT 0,20
mais il se trouve que quand je dois afficher par exemple la liste des blessés ou le critère se trouve dans une autre tables j'arrive pas .

jordane45 · Answer

OUI J'AI TENTE CA MARCHE PAS JE CROIS QUE JE ME MÉLANGE VU QUE IL Y A TROP DE TABLES
Et tu penses que si tu ne nous montres pas ce que tu as essayé de faire que nous allons pouvoir t'aider à corriger ??

Nb : juste pour info :
Pour créer/tester ses requêtes...il faut le faire en direct dans la BDD (pas dans ta page PHP).... 
 

Je te donne la première jointure :

SELECT * 
 FROM victime V
 LEFT JOIN prejudicemoraux PM ON  V.matricule  = PM.id_prejM

A toi de compléter...

NB² : Pense que L'ordre des Jointures dépend de la structure de ta table et des infos que tu souhaites avoir dans ta requête...donc commence par tester une première jointure... puis si elle te donne le résultat attendu..ajoute la secode..etc...


Avant de poser une question, merci de lire la charte du site.
Cordialement, Jordane

Exécution trop lente de requète

6 réponses

Votre réponse

Discussions similaires