Discussion :

[pmboutteau]

Voici mon problème. J'ai une table commande et je veux aller chercher les données de 2 autres tables qui sont liées à la table commande.

Voici ce que je fais:

Code :

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

1
2
3
4
5
SELECT c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.grade, c.NomClient, c1.NumCommande, c1.diametre, c1.epaisseur, c1.TypeDeFiletage, c1.grade, c1.NomClient
FROM Commande AS c INNER JOIN OperationCasing AS o ON c.NumCommande = o.NumCommande, Commande AS c1 INNER JOIN SupervisionCasing AS s ON c1.NumCommande = s.NumCommande
WHERE (((o.NomPoste) In ('Rec.Agt.Csg')) AND ((o.Poste)='c,Bertaux') AND ((o.DateSaisie) Between #1/1/2006# And #2/1/2006#))
GROUP BY c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.grade, c.NomClient, c1.NumCommande, c1.diametre, c1.epaisseur, c1.TypeDeFiletage, c1.grade, c1.NomClient, s.DateSaisie, s.Poste, s.NomPoste
HAVING (((s.DateSaisie) Between #1/1/2006# And #2/1/2006#) AND ((s.Poste)='c,Bertaux') AND ((s.NomPoste) In ('SupQuaCsg')));

N'y at'il pas plus simple que d'avoir 2 tables commandes c et c1?

Merci.

[TheRussian]

Bonjour,

est-ce normal qu'il n'y ai aucun lien entre les tables OperationCasing et SupervisionCasing, si ce n'est pas le cas la requete devrait être la suivante :

Code :

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

1
2
3
4
5
6
7
8
9
 
SELECT c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.grade, c.NomClient
FROM Commande AS c 
         INNER JOIN OperationCasing AS o ON c.NumCommande = o.NumCommande
         INNER JOIN SupervisionCasing AS s ON c.NumCommande = s.NumCommande 
WHERE (((o.NomPoste) In ('Rec.Agt.Csg')) AND ((o.Poste)='c,Bertaux') 
          AND ((o.DateSaisie) Between #1/1/2006# And #2/1/2006#)) 
GROUP BY c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.grade, c.NomClient, s.DateSaisie, s.Poste, s.NomPoste 
HAVING (((s.DateSaisie) Between #1/1/2006# And #2/1/2006#) AND ((s.Poste)='c,Bertaux') AND ((s.NomPoste) In ('SupQuaCsg')));

[pmboutteau]

Ben le lien ce serait le numéro de commande.

Je vais essayer ta requète.

[pmboutteau]

Je vais développer.

J'ai une table commande qui stocke les données d'une commande.

Ensuite j'ai 2 autres tables qui stockent des travaux réalisés sur les commandes.

C'est 2 tables ont pour clés primaires, un t-uples, (Numcommande,DateDeSaisie,Poste,NomPoste)

Je veux récupérer pour un poste et un nomposte donnés toutes les commandes avec leurs caractéristiques entre 2 dates données.

voilà en gros le sujet.


voici ce que j'ai pour l'instant:

Code :

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

1
2
3
SELECT c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.grade, c.NomClient,c1.NumCommande, c1.diametre, c1.epaisseur, c1.TypeDeFiletage, c1.grade, c1.NomClient,o.BoutsTraites/2 As Recette,o.BoutsBons/2 As RebutAgent,s.Supervise,s.NC
FROM Commande AS c INNER JOIN OperationCasing AS o ON c.NumCommande = o.NumCommande, Commande As c1  INNER JOIN SupervisionCasing As s ON c1.NumCommande = s.NumCommande
WHERE (((o.NomPoste) In ('Rec.Agt.Csg')) AND ((o.Poste)='c,Bertaux') AND ((o.DateSaisie) Between #1/1/2006# And #2/1/2006#)) AND  (((s.DateSaisie) Between #1/1/2006# And #2/1/2006#) AND ((s.Poste)='c,Bertaux') AND ((s.NomPoste) In ('SupQuaCsg')));

[Médiat]

Tout cela me paraît bien confus :

• 1) Pourquoi deux copies de la table commande
  2) Pour un GROUP BY sans fonction agrégative
  3) Pourquoi des conditions dans le HAVING qui pourraient être dans le WHERE
  4) Pourquoi un monstrueux produit cartésien ?

Peux-tu décrire ton problème fonctionnellement.

Telle que je comprends la requête, elle ne sert qu'à sélectionner des commandes en fonctions de contraintes sur les autres tables, si c'est le cas, un EXISTS devrait mieux répondre au problème.

[pmboutteau]

J'explique un peu au-dessus. Tu as du répondre pendant que j'éditais. En effet le having et le group by je les ai virés.

pour les 2 copies de la table commande,c'est tout ce que j'ai trouvé pour l'instant pour passer la difficulté mais ce n'est pas très concluant

[Médiat]

Après lecture de ta description, je ferais

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
SELECT c.*
FROM Commande c INNER JOIN (SELECT NumCommande
                            FROM OperationCasing
                            WHERE NomPoste In ('Rec.Agt.Csg') 
                              AND Poste = 'c,Bertaux' 
                              AND DateSaisie Between #1/1/2006# And #2/1/2006#
                           UNION
                            SELECT NumCommande
                            FROM SupervisionCasing
                            WHERE NomPoste In ('SupQuaCsg') 
                              AND Poste = 'c,Bertaux' 
                              AND DateSaisie Between #1/1/2006# And #2/1/2006#) AS a
                 ON a.NumCommande = c.NumCommande

Je ne suis pas certain que tes IN soient judicieux (soit il manque des apostrophes, soit un = suffirait)

[TheRussian]

Si il peut y avoir des actions sur les ommandes dans la table SupervisionCasing et pas dans OperationCasing, je pense qu'il vaut mieux utiliser les jointures externes avec un LEFT JOIN, RIGHT JOIN.

Je ne comprends pas l'utiliter de 2 tables Commande dans le FROM :

[pmboutteau]

J'ai plus de chose dans les IN mais c'était pour pas avoir trop de code.
JE vais essayer ton code.

Merci.

[pmboutteau]

Seulement dans l'union il me faut le même nombre de champs

hors dans supervision je vien cherhcer 2 valeurs et 3 dans opération.

[pmboutteau]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
SELECT  c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.grade, c.NomClient,SUM(a.BoutsTraites)/2 As Tubes
FROM Commande AS c INNER JOIN [SELECT NumCommande,BoutsTraites
                            FROM OperationCasing
                            WHERE NomPoste In ('Rec.Agt.Csg')
                              AND Poste = 'c,Bertaux'
                              AND DateSaisie Between #1/1/2006# And #2/1/2006#
                           UNION
                            SELECT NumCommande,Supervise
                            FROM SupervisionCasing
                            WHERE NomPoste In ('SupQuaCsg')
                              AND Poste = 'c,Bertaux'
                              AND DateSaisie Between #1/1/2006# And #2/1/2006#]. AS a ON c.NumCommande = a.NumCommande GROUP BY c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.grade, c.NomClient;

Bouts traites et Supervise sont différents pour moi je les veux tous les 2.
Là il me fait la somme des 2.

Comment éviter le Union?

[pmboutteau]

J'ai essayé çà

Code :

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

1
2
3
4
5
6
7
SELECT c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.grade, c.NomClient, SUM(a.Tube) AS Tubes,SUM(a.Process) AS Supervise
FROM Commande AS c INNER JOIN (SELECT  (o.NumCommande) as op,(s.NumCommande) as super,Sum(o.BoutsTraites)/2 as Tube,Sum(s.Supervise)/2 as Process
                            FROM OperationCasing AS o INNER JOIN SupervisionCasing AS s ON s.Poste=o.Poste
                            WHERE( (o.NomPoste In ('Rec.Agt.Csg')
                              AND o.Poste = 'c,Bertaux'
                              AND o.DateSaisie Between #1/1/2006# And #2/1/2006#) AND (s.NomPoste In ('SupQuaCsg')  AND s.Poste = 'c,Bertaux' AND s.DateSaisie Between #1/1/2006# And #2/1/2006#)) Group BY o.NumCommande,s.NumCommande) AS a ON (c.NumCommande = a.op OR c.NumCommande=a.super)
GROUP BY c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.grade, c.NomClient;

Seulement là j'ai des doublons dans mes additions!

[pmboutteau]

voici les 2 requètes bonnes séparées. Je voudrais les lier.

requète1:

Code :

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

SELECT c.NumCommande,c.diametre,c.epaisseur,c.TypeDeFiletage,c.Grade,c.NomClient,Sum(o.BoutsTraites)/2 AS TubesTraites,Sum(o.Rebuts) As RebutsAgent FROM Commande as c Inner Join OperationCasing as o ON c.NumCommande=o.NumCommande WHERE o.Poste='C,Bertaux'  AND O.NomPoste='Rec.Agt.Csg' AND o.DateSaisie BETWEEN #1/1/2006# AND #2/1/2006# Group by c.NumCommande,c.diametre,c.epaisseur,c.TypeDeFiletage,c.Grade,c.NomClient ;

requète2:

Code :

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

SELECT c.NumCommande,c.diametre,c.epaisseur,c.TypeDeFiletage,c.Grade,c.NomClient,Sum(o.Supervise) AS TubesSup,Sum(o.NC) As TubesNC FROM Commande as c Inner Join SupervisionCasing as o ON c.NumCommande=o.NumCommande WHERE o.Poste='C,Bertaux'  AND O.NomPoste='SupQuaCsg' AND o.DateSaisie BETWEEN #1/1/2006# AND #2/1/2006# Group by c.NumCommande,c.diametre,c.epaisseur,c.TypeDeFiletage,c.Grade,c.NomClient ;

[Médiat]

Essaye :

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
SELECT c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.Grade, c.NomClient,
       Sum(BoutsTraites)/2 AS TubesTraites,
       Sum(Rebuts)         AS RebutsAgent,
       Sum(Supervise)      AS TubesSup,
       Sum(NC)             AS TubesNC         
FROM Commande c INNER JOIN (SELECT NumCommande, BoutsTraites, Rebuts, 0 AS Supervise, 0 AS nc
                            FROM OperationCasing 
                            WHERE NomPoste In ('Rec.Agt.Csg') 
                              AND Poste = 'c,Bertaux' 
                              AND DateSaisie Between #1/1/2006# And #2/1/2006# 
                           UNION ALL
                            SELECT NumCommande, 0, 0, Supervise, nc 
                            FROM SupervisionCasing 
                            WHERE NomPoste In ('SupQuaCsg') 
                              AND Poste = 'c,Bertaux' 
                              AND DateSaisie Between #1/1/2006# And #2/1/2006#) AS a 
                 ON a.NumCommande = c.NumCommande
GROUP BY c.NumCommande, c.diametre, c.epaisseur, c.TypeDeFiletage, c.Grade, c.NomClient

Au point de vue performance tu peux essayer en mettant des SUM et des GROUP BY dans les requêtes de l'UNION

[pmboutteau]

Merci çà marche nickel. Je savais qu'on pouvais faire ce genre de manip avec le 0. C'est assez intéressant.