Discussion :

[txoovlis]

Bonjour à tous,
Voici mon problème.
J'ai 4 tables

Code :

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

1
2
3
4
5
 
tbl_magasin     tbl_produit    tbl_stock       tbl_vente
   no_mag          cod_pro        no_mag          no_mag
   cumul                          cod_pro         cod_pro
                                  qte             qte

J'ai plusieurs magasins. Pour un magasin cumul peut être à 0 ou 1.
Pour un produit je peux ne pas avoir de stock ni de vente.
Je voudrais afficher par produit pour lequel tbl_magasin.cumul=1
les stock et les ventes.
J'ai commencé à écraire ma requête là je bloque.

Code :

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

1
2
3
4
5
6
7
select 
tbl_produit.cod_pro,
sum(tbl_stock.qte) as stock,
sum(tbl_vente.qte) as vente
from tbl_produit
       left outer join tbl_stock on tbl_stock.cod_pro=tbl_produit.cod_pro
       letf outer join tbl_vente on tbl_vente_cod_pro=tbl_produit.cod_pro

??? comment faire intervenir no_magasin et cumul ???

Quelqu'un pourra-il avoir l'amabilité de m'aider ?
D'avance merci.
Txoovlis

[Maljuna Kris]

Saluton,
Pourrais-tu être un peu plus lisible s'agissant de structures de table et nous donner les colonnes sur lesquelles sont censées porter les relations.
Merci

[txoovlis]

Excuse-moi.
Voici les colonnes de mes tables
tbl_magasin : no_mag,cumul
tbl_produit : cod_pro
tbl_stock : no_mag,cod_pro,qte
tbl_vente : no_mag,cod_pro,qte

[Maljuna Kris]

Tu as un souci pour faire les jointures, car tu n'accèdes au magasin à partir du produit qu'en passant soit par les stocks, soit par les ventes.
Comme, conceptuellement, ce qui a été vendu a procédé du stock d'un magasin on se dit que ça va le faire.
Essaye comme çà :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
SELECT p.cod_pro,
           SUM(s.qte) AS stock,
           SUM(v.qte) AS vente,
FROM tbl_produit AS p
LEFT JOIN tbl_stock AS s ON s.cod_pro=p.cod_pro
LEFT JOIN tbl_vente  AS v ON v.cod_pro=p.cod_pro
LEFT JOIN tbl_magasin AS m 
             ON m.no_mag=s.no_mag OR m.no_mag=v.no_mag
WHERE m.cumul=1

Bon, le OR m.no_mag=v.no_mag dans la clause de jointure devrait nous faire faire une crise d'urticaire, mais je trouve amusant de l'essayer.

[Maljuna Kris]

En y réfléchissant, le WHERE m.cumul=1 rend inutile les LEFT JOIN.
En effet, si la jointure ne s'opère pas, m.cumul sera NULL et la ligne de produit ne sera pas retournée.
Donc on pourrait se contenter de INNER JOIN.

[txoovlis]

J'ai essayé mais ça marche bizarrement.
D'abord j'ai ajouté le group by p.cod_pro sinon il ne m'affiche pas tous les produits

Code :

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

1
2
3
4
5
6
7
8
9
SELECT p.cod_pro,
           SUM(s.qte) AS stock,
           SUM(v.qte) AS vente,
FROM tbl_produit AS p
LEFT JOIN tbl_stock AS s ON s.cod_pro=p.cod_pro
LEFT JOIN tbl_vente  AS v ON v.cod_pro=p.cod_pro
LEFT JOIN tbl_magasin AS m 
             ON m.no_mag=s.no_mag OR m.no_mag=v.no_mag
WHERE m.cumul=1

Avec le OR il m'affiche bien tous mes produits, mais les quantités sont fausses
J'ai remplacé le OR par le AND je n'ai plus tous les produits et je n'ai plus toutes mes quantités vendues

[Maljuna Kris]

J'ai essayé mais ça marche bizarrement.
D'abord j'ai ajouté le group by p.cod_pro sinon il ne m'affiche pas tous les produits

Oui, bravo parce que sur ce coup là on était vraiment boulets.

Avec le OR il m'affiche bien tous mes produits, mais les quantités sont fausses
J'ai remplacé le OR par le AND je n'ai plus tous les produits et je n'ai plus toutes mes quantités vendues

Je t'avais prévenu que ça risquait d'être folklo cette jointure à 3.
Donc, a priori, tu as certains produits qui ont été vendus mais qui ne sont pas connus en stock pour ce magasin et réciproquement.
Bref, c'est le souk !
Je crains qu'il ne nous faille passer par deux sous-requêtes dans la clause SELECT, ce qui est très lourd en charge pour le serveur.

[Maljuna Kris]

Et comme ça ?

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 p.cod_pro,
           (SELECT SUM(s.qte) 
            FROM tbl_stock AS s
            LEFT JOIN tbl_magasin AS m
                ON m.no_mag=s.no_mag
            WHERE s.cod_pro=p.cod_pro
            AND m.cumul=1) AS stock,
           (SELECT SUM(v.qte)
            FROM tbl_vente AS v 
            LEFT JOIN tbl_magasin AS m1
            ON m1.no_mag=v.no_mag
            WHERE v.cod_pro=p.cod_pro
            AND m1.cumul=1) AS vente
FROM tbl_produit AS p

[txoovlis]

Ca marche !!!
Mais un peu long. un peu plus de 3 secondes pour 3268 produits.
Merci beaucoup

[Maljuna Kris]

Je t'avais prévenu, les sous-requêtes corrélées c'est très gourmand.
Vérifie que toutes les colonnes qui participent aux jointures et aux corrélations sont bien des index dans chacune des tables, ça pourrait accélérer le bouzin.
Mais je présume que c'est déjà le cas.

[Sivrît]

Il serait peut-être plus efficace de faire deux requêtes, une pour chacune des sommes (à tester quoi).

Code :

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

1
2
3
4
5
6
7
 
SELECT p.cod_pro, SUM(s.qte*m.cumul) 
FROM tbl_produit AS p
LEFT JOIN tbl_stock AS s ON s.cod_pro=p.cod_pro
LEFT JOIN tbl_magasin AS m ON m.no_mag=s.no_mag
GROUP BY p.cod_pro
ORDER BY p.cod_pro

Et de même pour les ventes. C'est un peu moins pratique mais avec le tri le rapprochement entre les deux listes ne devrait pas être trop difficile. Ce serait probablement plus rapide avec des INNER JOIN et en partant des magasins (afin de filtrer directement ceux qui ne sont pas à cumuler), mais dans ce cas un produit qui n'apparait dans aucun stock cumulé ne sera pas listé.

D'un autre côté, si c'est pour être utilisé tel quel je suppose qu'il faut une unique requête, mais dans ce cas elle n'a pas forcément besoin d'être excessivement rapide. J'ai un peu de mal à juger de si les 3 secondes sont normales... Je serais tenté que c'est un peu lent pour des tables indexées mais bon. Les sous-requêtes devraient fonctionner à l'identique avec des INNER JOIN, reste à voir ce que ça peut faire gagner.