Discussion :

[HAbroc]

Bonjour à tous,

J'utilise cette requête :

Code :

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

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

[Nudger]

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 ?

[HAbroc]

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 :

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

ON shop_monnaies.id=shop_offres.article

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

[CinePhil]

C'est une horreur cette requête !

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

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 !

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 :

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

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 :

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.

[HAbroc]

Code :

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

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

Code :

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

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

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

Code :

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

HAVING o.max(up) < o.pr

[CinePhil]

Code :

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

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

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

Code :

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

HAVING o.max(up) < o.pr

Pas tout à fait :

Code :

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

HAVING MAX(o.up) < o.pr

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