Discussion :

[WDKyle]

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 :

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

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

[dsr57]

Bonjour,

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

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

Code :

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

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 :

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

Retour_Planning = '0'

Pourquoi ne pas mettre directement

Code :

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

Retour_Planning IS NULL

Bon dev

[fubuntu]

Bonjour,

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

Code :

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

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 :

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

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 ^^.

[WDKyle]

Bonjour,

Merci pour votre aide !

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

Code :

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

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

[Nicolas_Jeanneau]

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 :

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

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

[fubuntu]

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

[WDKyle]

Ca l'air d'etre bon alors ?

Merci a tous !