clause du WHERE non prise en compte

Bonjour à tous,

J'utilise cette requête :

Code:

---

1 2 3 4 5 6 7 8

SELECT shop_monnaies.*,shop_offres.* FROM shop_offres INNER JOIN shop_monnaies WHERE shop_monnaies.client = %s and shop_monnaies.vendue='INVENDUE' GROUP BY shop_offres.article DESC HAVING max(up)<pr ORDER BY shop_monnaies.id DESC

---

et je viens de m'apercevoir que si je mets dans ma base vendue='STOP' , par exemple, la requête m'affiche toujours l'objet ???
Pire! J'ai fait le test en enlevant shop_monnaies.vendue='INVENDUE' de la requête , et le résultat est inchangé ?

Habroc

Bonjour,

Le critère de regrouppement ne peut fonctionner avec avec le contenu de votre clause SELECT.

Quel est l'intérêt de la clause group by dans votre requête ?

Citation:

---

Envoyé par Nudger

Quel est l'intérêt de la clause group by dans votre requête ?

---

Dans shop_offres , il y a des articles qui peuvent avoir reçu plusieurs offres , et je ne veux afficher ces articles qu'une seule fois...

J'ai ajouté :

Code:

---

ON shop_monnaies.id=shop_offres.article

---

après le INNER JOIN et ça semble fonctionner correctement.

C'est une horreur cette requête ! :aie:

1) Il vaut mieux éviter la guerre des étoiles ! :P

2) Il n'y a pas de condition de jointure.
Apparemment, d'après votre dernier message, vous avez corrigé ce point.

3) DESC dans le GROUP BY est inutile. Il n'y a pas d'ordre dans un GROUP BY.
C'est même miraculeux que le SGBD ne donne pas une erreur de syntaxe sur ce point.
Encore une tolérance trompeuse du mauvais MySQL ! :roll:

4) Toutes les colonnes du SELECT ne faisant pas l'objet d'une fonction de groupage doivent figurer dans le GROUP BY sous peine de voir des valeurs aléatoires pour les colonnes manquantes.

5) L'utilisation d'alias pour les tables est fortement recommandé dès qu'il y a plus d'une table dans la requête. Cela facilite l'écriture et la lecture de celle-ci. Il faut ensuite utiliser systématiquement ces alias devant chaque colonne nommée pour savoir facilement de quelle table elle vient.

Voici votre requête récrite d'une manière plus correcte :

Code:

---

1 2 3 4 5 6 7 8

SELECT o.article FROM shop_offres o INNER JOIN shop_monnaies m ON m.id = o.article WHERE m.client = %s AND m.vendue = 'INVENDUE' GROUP BY o.article HAVING max(up) < pr -- de quelles tables viennent ces colonnes ? ORDER BY o.article DESC

---

Quant à votre problème :

Citation:

---

si je mets dans ma base vendue='STOP' , par exemple, la requête m'affiche toujours l'objet ???
Pire! J'ai fait le test en enlevant shop_monnaies.vendue='INVENDUE' de la requête , et le résultat est inchangé ?

---

Expliquez en français ce que vous cherchez à obtenir, donnez la structure de vos tables, un petit jeu de données et le résultat attendu.

Citation:

---

Envoyé par CinePhil

Code:

---

1 2 3 4 5 6 7 8

SELECT o.article FROM shop_offres o INNER JOIN shop_monnaies m ON m.id = o.article WHERE m.client = %s AND m.vendue = 'INVENDUE' GROUP BY o.article HAVING max(up) < pr -- de quelles tables viennent ces colonnes ? ORDER BY o.article DESC

---

---

Merci beaucoup CinePhil pour la mise au point :aie:

Code:

---

HAVING max(up) < pr -- de quelles tables viennent ces colonnes ?

---

Elles proviennent de shop_offres ; donc si j'ai bien compris :cry: , je devrais écrire :

Code:

---

HAVING o.max(up) < o.pr

---

Citation:

---

Envoyé par HAbroc

Code:

---

HAVING max(up) < pr -- de quelles tables viennent ces colonnes ?

---

Elles proviennent de shop_offres ; donc si j'ai bien compris :cry: , je devrais écrire :

Code:

---

HAVING o.max(up) < o.pr

---

---

Pas tout à fait :

Code:

---

HAVING MAX(o.up) < o.pr

---

Mais pas sûr que ça fonctionne ! Vérifiez les résultats obtenus.