Discussion :

[Vict0]

Bonjour,

J'effectue en ce moment une migration de MySQL vers PostgreSQL et j'ai quelques difficultés avec cette requête :

Code :

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

1
2
3
4
5
6
pg_query("SELECT membre.id_membre, article.id_article, photo_membre, pseudo_membre, titre_article, contenu_article, categorie_article, 
occasion_article, tranche_age_article, sexe_article, licence_article, parution_article, vision_article, COUNT(id_note) AS NombreNote, id_juge
FROM article, membre, note_article
WHERE article.id_article = ".$IdArticle."
AND article.id_article = note_article.id_article
AND membre.id_membre = article.id_membre") or die(pg_result_error());

Voici l'erreur qui est retournée :

Warning: pg_query() [function.pg-query]: Query failed: ERREUR: la colonne « membre.photo_membre » doit apparaître dans la clause GROUP BY ou doit être utilisé dans une fonction d'agrégat LINE 1: SELECT membre.id_membre, article.id_article, photo_membre, p... ^

Même si j'ajoute le GROUP BY à id_membre j'ai le même message avec les autres champs de ma requête.

membre.id_membre est la clé primaire de la table membre
article.id_article est la clé primaire de la table article

Merci de m'aider.

[SQLpro]

C'est normal, MySQL est une vrai passoire et laisse des requêtes sciemment fausses sans complexe donnant des résultats souvent faux.

Commencez par apprendre le langage SQL, et notamment le principe du GROUP BY :
1 - La présence de la clause GROUP BY est nécessaire dès que la clause de sélection, ou le filtre WHERE, ou encore les jointures comportent simultanément des calculs d'agrégation et la présence de colonnes de table hors de calculs d'agrégation.
2 - De plus, toutes les colonnes représentées hors des calculs d'agrégation doivent figurer dans la clause GROUP BY.
A lire : http://sqlpro.developpez.com/cours/sqlaz/ensembles/

De ce fait vous devez impérativement mettre dans votre GROUP BY toutes les colonnes, sauf celles agrégées !

A +

[Vict0]

Bonjour,

Merci d'avoir pris le temps de me répondre.
J'ai suivi vos conseils en ésperant ne pas avoir fait d'erreur :

Code :

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

1
2
3
4
5
6
7
8
9
 
"SELECT membre.id_membre, article.id_article, photo_membre, pseudo_membre, titre_article, contenu_article, categorie_article, 
occasion_article, tranche_age_article, sexe_article, licence_article, parution_article, vision_article, COUNT(id_note) AS NombreNote, id_juge
FROM article, membre, note_article
WHERE article.id_article = ".$IdArticle."
AND article.id_article = note_article.id_article
AND membre.id_membre = article.id_membre
GROUP BY membre.id_membre, article.id_article, photo_membre, pseudo_membre, titre_article, contenu_article, categorie_article,
occasion_article, tranche_age_article, sexe_article, licence_article, parution_article, vision_article, id_juge";

Je n'ai plus de message d'erreur venant de PHP mais le résultat provenant de la requête est vide.

Pourtant quand je vérifie dans ma table article, il y a bien un article qui correspond et un membre aussi.

Qu'en pensez-vous ?

Edit : On m'a proposé une solution :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
SELECT membre.id_membre, id_article, photo_membre, pseudo_membre, 
titre_article, contenu_article, categorie_article, 
occasion_article, tranche_age_article, sexe_article, 
licence_article, parution_article, vision_article, 
nombre_notes
 
FROM article
  JOIN membre USING (id_membre)
  JOIN (SELECT id_article, id_juge, count(*) AS nombre_notes FROM note_article GROUP BY id_article, id_juge) n USING (id_article)
WHERE id_article = ".$IdArticle

Je n'ai aucune erreur, mais la requête ne retourne pas de résultat.

Merci.

[SQLpro]

Évitez d'utiliser le USING pour les jointures (comme le NATURAL JOIN). Ce sont des gadget pour benêt ! On se sait pas sur quelle table il va faire les jointures. En supposant par exemple que votre colonne existe dans trois tables, comment résoud-il le USING ?
Bref, soyez clair net et précis, mettez des prédicat de jointure avec ON et précisez la provenance de toutes vos colonnes avec des alias de table.

Code :

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

1
2
3
4
5
6
7
FROM article AS A
  JOIN membre AS M 
         ON A.id_membre = M.id_membre
  JOIN (SELECT id_article, id_juge, count(*) AS nombre_notes 
        FROM   note_article GROUP BY id_article, id_juge) AS N 
         ON ??? = N.id_article
WHERE id_article = ".$IdArticle

Bref, apprenez SQL. Mon livre comme mon site web peuvent vous y aider...

A +

[Vict0]

Bonjour,

Ce matin j'ai lu quelques uns de vos articles sur SQLPro entres autres les jointures SQL

Voici ce que j'ai tenté :

Code :

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

1
2
3
4
5
6
7
8
9
10
SELECT 	article.id_article, titre_article, contenu_article, categorie_article, occasion_article, tranche_age_article, 
	sexe_article, licence_article, parution_article, vision_article,
	article.id_membre, pseudo_membre, sexe_membre, photo_membre
FROM 	article
JOIN 	membre ON article.id_membre = membre.id_membre
LEFT OUTER JOIN (
	SELECT note_article.id_article, id_juge, COUNT(*) AS NombreNote 
	FROM note_article GROUP BY id_article, id_juge) AS N 
	ON article.id_article = N.id_article
WHERE 	article.id_article = $IdArticle;

En utilisant la fonction print_r() sur mysql_fetch_assoc() la requête retourne bien des résultats.

Seulement, l'index NombreNote n'existe pas.
Je précise que la table note_article est vide.

Merci.

[iberserk]

Seulement, l'index NombreNote n'existe pas

L'index??? quid? Vous voulez dire la colonne ici de la sous requête ayant pour alias N?

Je ne comprends pas pourquoi vous dites qu'elle n'existe pas? elle n'est pas présente dans le SELECT principal...

Ceci dis elle sera à NULL si vous l'y ajoutez puisque note_article est vide...

[Vict0]

Bonsoir,

Dans ma requête SQL, je souhaite avoir le nombre de vote pour l'article. il s'agit de "N".

Si le résultat est NULL, mon script en PHP affichera 0. Sinon il affichera la valeurs retournée par la requête.

Vous m'avez mis la puce à l'oreille, grâce à votre post, j'ai eu l'idée d'ajouter N.NombreNote au SELECT principal de ma requête.

Problème résolu !

[iberserk]

Si le résultat est NULL, mon script en PHP affichera 0. Sinon il affichera la valeurs retournée par la requête.

Vous pouvez aussi faire ceci dans votre SELECT:

Code :

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

1
2
SELECT ....,COALESCE(NombreNote ,0) AS NombreNote 
FROM ...

Ce qui vous évite d'avoir des NULL dans NombreNote... vous aurez directement 0 à la place.

[Vict0]

Merci pour cette astuce.