Discussion :

[FrankOVD]

Bonjour,

Je travaille présentement à la correction d'un code disfonctionnel programmé par un stagiaire qui est passé avant moi et j'ai trouvé dans le code une requête dont le ne suis pas sur de la validité. Peut-être est-ce seulement que je n'en ai jamais fait comme ça mais je vous demande votre opinion sur sa validité:

Code :

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

1
2
3
4
5
6
7
8
9
 
SELECT c1.id, c2.id 
FROM carte_temps2 as c1, carte_temps2 as c2
WHERE c1.no_camion>0 
  AND c1.no_camion=c2.no_camion 
  AND (not (c1.id=c2.id)) 
  AND ((concat(if(c2.hrt < c2.hdq, c2.dfq, c2.ddq), " ", c2.hrt) 
        < concat(if(c1.htt<c1.hdq,c1.dfq,c1.ddq), " ", c1.htt)) 
       AND c1.od < c2.oaa );

C'est la partie dans le concat() qui me pose problème. Comme le nom des colonnes est très peu significatif, en voici la description:

Code :

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

1
2
3
4
5
6
7
8
9
 
ddq : Date Debut Quart (AAAA-MM-JJ)
dfq : Date Fin Quart (AAAA-MM-JJ)
hdq : Heure Début Quart (HH:MM:SS)
hrt : Heure Retour Terminal (HH:MM:SS)
htt : Heure Départ Terminal (HH:MM:SS)
oaa : Odomètre à l'Arrivée
odc : Odomètre Dernier Client
opc : Odomètre Premier Client

Est-ce correct? Lorsque le j'exécute en console ça ne me retourne aucune erreur de syntaxe mais aucun enregistrement non plus.

[Soutou]

Le résultat de ta requête dépend des données contenues dans les tables.

Si tu n'es pas sûr, exécute là en enlevant les conditions les unes après les autres. Des données vont apparaître au fur et à mesure.

Compose un jeu d'essai qui doit resortir et lance la requête initiale.

[pcaboche]

D'un point de vue syntaxe, il n'y a pas de problème, ça passe.

Maintenant, d'un point de vue sémantique, c'est moins évident.

En fait, ce qui se passe: Tu concatènes deux dates pour en faire une chaîne (avec un IF au millieu), puis tu prends deux autres dates, que tu concatènes aussi pour en faire une chaîne (également avec un IF au millieu). Tu auras donc deux chaînes de format "YYYY-MM-DD HH:MM:SS YYYY-MM-DD HH:MM:SS" que tu compares entre elles.

Vu que tes dates sont au format UNIX (année d'abord, puis mois...), tu peux faire des comparaisons de chaînes entre elles, il n'y a pas de problème. On a donc 4 dates dans 2 chaines: "Date1 Date2" et "Date3 Date4" (en faisant abstraction des IF...)

Si Date1<Date2, alors le test renvoie 'Vrai'. Si Date1==Date2 (à la seconde près!), alors c'est Date3<Date4 qui est évaluée. Si Date1>Date2, alors ça renvoie 'faux'.

Bref, ça peut éventuellement marcher. Par contre, pour le bien de tous (y compris de tes collègues qui vont devoir reprendre ton application), tu devrais essayer de comprendre ce que c'est sensé faire et faire des comparaisons de date classiques.

***

Egalement pour plus de clareté : pense à utiliser une jointure INNER JOIN et donc à séparer tes conditions de jointure (dans le ON) des conditions de restriction de tes ensembles (dans le WHERE)

Aussi, plutôt que d'écrire not(A=B), écrit directement A!=B, on comprend mieux.

Enfin, n'hésite pas à mettre dans ton rapport que le stagiaire précédent était un gros naze et que ses requêtes sont incompréhensibles, donc peu maintenables...(le fait que tu te sois poser la question "c'est normal de faire comme ça?" montre que tu te poses les bonnes questions).

[pcaboche]

Petite correction:

J'avais pas vu que dans :

Code :

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

1
2
3
4
5
6
7
8
9
 
ddq : Date Debut Quart (AAAA-MM-JJ)
dfq : Date Fin Quart (AAAA-MM-JJ)
hdq : Heure Début Quart (HH:MM:SS)
hrt : Heure Retour Terminal (HH:MM:SS)
htt : Heure Départ Terminal (HH:MM:SS)
oaa : Odomètre à l'Arrivée
odc : Odomètre Dernier Client
opc : Odomètre Premier Client

certains champs sont des dates et d'autres sont des heures (j'ai considéré que tout était en DATETIME). Ca ne change rien à ce que je viens de dire.

[pcaboche]

Un autre truc que je comprends pas:

Code :

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

if(c2.hrt < c2.hdq, c2.dfq, c2.ddq)

Dans la condition du IF, on compare des heures pour déterminer quelle date on va utiliser plus loin pour la comparaison. C'est assez aléatoire comme truc ! (pour 2 moments t1 et t2 tels que t1<t2, tu peux avoir heure(t2) > heure(t1) si t1 et t2 sont sur 2 jours différents)

Bref, ça m'a l'air bien pourrie comme requête !