Discussion :

[FranckyVegas]

Bonjour, voici mon problème :

J'ai le MRD suivant (juste un extrait) :

MATERIEL (idmateriel, libellemateriel, #idtype, #idmarque)
TYPE (idtype, libelletype, frequence)
REVISION (idrevision, daterevision, commrevision, #idmateriel)

Je travaille sous MS Access

J'aimerai créer une requête SQL me permettant de :

Faire une liste du matériel devant être révisé.

Sachant qu'on se base pour ça sur la dernière révision effectuée sur le matériel et la fréquence des révisions caractéristique de chaque type de matériel.

J'y travaille depuis quelques temps mais là je cale...

Merci d'avance pour votre aide !

[al1_24]

Pourriez-vous nous présenter une ébauche de la requête que vous tentez de mettre au point.
Cela sera plus facile pour la faire évoluer.

[FranckyVegas]

Voila une ébauche de ma requête :

Code :

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

1
2
3
4
5
6
7
8
SELECT M.idmateriel
FROM materiel M
WHERE M.idmateriel NOT IN
( SELECT M1.idmateriel
 FROM materiel M1, revision R, type T
 WHERE M1.idmateriel = R.idmateriel
  AND M1.idtype = T.idtype
AND DATEDIFF( "mm" , \* date dernière revision */ , now() ) < T.frequence )

Voila... je crois que la structure de la requête est bonne...
Ce qui me dérange c'est de récupérer la date de la dernière révision.

[al1_24]

J'aurais fait comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
SELECT    mat.idmateriel
FROM    materiel    AS mat
    INNER JOIN
        revision    AS rev
        ON    rev.idmateriel = mat.idmateriel
    INNER JOIN
        type    AS type
        ON    rev.idtype = mat.idtype
GROUP BY
        mat.idmateriel
    ,    typ.frequence
HAVING    MAX(rev.daterevision) + typ.frequence <= CURRENT_DATE
;

Utilisez INNER JOIN plutôt que des jointures dans la clause WHERE ; cela permet de séparer ce qui dépend de la jointure proprement dite du filtre sur les données et rend la requête plus lisible et plus maintenable.

[Yanika_bzh]

J'aurai meme fait une jointure externe sur revision (si il n'y a jamais eu de revision auparavant).

Bon courage

[defcon_suny]

Et voilà ma solution :

Code :

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

1
2
3
4
5
6
 
SELECT t1.*
FROM materiel AS t1
INNER JOIN type AS t2 ON t1.idtype = t2.idtype
INNER JOIN revision AS t3 ON t1.idmateriel = t3.idmateriel
WHERE ADDDATE(t3.daterevision, INTERVAL t2.frequence  DAY ) < CURDATE();

Mais personnellement, je n'aurai pas structuré mes tables de la sorte. A mon sens la date de revision est un attribut de me mon materiel et pas du type de materiel....

++

[Yanika_bzh]

Et voilà ma solution :

Code :

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

1
2
3
4
5
6
 
SELECT t1.*
FROM materiel AS t1
INNER JOIN type AS t2 ON t1.idtype = t2.idtype
INNER JOIN revision AS t3 ON t1.idmateriel = t3.idmateriel
WHERE ADDDATE(t3.daterevision, INTERVAL t2.frequence  DAY ) < CURDATE();

Mais personnellement, je n'aurai pas structuré mes tables de la sorte. A mon sens la date de revision est un attribut de me mon materiel et pas du type de materiel....

++

Et si il n'y a jamais eu de revision pour un type donné ??

[FranckyVegas]

Mais personnellement, je n'aurai pas structuré mes tables de la sorte. A mon sens la date de revision est un attribut de me mon materiel et pas du type de materiel....

Finalement j'ai modifié quelque peu ma BDD.
Matériel contient maintenant la date de révision la plus récente.

La requête est donc à mon goût beaucoup plus simple :

Code :

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

1
2
3
4
 
SELECT materiel.idmateriel, type.libelletype, marque.libellemarque, materiel.libellemateriel
FROM marque INNER JOIN (type INNER JOIN materiel ON type.idtype = materiel.idtype) ON marque.idmarque = materiel.idmarque 
WHERE DateDiff("m", materiel.daterev, Now()) > type.frequence

Comme ça tout fonctionne très bien.
Merci à tous pour votre aide