Clients n'ayant pas commandé depuis n mois

Bonjour,


Langage MySQL 4 + PHP 4

J'ai fait une requête pour renvoyer tous les clients n'ayant pas commandé depuis n mois mais je pense qu'elle n'est pas optimum.

Structure des tables :

La table des clients :

Code:

---

1 2 3 4 5

CREATE TABLE `clients` ( `cli_id` bigint(20) unsigned NOT NULL auto_increment, `cli_date` date default NULL, PRIMARY KEY (`cli_id`) );

---

La table d'en-tête des commandes :

Code:

---

1 2 3 4 5 6

CREATE TABLE `commandes` ( `cmd_id` bigint(20) unsigned zerofill NOT NULL auto_increment, `cli_id` bigint(20) unsigned NOT NULL, `cmd_date` datetime NOT NULL default '0000-00-00 00:00:00', PRIMARY KEY (`cmd_id`) );

---

Merci de votre aide

Si tu pouvais nous donner la requête en question... ça serait plus simple... la meilleur que je vois pour le moment est :

@mois est le nombre de moins (ton N)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

SELECT CLI.cli_id FROM clients CLI LEFT JOIN commandes COM ON COM.cli_id = CLI.cli_id WHERE PERIOD_DIFF(DATE_FORMAT(CLI.cli_date, '%Y%m'), DATE_FORMAT(NOW(), '%Y%m')) > @mois AND ( MAX(COM.cmd_date) IS NULL OR PERIOD_DIFF(DATE_FORMAT(MAX(COM.cmd_date), '%Y%m'), DATE_FORMAT(NOW(), '%Y%m')) > @mois ) GROUP BY CLI.cli_id

---

Merci de ton aide, j'ai essayé ta requête mais j'ai une erreur :
'invalid use of group function'

Swoög a juste fait une petite étourderie (de copier-coller sans doute).

La ligne GROUP BY doit être la dernière ligne normalement.

Citation:

---

Envoyé par Alexandre T

Swoög a juste fait une petite étourderie (de copier-coller sans doute).

La ligne GROUP BY doit être la dernière ligne normalement.

---

J'ai toujours un peu de mal avec l'ordre de différentes lignes... désolé, j'édite ;)

J'ai encore la même erreur, j'avais effectivement corrigé l'ordre.

Il semblerait qu'il n'accepte pas 2 conditions.

Avez-vous testé ?

Merci

N'ayant pas trouvé de solution, je relance le post :

Citation:

---

Envoyé par Swoög

Si tu pouvais nous donner la requête en question... ça serait plus simple... la meilleur que je vois pour le moment est :

@mois est le nombre de moins (ton N)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

SELECT CLI.cli_id FROM clients CLI LEFT JOIN commandes COM ON COM.cli_id = CLI.cli_id WHERE PERIOD_DIFF(DATE_FORMAT(CLI.cli_date, '%Y%m'), DATE_FORMAT(NOW(), '%Y%m')) > @mois AND ( MAX(COM.cmd_date) IS NULL OR PERIOD_DIFF(DATE_FORMAT(MAX(COM.cmd_date), '%Y%m'), DATE_FORMAT(NOW(), '%Y%m')) > @mois ) GROUP BY CLI.cli_id

---

---

La requête est inversé au niveau des dates (sinon on a des valeurs négatives) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

SELECT CLI.cli_id FROM clients CLI LEFT JOIN commandes COM ON COM.cli_id = CLI.cli_id WHERE PERIOD_DIFF(DATE_FORMAT(NOW(), '%Y%m'), DATE_FORMAT(CLI.cli_date, '%Y%m')) > @mois AND ( MAX(COM.cmd_date) IS NULL OR PERIOD_DIFF(DATE_FORMAT(NOW(), '%Y%m'), DATE_FORMAT(MAX(COM.cmd_date), '%Y%m')) > @mois ) GROUP BY CLI.cli_id

---

Par contre j'ai une erreur sur le GROUP BY qui refuse si on ajoute la deuxième condition dans le where :

Code:

---

1 2 3 4 5

AND ( MAX(COM.cmd_date) IS NULL OR PERIOD_DIFF(DATE_FORMAT(NOW(), '%Y%m'), DATE_FORMAT(MAX(COM.cmd_date), '%Y%m')) > @mois )

---

Pouvez me dire pourquoi on ne peut pas renseigner la deuxième partie et quel serait la solution ?

En faisant une requête simple sans cette deuxième conditions le retour met plus de 6 secondes pour 1000 clients et 10 000 commandes

J'ai pensé à faire plutôt une jointure par double select qui est instantanée (c'est peut être une piste) :

Code:

---

1 2 3 4 5 6

SELECT CLI.cli_id FROM clients CLI, commandes COM WHERE PERIOD_DIFF(DATE_FORMAT(NOW(), '%Y%m'), DATE_FORMAT(CLI.cli_date, '%Y%m')) > @mois AND COM.cmd_cli_id = CLI.cli_id GROUP BY CLI.cli_id

---

Merci à Swoög et Alexandre pour leur aide.

effectivement...

la dexième partie du WHERE utilise des clause regroupantes (MAX) qui utilisent le résultat du GROUP BY...

il faudrait le mettre dans un HAVING :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

SELECT CLI.cli_id FROM clients CLI LEFT JOIN commandes COM ON COM.cli_id = CLI.cli_id WHERE PERIOD_DIFF(DATE_FORMAT(CLI.cli_date, '%Y%m'), DATE_FORMAT(NOW(), '%Y%m')) > @mois GROUP BY CLI.cli_id HAVING MAX(COM.cmd_date) IS NULL OR PERIOD_DIFF(DATE_FORMAT(MAX(COM.cmd_date), '%Y%m'), DATE_FORMAT(NOW(), '%Y%m')) > @mois

---

ensuite, pour le temps d'exécution, j'ai pas d'idée, désolé...

Je préfère un double select pour la jointure, puisqu'elle est instantanée à la différence du left join.
Seulement je ne récupère pas les clients qui n'ont jamais commandés.

J'ai repris une partie de ta requête :

Code:

---

1 2 3 4 5 6 7 8 9 10

SELECT CLI.cli_id, CLI.cli_date, COM.cmd_date FROM clients CLI, commandes COM WHERE PERIOD_DIFF(DATE_FORMAT(NOW(), '%Y%m'), DATE_FORMAT(CLI.cmd_date, '%Y%m')) > @mois AND COM.cmd_cli_id = CLI.cli_id GROUP BY CLI.cli_id HAVING MAX(COM.cmd_date) IS NULL OR PERIOD_DIFF(DATE_FORMAT(NOW(), '%Y%m'),DATE_FORMAT(MAX(COM.cmd_date) , '%Y%m')) > @mois

---

Est-ce possible de garder cette base et d'ajouter les clients n'ayant jamais commandés sans alourdir ?

avec une requête externe, tu aura tous les clients...

De plus les reuqêtes directes sont déconseillées, ne serait-ce que d'un point de vue sémantique...

pour plus de renseignements, je t'invite à te pencher sur le tuto de SQLpro (un lien est dispo sur mon site)