un problème de sous-requête [Résolu]

[clanou01]

Bonjour,
On y est mes premiers pas sur le forum !!
Je viens chercher de l'aide après de nombreuses recherches vaines et un mal de tête pas possible.
J'ai donc une requête avec une sous-requête :

Code :

1
2
3
4
5
6
SELECT com.nom_commune, 
SEC_TO_TIME(SUM((SELECT TIME_TO_SEC(inter.tps_activite) FROM atl_intervention inter WHERE inter.atl_activite_id_activite=1))) AS surveillance, SEC_TO_TIME(SUM((SELECT TIME_TO_SEC(inter.tps_activite) FROM atl_intervention inter WHERE inter.atl_activite_id_activite=10))) AS antiadulte
FROM atl_intervention inter, atl_commune com
WHERE inter.atl_commune_id_commune = com.id_commune
AND inter.date_intervention <= '2011-06-30' AND inter.date_intervention >='2011-06-01' AND com.code_dep = '17'
GROUP BY com.id_commune

et bien sûr quand je teste la requête dans mon phpmyadmin, j'ai le message suivant: #1242 - Subquery returns more than 1 row
ah ah j'suis sûre qu'on vous l'a jamais faite celle-là.
Bon, donc j'ai d'abord vu qu'il pouvait s'agir d'un problème de version..ouais!!
Ensuite, j'ai vu qu'on pouvait transformer une sous-requête en requête, et alors je suis perdue

Quelqu'un a la solution ??

Merci

[Yanika_bzh]

Pourquoi utiliser une sous requete ?

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
SELECT 
	com.nom_commune, 
	SEC_TO_TIME(SUM( CASE WHEN inter.atl_activite_id_activite=1 THEN inter.tps_activite ELSE 0 END)) AS surveillance
 	SEC_TO_TIME(SUM( CASE WHEN inter.atl_activite_id_activite=10 THEN inter.tps_activite ELSE 0 END)) antiadulte
FROM 
	atl_intervention inter 
INNER JOIN
	atl_commune com
ON	(inter.atl_commune_id_commune = com.id_commune)
WHERE 
	inter.date_intervention <= '2011-06-30' 
	AND inter.date_intervention >='2011-06-01' 
	AND com.code_dep = '17'
GROUP BY com.id_commune

Ne fonctionne t il pas ?

Bon courage

[ced]

Et vous pouvez même remplacer :

Code :

1
2
3
WHERE 
	inter.date_intervention <= '2011-06-30' 
	AND inter.date_intervention >='2011-06-01'

par :

Code :

1
2
WHERE 
	inter.date_intervention BETWEEN '2011-06-01' AND '2011-06-30'

Ce qui sera plus performant.

[Yanika_bzh]

Envoyé par ced

Et vous pouvez même remplacer :

Code :

par :

Code :

Ce qui sera plus performant.

ah bon ?

[ced]

Voir le tutoriel : http://sqlpro.developpez.com/cours/optimiser/#L9, transformation n°7...
Dans BETWEEN, les bornes sont incluses.

ced

[CinePhil]

Envoyé par Yanika_bzh

ah bon ?

Ben d'après SQLPro, oui. Mais il précise qu'il faut quand même tester les transformations qu'il donne. En tout cas, cette transformation utilisant BETWEEN à au moins le mérite de rendre la requête plus claire.

Envoyé par Yanika_bzh

Code :

Pendant que j'y suis, inutile de mettre la condition de jointure entre parenthèses !

[Yanika_bzh]

@Cinephil : La mise entre parenthèse n'affectant en rien la syntaxe de l'instruction, je préfère la garder... C'est un reliquat de ma méthode de développement au cours de ma carrière. Elle me permet de bien différencier la condition de jointure de la syntaxe de jointure (dans le but par exemple de génération automatique de documentation de sources).

En ce qui concerne le between, il serait intéressant d'analyser le plan d’exécution.

En tout cas, de mémoire, ce n'est pas le cas sur un moteur SYBASE 12.5x

[clanou01]

Vous êtes génialissimes !!! lol
Merci yanika_bzh, tu m'as enlevé une grosse épine du pied !!
J'ai cependant ajouté quelques modif' au code que tu m'as envoyé et ça marche!!
je le rebascule du coup pour ceux qui auraient le même problème que moi :

Code :

1
2
3
4
5
6
7
8
9
SELECT com.nom_commune, 
SEC_TO_TIME(SUM( TIME_TO_SEC (CASE WHEN inter.atl_activite_id_activite=1 THEN inter.tps_activite ELSE 0 END))) AS surveillance,
SEC_TO_TIME(SUM( TIME_TO_SEC (CASE WHEN inter.atl_activite_id_activite=10 THEN inter.tps_activite ELSE 0 END))) AS antiadulte
FROM atl_intervention inter 
INNER JOIN atl_commune com ON inter.atl_commune_id_commune = com.id_commune
WHERE inter.date_intervention <= '2011-06-30' 
AND inter.date_intervention >='2011-06-01' 
AND com.code_dep = '17'
GROUP BY com.id_commune

encore merci de prendre de votre temps pour les novices comme moi !!

[clanou01]

pourquoi quand je le mets dans mon php, ça ne marche pas?
ça met : undefined

[Yanika_bzh]

regardez vos log

[clanou01]

c'est à dire ?

[NicoD.]

Le code PHP utilisé nous serait d'un grand secours pour t'aider.

[clanou01]

eh voilà,

Code php :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
function get_activite_byDep_byDate($code_dep, $date_debut, $date_fin)
{
  $sql = "
 
          SELECT com.nom_commune, 
          SEC_TO_TIME(SUM(TIME_TO_SEC(inter.tps_prospection))) as tps_prospection, SEC_TO_TIME(SUM(TIME_TO_SEC(inter.tps_traitement))) as tps_traitement, SEC_TO_TIME(SUM(TIME_TO_SEC(inter.tps_controle))) as tps_controle, SUM(inter.quantite_produit) as quantite_produit,
          SEC_TO_TIME(SUM( TIME_TO_SEC (CASE WHEN inter.atl_activite_id_activite=1 THEN inter.tps_activite ELSE 0 END))) AS surveillance,
          SEC_TO_TIME(SUM( TIME_TO_SEC (CASE WHEN inter.atl_activite_id_activite=10 THEN inter.tps_activite ELSE 0 END))) AS antiadulte,
          SEC_TO_TIME(SUM( TIME_TO_SEC (CASE WHEN inter.atl_activite_id_activite=11 THEN inter.tps_activite ELSE 0 END))) AS urbain
          FROM atl_intervention inter 
          INNER JOIN atl_commune com ON (inter.atl_commune_id_commune = com.id_commune)
          WHERE inter.date_intervention <= '$date_fin' AND inter.date_intervention >='$date_debut' 
          AND com.code_dep = '$code_dep'
          GROUP BY com.id_commune
        ";
 
  $mysql = new base_mysql();
  //$mysql->base_Debug(true);
  $mysql->base_Open();
  $mysql->base_Query($sql);
  $mysql->base_Close();
 
  $retour = array();
  while ($row = $mysql->base_Fetch_Assoc()) {
    array_push($retour, $row);
  }
 
  return $retour;
}

[clanou01]

c'est bon, j'ai trouvé, cela venait de la requête mais de mon code pour générer un tableau où mettre les résultats.
encore merci

[clanou01]

cela NE venait PAS de la requête... punaise, vivement le w-e !!
bye bye