Discussion :

[WeDgEMasTeR]

Bonjour,

Je galère avec une requete (voir ci-dessous)
le problème avec cette requête c'est qu'elle affiche que les entrés qui vérifie les conditions de la table products_options_price pop

Mais moi j'aimerai qu'elle affiche aussi tout les autres entrées meme si elle n'existe pas pas dans cette table...

Je crois qu'il faire une jointure mais j'arrive pas à la faire...


Quelqu'un pourrait me donner un coup de pouce ?

Merci

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 pop.promo, pop.price, p2c.categories_id, cd.categories_name, c.parent_id, pd.products_id, pd.products_name, p.products_ref, p.products_price, s.specials_new_products_price, couleur.products_options_values_id, couleur.products_options_values_name AS couleur,taille.products_options_values_name AS taille, pa.options_values_price, (
pa.options_values_price + p.products_price ) AS final_price, (pa.options_values_price + s.specials_new_products_price ) AS final_price_promo
FROM products_stock ps, products_description pd LEFT JOIN specials s USING(products_id), products_options_values couleur, products_options_values taille, products_attributes pa, products p, products_to_categories p2c, categories c, categories_description cd, products_options_price pop
WHERE pd.products_id = ps.products_id
AND pd.products_id = pa.products_id
AND pd.products_id = p.products_id
AND p2c.products_id = pd.products_id
AND p2c.categories_id = c.categories_id
AND cd.categories_id = c.categories_id
AND c.categories_id <> 64
AND c.parent_id = 1
AND p.products_status <> 0
AND pop.products_id = pd.products_id
AND pop.couleur_id = couleur.products_options_values_id
AND pop.taille_id = taille.products_options_values_id
AND SUBSTRING( ps.products_stock_attributes FROM 3 FOR 2 ) = couleur.products_options_values_id
AND SUBSTRING( ps.products_stock_attributes FROM 8 FOR 2 ) = taille.products_options_values_id
AND pa.options_values_id = taille.products_options_values_id
GROUP BY pd.products_id,couleur
ORDER BY p2c.categories_id, pd.products_name, couleur.products_options_values_id, taille.products_options_values_id

[mrcmrc]

C'est le cas il faut faire un LEFT JOIN:

Code :

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

1
2
3
4
5
6
 
...
FROM
....
LEFT JOIN
products_options_price pop ON pop.products_id = pd.products_id

Par ailleurs, c'est un peu le bronx ta requête, tu t'y retrouveras mieux en écrivant proprement:

Code :

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

1
2
3
4
5
6
7
 
FROM
 table2
INNER JOIN table2 ON table1.id=table2.id
INNER JOIN table3 ON table2.id=table3.id
LEFT JOIN ...
...

En plus d'être plus correct, cela fera un peu de place dans ton WHERE qui surcharge un peu.

[WeDgEMasTeR]

Merci pour ta réponse, j'ai essayé de faire un left join,
mais comment inclure ces conditions:

Code :

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

1
2
AND pop.couleur_id = couleur.products_options_values_id
AND pop.taille_id = taille.products_options_values_id

Je les laisse dans le where ?

[CinePhil]

Quand je tombe sur requête aussi complexe et qui utilise l'ancienne syntaxe de jointure obsolète depuis 1992, j'essaie de la transformer en appliquant la syntaxe normalisée avec JOIN.
Voilà ce que ça donne pour ta requête, avec un peu de mise en forme, ce qui ne peut pas faire de mal pour la compréhension :

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
25
26
27
28
29
30
31
32
SELECT pop.promo, pop.price, 
  p2c.categories_id, 
  cd.categories_name, 
  c.parent_id, 
  pd.products_id, pd.products_name, 
  p.products_ref, p.products_price, 
  s.specials_new_products_price, 
  couleur.products_options_values_id, couleur.products_options_values_name AS couleur,
  taille.products_options_values_name AS taille, 
  pa.options_values_price, 
  (pa.options_values_price + p.products_price ) AS final_price, 
  (pa.options_values_price + s.specials_new_products_price ) AS final_price_promo
FROM products_stock AS ps
JOIN products_description AS pd ON pd.products_id = ps.products_id
LEFT JOIN specials AS s USING(products_id)
JOIN products_options_values AS couleur ON  pop.couleur_id = couleur.products_options_values_id
JOIN products_options_values AS taille ON
  JOIN products_attributes AS pa ON pd.products_id = pa.products_id
  JOIN products AS p ON pd.products_id = p.products_id
  JOIN products_to_categories AS p2c ON p2c.products_id = pd.products_id
    JOIN categories AS c ON p2c.categories_id = c.categories_id
      JOIN categories_description AS cd ON cd.categories_id = c.categories_id
  JOIN products_options_price AS pop ON pop.products_id = pd.products_id
WHERE c.categories_id <> 64
  AND c.parent_id = 1
  AND p.products_status <> 0
  AND pop.taille_id = taille.products_options_values_id
  AND SUBSTRING( ps.products_stock_attributes FROM 3 FOR 2 ) = couleur.products_options_values_id
  AND SUBSTRING( ps.products_stock_attributes FROM 8 FOR 2 ) = taille.products_options_values_id
  AND pa.options_values_id = taille.products_options_values_id
GROUP BY pd.products_id,couleur
ORDER BY p2c.categories_id, pd.products_name, couleur.products_options_values_id, taille.products_options_values_id

Problème : J'ai déjà épuisé toutes les tables qui figuraient dans FROM et j'ai encore des conditions de jointure :

Code :

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

1
2
3
4
  AND pop.taille_id = taille.products_options_values_id
  AND SUBSTRING( ps.products_stock_attributes FROM 3 FOR 2 ) = couleur.products_options_values_id
  AND SUBSTRING( ps.products_stock_attributes FROM 8 FOR 2 ) = taille.products_options_values_id
  AND pa.options_values_id = taille.products_options_values_id

Généralement, ça veut dire qu'il manque des instances de tables pour compléter correctement les jointures.
Sans le schéma de la base de données, il va être un peu difficile d'aller plus loin.
A toi de continuer ce travail de normalisation de ta requête, ensuite tu trouveras où placer un ou plusieurs LEFT devant des JOIN.

[WeDgEMasTeR]

hm ok merci!

Je vais m'y pencher,
j'avoue que j'ai pris la mauvaise habitude de faire à l'ancienne...
Et c'est sur ce genre de requête où je trouve les limites des anciennes methodes

[WeDgEMasTeR]

J'ai quand meme un peu du mal a raisonner...

j'ai essayé cette requete:

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
25
26
27
28
29
30
31
SELECT pop.promo, pop.price, 
  p2c.categories_id, 
  cd.categories_name, 
  c.parent_id, 
  pd.products_id, pd.products_name, 
  p.products_ref, p.products_price, 
  s.specials_new_products_price, 
  couleur.products_options_values_id, couleur.products_options_values_name AS couleur,
  taille.products_options_values_name AS taille, 
  pa.options_values_price, 
  (pa.options_values_price + p.products_price ) AS final_price, 
  (pa.options_values_price + s.specials_new_products_price ) AS final_price_promo
FROM products_stock AS ps
JOIN products_description AS pd ON pd.products_id = ps.products_id
LEFT JOIN specials AS s USING(products_id)
LEFT JOIN products_options_values AS couleur ON  pop.couleur_id = couleur.products_options_values_id
LEFT JOIN products_options_values AS taille ON pop.taille_id = taille.products_options_values_id
  JOIN products_attributes AS pa ON pd.products_id = pa.products_id
  JOIN products AS p ON pd.products_id = p.products_id
  JOIN products_to_categories AS p2c ON p2c.products_id = pd.products_id
    JOIN categories AS c ON p2c.categories_id = c.categories_id
      JOIN categories_description AS cd ON cd.categories_id = c.categories_id
  JOIN products_options_price AS pop ON pop.products_id = pd.products_id
WHERE c.categories_id <> 64
  AND c.parent_id = 1
  AND p.products_status <> 0
  AND SUBSTRING( ps.products_stock_attributes FROM 3 FOR 2 ) = couleur.products_options_values_id
  AND SUBSTRING( ps.products_stock_attributes FROM 8 FOR 2 ) = taille.products_options_values_id
  AND pa.options_values_id = taille.products_options_values_id
GROUP BY pd.products_id,couleur
ORDER BY p2c.categories_id, pd.products_name, couleur.products_options_values_id, taille.products_options_values_id

Mais il m'affiche une erreur suivante:

1052 - Column 'products_id' in from clause is ambiguous

Je comprend pas quel products_id causerai un problème..
c'est pas simple a tout assimiler...

[CinePhil]

Je pense que c'est products_id dans le USING qui est ambigü.

Il vaut mieux utiliser la syntaxe complète :

Code :

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

1
2
FROM uneTable AS a
JOIN autreTable AS b ON a.UneColonne = b.UneColonne

[WeDgEMasTeR]

Merci j'ai bien compris maintenant