Code SQL requête correct ?

Bonsoir,

J'ai un code SQL assez simple pour récupérer des élements de mon fichier comme IDJeux si seulement celui-ci ne figure pas dans le fichier Planning.

Cela fonctionne bien mais je voudrais savoir si ce n'est pas trop de la bidouille ?

Code:

---

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

SELECT Jeux.IDJeux AS IDJeux, Jeux.Nom AS Nom, Jeux.Date_Debut_Jeu AS Date_Debut_Jeu, Jeux.Date_Fin_Jeu AS Date_Fin_Jeu, CASE WHEN Planning.IDJeux IS NULL THEN '0' ELSE '1' END AS Retour_Planning FROM Jeux LEFT OUTER JOIN Planning ON Planning.IDJeux = Jeux.IDJeux   WHERE (Retour_Planning = '0') AND (Date_Debut_Jeu <= '20131128' AND Date_Fin_Jeu >= '20131128')

---

Merci

Bonjour, :coucou:

Je ne sais pas si tu as posté le message dans le bon forum.

Mais j'ai quand même jeté un coup d’œil à ta requête :zoubi:

Personnellement, je supprimerai les parenthèses dans le WHERE.
De plus dans ton SELECT tu as

Code:

---

CASE WHEN Planning.IDJeux IS NULL THEN '0' ELSE '1' END AS Retour_Planning

---

et dans tes conditions de sélection tu effectues le test :

Code:

---

Retour_Planning = '0'

---

Pourquoi ne pas mettre directement

Code:

---

Retour_Planning IS NULL

---

Bon dev :ccool:

Bonjour,

personnellement, j'aurait plutot fait quelque chose du genre :

Code:

---

1 2 3 4 5 6 7 8 9

SELECT Jeux.IDJeux AS IDJeux, Jeux.Nom AS Nom, Jeux.Date_Debut_Jeu AS Date_Debut_Jeu, Jeux.Date_Fin_Jeu AS Date_Fin_Jeu FROM Jeux WHERE Jeux.IDJeux NOT IN(SELECT DISTINCT(Planning.IDJeux) FROM Planning) AND (Planning.IDJeux IS NULL) AND (Date_Debut_Jeu <= '20131128' AND Date_Fin_Jeu >= '20131128')

---

ou

Code:

---

1 2 3 4 5 6 7 8 9

SELECT Jeux.IDJeux AS IDJeux, Jeux.Nom AS Nom, Jeux.Date_Debut_Jeu AS Date_Debut_Jeu, Jeux.Date_Fin_Jeu AS Date_Fin_Jeu FROM Jeux WHERE NOT EXISTS(SELECT * FROM Planning WHERE Planning.IDJeux = Jeux.IDJeux) AND (Planning.IDJeux IS NULL) AND (Date_Debut_Jeu <= '20131128' AND Date_Fin_Jeu >= '20131128')

---

(sur un test que j'avait effectué sur une requête similaire, le 'not exists' était beaucoup moins performant)

Après je ne sais pas quelle solution est la meilleure, la plus performante, la plus facile ... Donc a toi de choisir ^^.

Bonjour,

Merci pour votre aide !

j'ai repris tes idées dsr57 et ce code pourrait convenir ? Il fonctionne chez moi :)

Code:

---

1 2 3 4 5 6 7 8 9 10 11

SELECT Jeux.IDJeux AS IDJeux, Jeux.Nom AS Nom, Jeux.Date_Debut_Jeu AS Date_Debut_Jeu, Jeux.Date_Fin_Jeu AS Date_Fin_Jeu FROM Jeux LEFT OUTER JOIN Planning ON Planning.IDJeux = Jeux.IDJeux   WHERE (Planning.IDJeux IS NULL) AND (Date_Debut_Jeu <= '20131128' AND Date_Fin_Jeu >= '20131128')

---

J'aime bien mettre dans parenthèses pour bien distinguer mes blocs, ce n'est pas utile ?

Merci

Bonjour,

ça ne dérange pas les parenthèses.
Ce qui me questionne c'est cette jointure sachant que ton objectif semble être de ne pas prendre les IDJeux qui sont null dans la table Jeux ...

dans ce cas :

Code:

---

1 2 3 4 5 6 7 8 9

SELECT Jeux.IDJeux AS IDJeux, Jeux.Nom AS Nom, Jeux.Date_Debut_Jeu AS Date_Debut_Jeu, Jeux.Date_Fin_Jeu AS Date_Fin_Jeu FROM Jeux WHERE (Jeux.IDJeux IS NOT NULL) AND (Jeux.Date_Debut_Jeu <= '20131128' AND Jeux.Date_Fin_Jeu >= '20131128')

---

à bientôt,

Nicolas

Salut Nicolas_Jeanneau,

Tu as mal compris ce qu'il veut :
Il veut les Jeux qui n'ont pas d'occurrence dans Planning.
IDJeux est la clé primaire de Jeux, donc ne sera jamais NULL dans la table Jeux. C'est une clé étrangère dans Planning.
Donc ce qu'il fait c'est il récupère tout ces jeux et fait un Left Outer Join avec la table planning.
Avec cette jointure, il récupère même les jeux pour lesquel il n'y a pas de planning. Dans ce cas Planning.IDJeux est à Null, sinon Planning.IDJeux = Jeux.IDJeux -> l'identifiant de Jeux.

WDKyle, si ton problème est réglé, pense à mettre le post en Résolu

Ca l'air d'etre bon alors ? :)

Merci a tous !