Discussion :

[franck10]

Bonjour à tous,

J'exécute la requête suivante :

Code :

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

1
2
3
4
5
6
7
8
9
SELECT postes.site, count(distinct postes.pi) as somme
FROM tpr_appli, installer, applications, postes, tpr_poste
where applications.nom_appli = installer.noms_appli
AND STR_TO_DATE( tpr_appli.date_valid, '%d/%m/%Y' ) < CURDATE( )
AND applications.nom_appli = tpr_appli.noms_appli
AND postes.pi = installer.pi_poste
AND tpr_poste.maj_scope <> 'fait'
AND tpr_poste.pi_poste = postes.pi
group by site

Le résultat doit me retourner un nombre de poste sur lesquels les applications ont une date inférieur à la date du jour.
La requête fonctionne mais mon problème c'est qu'elle compte aussi les postes qui ont des appli avec la date supérieur à celle du jour.
Par exemple j'ai le poste 1 qui a deux applications toto et tata
L'application toto a la date de validation au 31/10/2011 et tata au 16/01/2012.
Dans mon résultat le poste 1 va apparaître alors que ses deux applications n'ont pas une date inférieur.

Est-ce qu'il y a moyen de corriger cette erreur?

Merci par avance.

[CinePhil]

1) Les jointures s'écrivent depuis 1992 avec l'opérateur JOIN ; il serait temps de s'y mettre !

2) Votre restriction sur la date laisse à penser que la colonne tpr_appli.date_valid n'est pas de type DATE, ce qui est une erreur d'implémentation.

3) Votre besoin est ambigu.

Le résultat doit me retourner un nombre de poste sur lesquels les applications ont une date inférieur à la date du jour.

Par exemple j'ai le poste 1 qui a deux applications toto et tata
L'application toto a la date de validation au 31/10/2011 et tata au 16/01/2012.
Dans mon résultat le poste 1 va apparaître alors que ses deux applications n'ont pas une date inférieure.

Ceci veut-il dire que le poste 1 ne doit pas apparaître parce que au moins une de ses applications n'a pas une date inférieure ?
Ce n'est pas tout à fait la même chose que la première phrase.

Exprimez mieux votre besoin.

[franck10]

1) Les jointures s'écrivent depuis 1992 avec l'opérateur JOIN ; il serait temps de s'y mettre !

2) Votre restriction sur la date laisse à penser que la colonne tpr_appli.date_valid n'est pas de type DATE, ce qui est une erreur d'implémentation.

3) Votre besoin est ambigu.

Ceci veut-il dire que le poste 1 ne doit pas apparaître parce que au moins une de ses applications n'a pas une date inférieure ?
Ce n'est pas tout à fait la même chose que la première phrase.

Exprimez mieux votre besoin.

Oui en effet c'est bien ça, le poste 1 ne doit pas apparaître parce qu'une de ses applications n'a pas une date inférieure à celle du jour. A partir du moment qu'une application à une date inférieur alors il m'affiche le poste sans prendre en compte la date des autres applications.

[CinePhil]

Oui en effet c'est bien ça, le poste 1 ne doit pas apparaître parce qu'une de ses applications n'a pas une date inférieure à celle du jour. A partir du moment qu'une application à une date inférieur alors il m'affiche le poste sans prendre en compte la date des autres applications.

Votre phrase est encore contradictoire !

le poste 1 ne doit pas apparaître parce qu'une de ses applications n'a pas une date inférieure à celle du jour

Donc si le poste 1 a une application avec 2011-12-31, comme nous sommes le 5 décembre 2011, le poste n'apparaît pas, quelles que soient les autres dates des applications du poste.

A partir du moment qu'une application à une date inférieur alors il m'affiche le poste sans prendre en compte la date des autres applications.

Et là si le même poste a une autre application avec 2011-11-30, comme nous sommes le 5 décembre 2011, le poste doit cette fois apparaître sans tenir compte de l'application qui a une date dans le futur !

Bis repetita :

Exprimez mieux votre besoin.

[franck10]

Tout d'abord merci de votre aide j'ai réussi à solutionner mon problème avec cette requête (je sais mes jointure ne sont pas up to date mais je ne suis pas un professionnel de la requête et je fais comme je connais et ça fonctionne très bien ainsi)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
SELECT postes.site, count(DISTINCT postes.pi) AS somme
FROM tpr_appli, installer, applications, postes, tpr_poste
WHERE postes.pi not in (SELECT DISTINCT postes.pi
FROM tpr_appli, postes, installer, applications, tpr_poste
WHERE STR_TO_DATE( tpr_appli.date_valid, '%d/%m/%Y' ) > CURDATE( )
AND applications.nom_appli = installer.noms_appli
AND applications.nom_appli = tpr_appli.noms_appli
AND tpr_poste.pi_poste = postes.pi
AND postes.pi = installer.pi_poste)
AND applications.nom_appli = installer.noms_appli
AND applications.nom_appli = tpr_appli.noms_appli
AND postes.pi = installer.pi_poste
AND tpr_poste.maj_scope <> 'fait'
AND tpr_poste.pi_poste = postes.pi
GROUP BY site

[CinePhil]

Vous avez juste 19 ans de retard !

C'est pourtant pas dur d'écrire des jointures correctement !

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
SELECT postes.site, count(DISTINCT postes.pi) AS somme
FROM applications
INNER JOIN tpr_appli ON applications.nom_appli = tpr_appli.noms_appli
INNER JOIN installer ON applications.nom_appli = installer.noms_appli
	INNER JOIN postes ON postes.pi = installer.pi_poste
		INNER JOIN tpr_poste.pi_poste = postes.pi
WHERE postes.pi NOT IN 
(
	SELECT DISTINCT postes.pi
	FROM applications
	INNER JOIN tpr_appli ON applications.nom_appli = tpr_appli.noms_appli
	INNER JOIN installer ON applications.nom_appli = installer.noms_appli
		INNER JOIN postes ON postes.pi = installer.pi_poste
			INNER JOIN tpr_poste.pi_poste = postes.pi
	WHERE STR_TO_DATE( tpr_appli.date_valid, '%d/%m/%Y' ) > CURRENT_DATE
)
AND tpr_poste.maj_scope <> 'fait'
GROUP BY postes.site

C'est plus clair et en indentant le code, on voit bien les associations entre les tables ; on peut presque voir le MCD en lisant la requête !

Si je lis bien la requête, vous comptez pour chaque site le nombre de postes qui n'ont pas d'application avec une date valide dans le futur et qui ont une "tpr_poste.maj_scope" faite.

Vu la position de la restriction sur "tpr_poste.maj_scope", la jointure sur tpr_poste est inutile dans la sous-requête.

Je pense qu'on peut faire plus simple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
SELECT p.site, count(DISTINCT p.pi) AS somme
FROM postes p
INNER JOIN tpr_poste.pi_poste = p.pi
WHERE NOT EXISTS
(
	SELECT *
	FROM applications
	INNER JOIN tpr_appli ON applications.nom_appli = tpr_appli.noms_appli
	INNER JOIN installer ON applications.nom_appli = installer.noms_appli
	WHERE installer.pi_poste = p.pi
		AND STR_TO_DATE( tpr_appli.date_valid, '%d/%m/%Y' ) > CURRENT_DATE
)
AND tpr_poste.maj_scope <> 'fait'
GROUP BY postes.site

Traduction de la requête :
Compter par site le nombre distinct de "poste.pi" à "tpr_poste.maj_scope" faits pour lesquels il n'existe pas d'application installée à date valide supérieure à aujourdhui.

À vous de voir si ça correspond à votre besoin.