Discussion :

[widi70]

Bonjour, je fais une recherche pour mon site internet pour filtrer mes produits.
Ma requête ressemble à ceci :

Code :

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

select * from Gamme gd, AssociationGamme a, machine m LEFT JOIN marque ma ON m.marqueMachine=ma.idMarque, gerer g,ProduitFiltre p where g.idproduit=m.codeMachine and g.idSousFam='118' and ma.idMarque='45' And (p.idFiltre='2' and substr(p.Valeur,1,2)>=10 AND substr(p.Valeur,1,2)<=19) or (p.idFiltre='3' and p.Valeur<15)

Mais le résultat est

Code :

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

Query execution was interrupted

Pourtant mes tables ne sont pas très lourdes...
Comment faire?
Merci de votre aide

[aieeeuuuuu]

Bonjour,

Il semble manquer un certain nombre de condition de jointure dans votre requête.
Vous semblez faire de multiple produit cartésien, ce qui pourrait expliquer votre problème.

Pourquoi mixer les jointures dans le WHERE et les jointures normalisées (votre jointure externe) ?

Je pense qu'en écrivant toutes vos jointures sous la forme ... JOIN ... ON ... vous verriez immédiatement vos erreurs.

[CinePhil]

Effectivement...
1) Les jointures s'écrivent depuis plus de 20 ans avec l'opérateur JOIN ; il serait temps de s'y mettre !

2) Inutile d'entourer les valeurs numériques avec des apostrophes !

3) Un code indenté et aéré est plus agréable à lire et à débugguer.

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

Voici votre requête remise en forme :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
select les_colonnes_necessaires_et_pas_etoile
from Gamme gd
JOIN AssociationGamme a ON -- manque condition de jointure !
JOIN machine m ON -- manque condition de jointure ! 
JOIN gerer g ON g.idproduit = m.codeMachine
LEFT JOIN marque ma ON m.marqueMachine = ma.idMarque
JOIN ProduitFiltre p ON -- manque condition de jointure ! 
where  g.idSousFam = 118 
	and ma.idMarque = 45 
	And 
	(
		p.idFiltre = 2 
		and substr(p.Valeur,1,2) >= 10 
		AND substr(p.Valeur,1,2) <= 19
	) 
	or 
	(
		p.idFiltre = 3 
		and p.Valeur < 15
	)

Vous remarquerez immédiatement qu'il manque des conditions de jointure.

[widi70]

OK alors j'ai refait ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
select * from GammeDestructeur gd, 
AssociationGamme a, 
machine m LEFT JOIN marque ma ON m.marqueMachine=ma.idMarque, 
gerer g,
ProduitFiltre p 
where 
gd.idGamme=a.idGamme
AND a.CodeMachine=m.codeMachine 
AND m.codeMachine=g.idproduit
And m.codeMachine=p.codeMachine
and g.idSousFam='118' 
and ma.idMarque='45' 
And (p.idFiltre='2' and substr(p.Valeur,1,2)>=10 
AND substr(p.Valeur,1,2)<=19) or (p.idFiltre='3' and p.Valeur<15)

J'ai fait sans les jointures, car je ne sais pas trop comment ça fonctionne lorsqu'il y a plusieurs jointures pour une même table....
Merci de ton aide.

[CinePhil]

Il suffit de faire comme j'ai fait pour réécrire ta requête :

À la place des virgules entre les tables dans la partie FROM, tu passes à la ligne et tu met un JOIN. Et derrière l'alias de la table, tu mets un ON.
Ensuite, les conditions de jointure qui sont dans le WHERE, tu les déplaces au bon endroit derrière les ON.

[widi70]

oups je m'excuse CinePhil je n'avais pas vu votre réponse...
Merci beaucoup de votre aide

[widi70]

J'avais mis résolu mais en fait non....

Code :

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

1
2
3
4
5
6
7
8
9
10
11
select * from 
Gamme gd JOIN AssociationGamme a ON gd.idGamme=a.idGamme 
JOIN machine m ON a.CodeMachine=m.codeMachine 
JOIN gerer g ON m.codeMachine =g.idproduit 
LEFT JOIN marque ma ON m.marqueMachine = ma.idMarque 
JOIN ProduitFiltre p ON m.codeMachine=p.codeMachine 
where 
g.idSousFam='118' 
And (p.idFiltre='2' and substr(p.Valeur,1,2)>=10 AND substr(p.Valeur,1,2)<=19) 
or (p.idFiltre='14' and p.Valeur='1') 
and m.occassion='0' group by gd.idGamme order by prixMachine

Si je ne mets que 1 filtre, il m’affiche les bons produits, mais si je mets 2 filtres, il prend soit l'un ou soit l'autre, mais pas les deux conditions. J'ai donc remplacé le "or" par un "and", mais ça ne fonctionne pas non plus...
Merci de votre aide

[aieeeuuuuu]

C'est logique non ?
OR fait un OU logique : ça renvoie les lignes qui vont satisfaire une des deux conditions.

AND fait un ET logique : ça renvoie les lignes qui vont satisfaire les deux conditions, soit aucune dans votre cas tant que 2 ne sera pas égal à 14 (ce qui ne devrait pas arriver prochainement )

[widi70]

Comment est-ce que je peux lui dire alors que
si (p.idFiltre=='2') il faut que substr(p.Valeur,1,2)>=10 AND substr(p.Valeur,1,2)<=19
et si (p.idFiltre='14') il faut que p.Valeur='1'

J'avoue que je suis perdu.

[skuatamad]

Quelque chose comme ça :

Code :

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

1
2
3
4
5
case when p.idFiltre ='2' and substr(p.Valeur,1,2)>=10 AND substr(p.Valeur,1,2)<=19
     then 1
     when p.idFiltre='14' and p.Valeur='1'
     then 1
 end = 1

[al1_24]

Et pourquoi pas tout simplement

Code :

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

1
2
   p.idFiltre = '2' AND SUBSTR(p.Valeur, 1, 2) * 1 BETWEEN 10 AND 19
OR p.idFiltre = '14' AND p.Valeur = '1'

[aieeeuuuuu]

comment est ce que je peut lui dire alors que
si (p.idFiltre=='2') faut que substr(p.Valeur,1,2)>=10 AND substr(p.Valeur,1,2)<=19
et si (p.idFiltre='14') faut que p.Valeur='1'

Voulez-vous garder également les lignes pour lesquelles idFiltre ne vaut ni 2 ni 14 ?

Dans ce cas, comme ceci :

Code :

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

1
2
3
4
5
 
WHERE 
    (p.idFiltre<>'2' OR (substr(p.Valeur,1,2)>=10 AND substr(p.Valeur,1,2)<=19 ))
AND 
    (p.idFiltre<>'14' OR p.Valeur='1')

S'il est possible que idFiltre soit NULL, il faudra intégrer cette possibilité...

[widi70]

Voici ma requête :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
select URLGamme, prixMachine, idDisponibiliteMachine, NomGamme,photo1,DesCourteGamme,VideoGamme, 
case 
	when p.idFiltre ='2' and SUBSTR(p.Valeur, 1, 2) * 1 BETWEEN 10 AND 19 then 1 
	when p.idFiltre='14' and p.Valeur='1' 
then 1 
end = 1	
from GammeDestructeur gd JOIN AssociationGamme a ON gd.idGamme=a.idGamme 
JOIN machine m ON a.CodeMachine=m.codeMachine 
JOIN gerer g ON m.codeMachine =g.idproduit 
LEFT JOIN marque ma ON m.marqueMachine = ma.idMarque 
JOIN ProduitFiltre p ON m.codeMachine=p.codeMachine 
where g.idSousFam='118' 
and ma.idMarque='45' 
and m.occassion='0' 
group by gd.idGamme 
order by prixMachine

Mais malheureusement aucun changement ça ne m'affiche pas ce que je veux...
Merci de votre aide

[SQLpro]

Code :

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

1
2
3
4
5
6
7
select ...
 
where 
g.idSousFam='118' 
And (p.idFiltre='2' and substr(p.Valeur,1,2)>=10 AND substr(p.Valeur,1,2)<=19) 
or (p.idFiltre='14' and p.Valeur='1') 
and m.occassion='0' group by gd.idGamme order by prixMachine

Au passage il est assez remarquable de voir une condition WHERE aussi mal écrite !
En effet, tantôt vous considérez que Valeur est numérique tantôt que c'est une chaîne de caractères....
Bonjour les performances de cette lamentable écriture !

Pensez un jour à vous former sur le langage SQL ! Mon site web comme mon bouquin peuvent vous y aider....

A +

[widi70]

Merci pour cet aimable commentaire

[skuatamad]

Le CASE est à placer dans la partie filtre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
select URLGamme, prixMachine, idDisponibiliteMachine, NomGamme,photo1,DesCourteGamme,VideoGamme, 
from GammeDestructeur gd JOIN AssociationGamme a ON gd.idGamme=a.idGamme 
JOIN machine m ON a.CodeMachine=m.codeMachine 
JOIN gerer g ON m.codeMachine =g.idproduit 
LEFT JOIN marque ma ON m.marqueMachine = ma.idMarque 
JOIN ProduitFiltre p ON m.codeMachine=p.codeMachine 
where g.idSousFam='118' 
and ma.idMarque='45' 
and m.occassion='0' 
and case 
	when p.idFiltre ='2' and SUBSTR(p.Valeur, 1, 2) * 1 BETWEEN 10 AND 19 then 1 
	when p.idFiltre='14' and p.Valeur='1' then 1 
end = 1
group by gd.idGamme 
order by prixMachine

[widi70]

OK merci parfait....
À bientôt

[CinePhil]

1) La condition du WHERE sur la table marque doit être déportée vers la condition de jointure sinon on se retrouve avec une jointure interne.

2) Pourquoi un GROUP BY alors qu'il n'y a pas de fonction d'aggrégation ?

comment est ce que je peut lui dire alors que
si (p.idFiltre=='2') faut que substr(p.Valeur,1,2)>=10 AND substr(p.Valeur,1,2)<=19
et si (p.idFiltre='14') faut que p.Valeur='1'

Ne serait-ce pas ceci ?
(p.idFiltre=='2' ET substr(p.Valeur,1,2)>=10 ET substr(p.Valeur,1,2)<=19)
OU
(p.idFiltre='14' ET p.Valeur='1')

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
SELECT URLGamme, prixMachine, idDisponibiliteMachine, NomGamme,photo1,DesCourteGamme,VideoGamme, 
FROM GammeDestructeur gd 
JOIN AssociationGamme a ON gd.idGamme=a.idGamme 
	JOIN machine m ON a.CodeMachine=m.codeMachine 
		JOIN gerer g ON m.codeMachine =g.idproduit 
		JOIN ProduitFiltre p ON m.codeMachine=p.codeMachine 
		LEFT JOIN marque ma 
			ON m.marqueMachine = ma.idMarque 
			AND ma.idMarque = 45
WHERE g.idSousFam =  118 
	AND m.occassion = 0 
	AND 
	(
		(
			p.idFiltre = 2 
			AND CAST(SUBSTR(p.Valeur, 1, 2) AS UNSIGNED) BETWEEN 10 AND 19
		)
		OR
		(
			p.idFiltre = 14 
			AND p.Valeur = '1'
		)
	)
ORDER BY prixMachine

EDIT : Suite à la judicieuse remarque de Waldar, remplacement de la multiplication de la SUBSTR par un CAST AS UNSIGNED. Ajout des apostrophes autour de la valeur dans la dernière condition puisqu'il semble que Valeur soit de type alphanumérique.

Ça semble être un beau bordel dans cette BDD !

[Waldar]

1) Que représente le GROUP BY à la fin, c'est pour faire joli ?
Il n'y a aucun agrégat dans la requête, je doute de son utilité.
Edit : Cinephil a été plus rapide

2) Il manque les alias sur toutes les colonnes du select.

3) L'indentation l'indentation l'indentation ! Utilisez au moins une mise en forme, celle que vous voulez mais vous ne pouvez pas laisser tout ce code tout moche comme ça.
Utilisez aussi des alias de table de même longueur. Ça peut paraître idiot, mais avoir les colonnes bien alignées facilite grandement la relecture.

4) Comme l'a soulevé SQLPro, il y a un problème avec votre colonne Valeur. Si la multiplication par 1 est rigolote, elle peut d'une part être source de problème et d'autre part elle m'apparaît inutile, on peut comparer des chaînes de caractères. Par ailleurs, c'est bien un varchar ?

5) Quels sont les types des colonnes idMarque, idFiltre, idSousFam et Occasion ? Actuellement on n'est pas sûr s'il faut mettre des chaînes ou des nombres.