Discussion :

[mcdelay]

Bonjour,


Je souhaite par le biais d'une requête compter des types de points collecte sur des communes , même pour certaines d'entres elles il n'y a pas de points de collecte.

Voici ma requête :

Code :

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

1
2
3
4
5
WITH all_pc as (SELECT public.env_point_collecte.commune as commune_all FROM public.env_point_collecte WHERE public.env_point_collecte.commune <> '' GROUP BY public.env_point_collecte.commune)
SELECT COUNT(*) as data , all_pc.commune_all as legend
FROM all_pc LEFT  JOIN public.env_point_collecte ON all_pc.commune_all = env_point_collecte.commune 
WHERE public.env_point_collecte.type_flux = 'Collecte sélective ' 
GROUP BY all_pc.commune_all,env_point_collecte.commune ORDER BY all_pc.commune_all ASC

Dans le with , je recupère toutes les communes. Je ne sais pas si je dois forcement passer par ce WITH


Merci pour vos retours !

[SQLpro]

Vous n'avez absolument pas besoin du premier GROUP BY qui est stupide !
Faites un RIGHT OUTER JOIN et récupérer les infos de la table de droite

Connaissez vous les alias de table ?
Savez-vous ce qu'est l'indentation d'une requête ??

Formez vous à SQL... Mon livre sur SQL peut vous y aider...




A +

[Baldrick]

Bonjour,

Je ne vois pas l'intérêt de faire une sous-requête qui n'a d'autre utilité que de lister les différentes valeurs d'une colonne d'une table, pour ensuite faire une jointure externe avec cette même table et un count(*) sur l'ensemble. Si vous voulez compter ou non des lignes en fonction d'une condition, alors passez en paramètre de la fonction count quelque chose qui vous renvoie NULL quand votre condition n'est pas remplie (count(*) compte toutes les lignes quelle qu'elles soient, mais si vous passez en paramètre une expression, seules les valeurs non nulles seront comptées).

Je vous conseille donc de faire quelque chose dans ce genre :

Code :

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

1
2
3
4
5
6
SELECT   commune legend, 
         count(case when env_point_collecte.type_flux = 'Collecte sélective ' then 1 else null end)  data
FROM     PUBLIC.env_point_collecte 
WHERE  commune IS NOT NULL 
GROUP BY commune
ORDER BY legend

[mcdelay]

bonjour,

Merci pour vos réponses ... effectivement à la base je ne vouslais pas passer par une sous requêtes . J'avais essayé avec "CASE " .. mais je l'avais mal placé ! Baldrick , ta solution fonctionne très bien, je te remercie !

.. par contre SQLPro , votre réponse n'est pas juste ... et encore moins si j'enlève le group by de la premiere requête .

SD

[SQLpro]

.. par contre SQLPro , votre réponse n'est pas juste ... et encore moins si j'enlève le group by de la premiere requête.

SD

Je persiste et signe ! Un group by sans calcul d'agrégat est de la plus haute stupidité ! Sans doute confondez vous l'opérateur DISTINCT avec la clause GROUP BY.... Ce n'est ni la même opération, ni le même coût !

Bref, formez vous à SQL !

A +