Discussion :

[maysa]

Bonjour,

j'ai une requete à faire qui consiste à récupérer le numéro des clients qui n'ont pas passé commande depuis plus de 30 jours.

J'ai donc fait ceci..

Code :

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

1
2
3
4
5
6
7
8
9
 
 
SELECT distinct(facture.numClient)
	FROM facture 
	WHERE (datediff(now(),dateFacture)>=30) 
	AND facture.numClient NOT IN (
					SELECT facture.numClient
					FROM facture
					WHERE datediff(now(),dateFacture) <30)

L'inconvénient c'est que c'est assez long comme requete , mais elle semble donner des résultats exacts.

A votre avis, ma requete est elle correcte ou y-a t-il plus simple ?

Merci beaucoup

[ced]

Bonjour,

Je ne vois pas, à part dégrader les performances, à quoi peut bien te servir la sous-requête .
La requête suivante ne suffit-elle pas :

Code :

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

1
2
3
SELECT DISTINCT(facture.numClient)
FROM facture 
WHERE datediff(now(),dateFacture) >= 30

ced

[CinePhil]

Euh... c'est normal que tu cherches des clients qui n'ont pas passé commande et tu interroges une table factures ?

[maysa]

Euh... c'est normal que tu cherches des clients qui n'ont pas passé commande et tu interroges une table factures ?

Oui pour l'instant c'est normal, la table cmd se nomme facture...

[maysa]

Bonjour,

Je ne vois pas, à part dégrader les performances, à quoi peut bien te servir la sous-requête .
La requête suivante ne suffit-elle pas :

Code :

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

1
2
3
SELECT DISTINCT(facture.numClient)
FROM facture 
WHERE datediff(now(),dateFacture) >= 30

ced

Hello Ced,

Eh bien si je fais ce que tu me dis, j'ai bien les numClients des personnes qui n'ont pas commandé depuis 30 jours mais rien ne me dit que ces memes clients n'ont pas passé une autre commande il y a moins de 30 jours.
Ils peuvent très bien avoir passé une commande hier et une autre il y a plus de 30 jours. Et moi je veux trouver tout ceux inactifs depuis 30 jours.

En espérant avoir été clair..
Merci

[Cybher]

peut etre comme ceci (à tester)

Code :

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

1
2
3
4
SELECT facture.numClient,max(dateFacture)
FROM facture 
group by numclient
having datediff(now(),max(dateFacture)) >= 30

[ced]

C'est plus clair... Effectivement, c'est logique .
Donc, ce qui dégrade les performances de ta requête, c'est la sous-requête. Avec de grosses tables, dans MySQL, les sous-requêtes se dégradent rapidement...

Donc, tu peux très bien réécrire ta première requête en passant par une jointure externe, dont les performances seront sans doute meilleures :

Code :

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

1
2
3
4
5
SELECT DISTINCT(f1.numClient)
FROM facture f1
LEFT OUTER JOIN facture f2 ON f1.numClient = f2.numClient AND datediff(now(), f2.dateFacture) <30
WHERE (datediff(now(), f1.dateFacture)>=30) 
AND f2.numClient IS NULL

ced

[maysa]

peut etre comme ceci (à tester)

Code :

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

1
2
3
4
SELECT facture.numClient,max(dateFacture)
FROM facture 
group by numclient
having datediff(now(),max(dateFacture)) >= 30

Effectivement oui ça marche, mais curieusement hier en la faisant, je n'avais pas les memes données. Mais elle est beaucoup plus rapide , merci

Ced, c'est vrai j'avais oublié cette technique, mais curieusement elle est encore plus longue que ma requete imbriquée !