Discussion :

[PIEPLU]

Bonjour tout le monde,

J'aurais besoin de vos précieux conseils pour optimiser une requête que j'ai fait mais qui est bien trop lourde.

Voici 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
14
15
16
SELECT E.nom_entreprise,E.id_entreprise,A.dt_avis
FROM Entreprises E
JOIN Avis A
USING ( id_entreprise ) 
JOIN (
	SELECT A.id_entreprise,MAX( A.dt_avis ) AS dt_avis
	FROM Avis A
	group by A.id_entreprise
	ORDER BY dt_avis DESC
)t
USING ( id_entreprise, dt_avis ) 
WHERE E.ville_entreprise =  'paris'
AND E.cp_entreprise =  '75015'
GROUP BY E.id_entreprise
ORDER BY dt_avis DESC 
LIMIT 0 , 30

Voici l'explication de la SQL :
J'ai deux tables. Entreprises et Avis
La première table, j'ai dedans id_entreprise, nom_entreprise, ville_entreprise
La seconde table, j'ai dedans id_avis, id_entreprise, dt_avis (date de l'avis)

Le but de la requête, c'est d'afficher les avis les plus récent (dt_avis desc) selon la ville dans Entreprises en groupant les id_entreprise de Entreprises identiques.

Merci de votre aide !!!

[RunCodePhp]

Salut

En faisant comme ceci :

Code sql :

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

1
2
3
4
5
6
7
8
 
SELECT E.nom_entreprise,E.id_entreprise,A.dt_avis
FROM Entreprises E
JOIN Avis A ON E.id_entreprise = A.id_entreprise
WHERE E.ville_entreprise =  'paris'
AND E.cp_entreprise =  '75015'
ORDER BY dt_avis DESC 
LIMIT 0, 30

En faite, comme ça je ne perçois pas l'utilité de la sous-requête et même du GROUP BY.
Est-ce volontaire ?


Théoriquement, en faisant une jointure naturelle telle que JOIN, ça devrait grouper les id_entreprises de manière naturelle.
Il resterait donc juste à faire un trie.
J'ai peut être pas tout compris ceci dit.

[PIEPLU]

Yes, pas mal, reste plus qu'à grouper les E.id_entreprise

Merci

[RunCodePhp]

Yes, pas mal, reste plus qu'à grouper les E.id_entreprise

Comment ça grouper le E.id_entreprise ?

En rajoutant un GROUP BY E.id_entreprise, ça donne quoi ?


Ce ne serait pas un tri sur E.id_entreprise et dt_avis que tu souhaiterais par hasard ?

[PIEPLU]

Pourtant j'ai bien plusieurs fois le meme id_entreprise qui ressort

[RunCodePhp]

Pourtant j'ai bien plusieurs fois le meme id_entreprise qui ressort

Oui, je viens de me rencontre que j'ai dis une bêtise.

La requête d'ailleurs me semblerais plus logique comme ceci :

Code :

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

1
2
3
4
5
6
7
8
 
SELECT E.nom_entreprise,E.id_entreprise,A.dt_avis
FROM Avis A
JOIN Entreprises E ON A.id_entreprise = E.id_entreprise
WHERE E.ville_entreprise =  'paris'
AND E.cp_entreprise =  '75015'
ORDER BY dt_avis DESC 
LIMIT 0, 30

Théoriquement ça ne devrait rien changer, c'est juste qu'on perçois mieux que le but 1er est de rechercher des avis, (donc table Avis) qui elle est liée aux entreprises.

On obiendra bien une liste d'avis, donc potentiellement plusieurs id_entreprises s'il y a plusieurs avis pour 1 entreprise.


Mais j'ai corrigé mon post plus entre temps, et la question que je me pose serait s'il te faut la liste de tous les avis ou pas ?

- S'il te faut cette liste des avis, qu'il y ait 1 seul ou plusieurs par entreprise, il ne sera pas possible de faire un regroupement, dans ce cas là il vaut mieux le faire en Php.
C'est bien souvent un soucis de présentation.
Sinon il faut faire ça en 2 requêtes imbriquées. Une 1ère qui récupère les entreprises, une 2ème imbriquée pour rechercher les différents avis.

- S'il te faut juste par exemple 1 seul avis par entreprise, ou par exemple le nombre d'avis par entreprise, là un regroupent sera possible.


Si tu fais un regroupement sur les entreprise, on obtiendra justement qu'une seule entreprise, et du coup aussi 1 seul avis, et c'est théoriquement le 1er qui sera récupéré.


Sinon, la question que je posais dernièrement c'est si ce n'était pas un tri sur E.id_entreprise et dt_avis que tu souhaiterais.


Tout ce ci sauf erreur.

[PIEPLU]

Merci pour ton implication et ton travail sur mon sujet !!


bon WE