Discussion :

[dr0zz]

Bonjour,

j'ai développé une requête fonctionnelle avec MySQL 5.5 mais qui ne marche pas avec MySQL 5.0.

Etant donné que je n'arrive pas à avoir un message d'erreur parlant, j'aimerai savoir ce qui bloque :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
SELECT
COALESCE(a_ouv.NB_ACTION,0) AS NB_ACTION_OUV,
COALESCE(a.NB_ACTION,0) AS NB_ACTION,
COALESCE(SUM(CASE WHEN r.statut_risque = 'Ouvert' THEN 1 ELSE 0 END ),0)  AS NB_RISQUE_OUV,
COALESCE(COUNT(*),0) AS NB_RISQUE,
COALESCE(SUM(CASE WHEN (GREATEST(COALESCE(r.gravite_planning,0), COALESCE(r.gravite_cout,0), COALESCE(r.gravite_performance,0), COALESCE(r.gravite_accident,0)) * r.probabilite) <= p.acceptabilite THEN 1 ELSE 0 END ),0)  AS NB_ACCEPTABLE
FROM risque r
LEFT JOIN (	SELECT COUNT(*) AS NB_ACTION, identifiant_projet AS id_r
		FROM action GROUP BY identifiant_projet) AS a ON a.id_r=r.identifiant_projet
LEFT JOIN (	SELECT COUNT(*) AS NB_ACTION, identifiant_projet AS id_r
		FROM action WHERE statut_action='Ouvert' GROUP BY identifiant_projet) AS a_ouv ON a_ouv.id_r=r.identifiant_projet
LEFT JOIN projet p ON r.identifiant_projet=p.identifiant_projet

Merci d'avance..

[CinePhil]

Au minimum, il manque un GROUP BY puisqu'il y a mélange de calculs et de colonnes dans le SELECT !

Un autre SGBD que le mauvais MySQL aurait probablement refusé cette requête !

En plus j'ai de gros doutes sur la pertinence des résultats donnés par la requête !

D'après ce que j'en comprends, elle est censée donner des éléments sur les risques liés à des projets et les actions qui ont été ouvertes pour limiter les risques.
Du fait des jointures, il est fort probable que le COALESCE(COUNT(*), 0) AS NB_RISQUE ne donnera pas le nombre de risque mais le nombre d'actions.

Ne faudrait-il pas grouper à minima par identifiant projet.

Si vous nous donnez une explication du but de cette requête et la structure des tables impliquées, on pourra vous aider à faire une requête plus juste et peut-être plus simple.

[ericd69]

salut,

un truc: coalesce n'a de sens que si tu risques d'avoir un null qui sort d'une évaluation...

pour count aucune chance que tu ais un null
pour le sum ça dépend de ce que te sort le case... or il ne te sort que 0 ou 1...

piqure de rappel pour cinephil au passage: case n'a d’intérêt que si tu as plus de 2 cas à gérer... sinon on utilise:

Code sql :

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

if(condition,si_vrai,si_faux)

plus compact et surtout optimiser pour ce genre de cas
tu aurais normalement ce code qui fait donc la même chose:

Code sql :

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

1
2
3
4
5
6
7
8
9
10
11
12
SELECT
a_ouv.NB_ACTION AS NB_ACTION_OUV,
a.NB_ACTION AS NB_ACTION,
SUM(if(r.statut_risque = 'Ouvert', 1, 0))  AS NB_RISQUE_OUV,
COUNT(*) AS NB_RISQUE,
SUM(if(GREATEST(COALESCE(r.gravite_planning,0), COALESCE(r.gravite_cout,0), COALESCE(r.gravite_performance,0), COALESCE(r.gravite_accident,0)) * r.probabilite) <= p.acceptabilite, 1, 0))  AS NB_ACCEPTABLE
FROM risque r
LEFT JOIN (	SELECT COUNT(*) AS NB_ACTION, identifiant_projet AS id_r
		FROM action GROUP BY identifiant_projet) AS a ON a.id_r=r.identifiant_projet
LEFT JOIN (	SELECT COUNT(*) AS NB_ACTION, identifiant_projet AS id_r
		FROM action WHERE statut_action='Ouvert' GROUP BY identifiant_projet) AS a_ouv ON a_ouv.id_r=r.identifiant_projet
LEFT JOIN projet p ON r.identifiant_projet=p.identifiant_projet

pour le reste je partage les remarques de cinephil...

[dr0zz]

Merci pour ces retours,

j'ai réorganisé la requête en partant de la table des projets et en mettant les calculs au niveau des jointures. Je pense que MySQL 5.0 va arrêter de crier.

J'ai quand même été obligé de mettre du COALESCE sur les champs liés aux jointures car il est possible que je me retrouve avec des NULL.

Pour les curieux (ça reste un peu moche mais je fais avec ) :

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
19
20
21
 
SELECT
	COALESCE(a_ouv.NB_ACTION,0) AS NB_ACTION_OUV,
	COALESCE(a_all.NB_ACTION,0) AS NB_ACTION,
	COALESCE(r_ouv.NB_RISQUE_OUV,0)  AS NB_RISQUE_OUV,
	COALESCE(r_all.NB_RISQUE,0) AS NB_RISQUE,
	COALESCE(r_acc.NB_ACCEPTABLE,0) AS NB_ACCEPTABLE
FROM
	projet p
	LEFT JOIN (SELECT COUNT(*) AS NB_RISQUE, r.identifiant_projet FROM risque r WHERE 1=1 '.$filtre.' GROUP BY r.identifiant_projet) AS r_all 
	ON r_all.identifiant_projet=p.identifiant_projet
	LEFT JOIN (SELECT COUNT(*) AS NB_RISQUE_OUV, r.identifiant_projet FROM risque r WHERE r.statut_risque="Ouvert" '.$filtre.' GROUP BY r.identifiant_projet) AS r_ouv 
	ON r_ouv.identifiant_projet=p.identifiant_projet
	LEFT JOIN (SELECT SUM(IF(GREATEST(COALESCE(r.gravite_planning,0), COALESCE(r.gravite_cout,0), COALESCE(r.gravite_performance,0), COALESCE(r.gravite_accident,0)) * r.probabilite <= p.acceptabilite, 1, 0))  AS NB_ACCEPTABLE, r.identifiant_projet FROM risque r JOIN projet p ON r.identifiant_projet=p.identifiant_projet WHERE 1=1 '.$filtre.' GROUP BY r.identifiant_projet) AS r_acc 
	ON r_acc.identifiant_projet=p.identifiant_projet
	LEFT JOIN (	SELECT COUNT(*) AS NB_ACTION, a.identifiant_projet FROM action a LEFT JOIN risque r ON r.identifiant_risque = a.identifiant_risque WHERE 1=1 '.$filtre.' GROUP BY a.identifiant_projet) AS a_all 
	ON a_all.identifiant_projet=p.identifiant_projet
	LEFT JOIN (	SELECT COUNT(*) AS NB_ACTION, a.identifiant_projet FROM action a LEFT JOIN risque r ON r.identifiant_risque = a.identifiant_risque WHERE a.statut_action="Ouvert" '.$filtre.' GROUP BY a.identifiant_projet) AS a_ouv 
	ON a_ouv.identifiant_projet=p.identifiant_projet
WHERE
	p.identifiant_projet=:identifiant_projet