Discussion :

[bigben89]

Bonjour

J'ai 3 tables: fiche (IdFiche, Nom, ...), passages (IdPassage, IdFiche, DatePassage, ...) et piecesjointes (IdPieceJointe, IdFiche, DateCre, ...).
Je dois récupérer la liste des fiches n'ayant aucune pièce jointe ni aucun passage datant de moins de 18 mois.
J'ai donc fait la requête suivante :

Code :

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

1
2
3
4
5
6
SELECT COUNT(fe.IdFiche) AS NbFiches
FROM fiche AS fe
LEFT JOIN passages AS p ON fe.IdFiche = p.IdFiche AND p.DatePassage > NOW() - INTERVAL 18 MONTH
LEFT JOIN piecesjointes AS pj ON fe.IdFiche = pj.IdFiche AND pj.DateCre > NOW() - INTERVAL 18 MONTH
GROUP BY fe.IdFiche
HAVING COUNT(IdPassage) = 0 AND COUNT(IdPieceJointe) = 0

Le petit hic, c'est que au lieu de me renvoyer le nombre n d'éléments, il me renvoie n fois 1. Je sais qu'il me suffirait de compter le nombre de 1, mais je préfèrerais quand même récupérer directement la bonne valeur. Je sais pas vraiment comment résoudre le problème, sur le coup.

PS: je travaille sur MySQL 5.0 en développement et MySQL 3.23 en production (pas le choix de la version, malheureusement)

[CinePhil]

Il faut séparer les conditions de jointures des conditions de restrictions :

Code :

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

1
2
3
4
5
6
7
8
SELECT fe.IdFiche, fe.nom...
FROM fiche AS fe
LEFT JOIN passages AS p ON fe.IdFiche = p.IdFiche
LEFT JOIN piecesjointes AS pj ON fe.IdFiche = pj.IdFiche
WHERE p.DatePassage > NOW() - INTERVAL 18 MONTH
  AND pj.DateCre > NOW() - INTERVAL 18 MONTH
GROUP BY fe.IdFiche
HAVING COUNT(IdPassage) = 0 AND COUNT(IdPieceJointe) = 0

[bigben89]

Mettre les conditions dans le where pose problème parce que justement, je ne veux que les fiches n'ayant pas de passage ou de pièce jointe dans les 18 derniers mois :
Les mettre dans le where ne me gardera dans les résultats (après le where) que les fiches ayant des passages ou des pièces jointes dans les 18 derniers mois alors que ce sont justement celles que je veux éliminer (HAVING COUNT(...) = 0)

[Waldar]

Merci d'être clair sur votre besoin :

Je dois récupérer la liste des fiches n'ayant aucune pièce jointe ni aucun passage datant de moins de 18 mois

est différent de :

je ne veux que les fiches n'ayant pas de passage ou de pièce jointe dans les 18 derniers mois

Une fois ce point éclairci, vous pourrez essayer le code suivant :

Code :

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

1
2
3
4
5
6
7
8
SELECT fe.IdFiche, fe.nom...
FROM fiche AS fe
WHERE NOT EXISTS (SELECT 1 FROM passages AS p
                  WHERE p.IdFiche = fe.IdFiche
                  AND p.DatePassage > NOW() - INTERVAL 18 MONTH)
AND NOT EXISTS (SELECT 1 FROM piecesjointes AS pj 
                WHERE pj.IdFiche = fe.IdFiche
                AND pj.DateCre > NOW() - INTERVAL 18 MONTH)

[Waldar]

Je viens de voir qu'en prod vous avez une version ancestrale.
Je ne suis pas certain que la solution que je vous ai proposé fonctionnera.

Vous pourrez essayer celle-ci :

Code :

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

1
2
3
4
5
6
SELECT fe.IdFiche, fe.nom...
FROM fiche AS fe
LEFT JOIN passages AS p ON fe.IdFiche = p.IdFiche
LEFT JOIN piecesjointes AS pj ON fe.IdFiche = pj.IdFiche
WHERE (p.DatePassage > NOW() - INTERVAL 18 MONTH OR p.IdFiche IS NULL)
AND (pj.DateCre > NOW() - INTERVAL 18 MONTH OR pj.IdFiche IS NULL)

[bigben89]

SELECT fe.IdFiche, fe.nom...
FROM fiche AS fe
LEFT JOIN passages AS p ON fe.IdFiche = p.IdFiche
LEFT JOIN piecesjointes AS pj ON fe.IdFiche = pj.IdFiche
WHERE (p.DatePassage > NOW() - INTERVAL 18 MONTH OR p.IdFiche IS NULL)
AND (pj.DateCre > NOW() - INTERVAL 18 MONTH OR pj.IdFiche IS NULL)

Le problème de cette requête, c'est qu'elle inclue les fiches ayant au moins un passage et une pièce jointe datant de plus de 18 mois, mais n'exclue pas celles ayant des passages ou pièces jointes plus récentes.
Or chaque fiche pouvant avoir un bon nombre de passages/pièces jointes, il y en a pas mal qui en ont des 2 types.

Merci d'être clair sur votre besoin : Citation:
Envoyé par bigben89
Je dois récupérer la liste des fiches n'ayant aucune pièce jointe ni aucun passage datant de moins de 18 mois

est différent de : Citation:
Envoyé par bigben89
je ne veux que les fiches n'ayant pas de passage ou de pièce jointe dans les 18 derniers mois

Je suis désolé, je m'explique peut-être pas forcément très clairement, mais je ne vois pas trop la différence entre les deux.

[Waldar]

Le problème de cette requête, c'est qu'elle inclue les fiches ayant au moins un passage et une pièce jointe datant de plus de 18 mois, mais n'exclue pas celles ayant des passages ou pièces jointes plus récentes.

Et la requête avec le NOT EXISTS, que donne-t'elle ?

Avec les jointures externes :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
SELECT ...
FROM fiche AS fe
    LEFT JOIN passages AS p
      ON fe.IdFiche = p.IdFiche
     AND p.DatePassage > NOW() - INTERVAL 18 MONTH
    LEFT JOIN piecesjointes AS pj
      ON fe.IdFiche = pj.IdFiche
     AND pj.DateCre > NOW() - INTERVAL 18 MONTH 
WHERE
    p.IdFiche IS NULL
AND pj.IdFiche IS NULL

Je suis désolé, je m'explique peut-être pas forcément très clairement, mais je ne vois pas trop la différence entre les deux.

En simple, dans la première phrase vous avez un ET, dans la seconde un OU.