Discussion :

[sasuma]

Salut à tous,

j'ai fait deux requetes qui me cherchent le nombre de dossiers par groupe et ce dans deux tables.Pour les grouper dans un seul tableau j ai fait une UNION. Le pb est que je veux avoir le total de ts les dossiers. J'ai essayé de faire un select avec un SUM (cf requete ligne 1 ) mais ca marche pa ???

ou est le pb SVP??

merci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
select T1.[Closed Group:],sum([Incident #]) AS Dossier_NBR
from 
(SELECT T1.[Closed Group:],COUNT([Incident #]) AS Dossier_NBR
FROM _SMDBA_.Incident T1 
WHERE T1.[Open Date] >= convert(DATETIME,Convert(varchar(10), getDate(),103),103) GROUP BY T1.[Closed Group:])
 
 
UNION
 
(SELECT T1.[Closed Group:],COUNT([Work Order #]) Dossier_NBR
FROM _SMDBA_.[Work Orders] T1
WHERE T1.[Open Date] >= convert(DATETIME,Convert(varchar(10), getDate(),103),103)GROUP BY T1.[Closed Group:])

[SQLpro]

Code :

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

1
2
3
4
5
6
7
8
9
10
select T1.[Closed Group:] AS Closed_Group, sum([Incident #]) AS Dossier_NBR
from   (SELECT T1.[Closed Group:], COUNT([Incident #]) AS Dossier_NBR
        FROM   _SMDBA_.Incident T1
        WHERE  T1.[Open Date] >= CURRENT_TIMESTAMP
        GROUP BY T1.[Closed Group:]) AS T
UNION ALL
SELECT T1.[Closed Group:], COUNT([Work Order #])
FROM   _SMDBA_.[Work Orders] T1
WHERE  T1.[Open Date] >= CURRENT_TIMESTAMP
GROUP  BY T1.[Closed Group:])

Cea dit, que d'horreur dans votre base et votre requête :
1) les noms des objets ne doivent comporter ni blanc ni signe de ponctuation (: # ...). SQL exige que les seuls caractères de composition des noms des tables, colonnes, etc soient des chiffres des lettres sans accents et le blanc souligne. Ne pas respecter cette règle ne peut que vous nuire un jour ou l'autre. A Lire : http://sqlpro.developpez.com/cours/sqlaz/erreurs/#L1
2) si votre colonne "Open Date" est un type datetime, il est strictement inutile et idiot de transtyper la date obtenue par la commande GETDATE qui fournit une date, cela fait un double transtypage !
3) Préférez utiliser la fonction CURRENT_TIMESTAMP à GetDate(). CURRENT_TIMESTAMP est la norme SQL et GetDate est spécifique à SQL Server.
4) lorsque vous faites des sous requêtes en table dérivée, il faut un alias à la table ainsi décrite.
5) lorsque vous faites des requêtes ensembliste il ne faut pas de parenthèses
6) dans les opérations ensemblistes (UNION, INTERSECT, EXCEPT) lorsqu'il n'y a aucun doublons possible ou souhaité, il est important de faire un UNION ALL, sinon, le dédoublonnement pourra vous donner des résultats faux.

Bref, je vous conseille fortement d'apprendre le SQL. C'est un vrai langage. Mon livre, comme mon site peuvent vous y aider !

A +

[sasuma]

Bonjour,

merci mais je me suis trompée c'est plutot du TransactSQL et la BD est en SQLServer j'ai un code VBA qui execute la requete et donne le resultat dans une autre table...

avec ces données je faisait marché d'autres requetes et ma base est bonne..

merci

[sasuma]

Bonjour,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
SELECT [Closed Group:] as Groupe,(sum([Incident #])) AS Dossier
FROM 
(SELECT [Closed GROUP:], COUNT([Incident #]) AS Dossier_NBR
        FROM   _SMDBA_.[Incident] 
        WHERE  [Open Date] >= CURRENT_TIMESTAMP
        GROUP BY [Closed Group:]
UNION ALL
SELECT [Closed GROUP:], COUNT([Work Order #]) AS Order_NBR
FROM   _SMDBA_.[Work Orders] 
WHERE  [Open Date] >= CURRENT_TIMESTAMP
GROUP BY [Closed Group:])

ca me me donne une erreur :

Msg 102, Level 15, State 1, Line 12
Incorrect syntax near ')'.

ca n'accepte pas la ')' de la ligne dernière????

Merci

[Chtulus]

Bonjour,

Mettez un ALIAS

[sasuma]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
SELECT T1.[Closed Group:] as Groupe, SUM(T1.[Incident #]) AS Dossier
FROM 
(SELECT T1.[Closed Group:], COUNT(T1.[Incident #]) AS Dossier_NBR
        FROM   _SMDBA_.[Incident] T1
        WHERE  T1.[Open Date] = convert(DATETIME,Convert(varchar(10), getDate(),103),103)
        GROUP BY T1.[Closed Group:]
UNION ALL
SELECT T2.[Closed Group:], COUNT(T2.[Work Order #]) AS Order_NBR
     FROM   _SMDBA_.[Work Orders] T2
     WHERE  T2.[Open Date] = convert(DATETIME,Convert(varchar(10), getDate(),103),103)
     GROUP BY T2.[Closed Group:])

et c'est le meme message d'erreur.
Pensez vous qu'il faut utiliser la clause INTO pour mettre le resultat de UNION ALL dans une table résultat par exp??? sauf que je n'ai pas le droit d'en faire j'ai juste le droit de lecture((

merci

[al1_24]

Une table dérivée (sous-requête utilisée dans la clause FROM) doit avoir un alias.
Par ailleurs, les colonnes de T1 de sont pas visibles de la requête SELECT principale.

[sasuma]

Une table dérivée (sous-requête utilisée dans la clause FROM) doit avoir un alias.
Par ailleurs, les colonnes de T1 de sont pas visibles de la requête SELECT principale.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
SELECT [Closed Group:] as Groupe, SUM([Incident #]) AS Dossier
FROM 
(SELECT T1.[Closed Group:], COUNT(T1.[Incident #]) AS Dossier_NBR
FROM _SMDBA_.[Incident] T1
WHERE T1.[Open Date] = convert(DATETIME,Convert(varchar(10), getDate(),103),103)
GROUP BY T1.[Closed Group:]
UNION ALL
SELECT T2.[Closed Group:], COUNT(T2.[Work Order #]) AS Order_NBR
FROM _SMDBA_.[Work Orders] T2
WHERE T2.[Open Date] = convert(DATETIME,Convert(varchar(10), getDate(),103),103)
GROUP BY T2.[Closed Group:])

et c'est tj pareil((

[Chtulus]

Non,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
SELECT [Closed GROUP:] AS Groupe,(sum([Incident #])) AS Dossier
FROM 
(SELECT [Closed GROUP:], COUNT([Incident #]) AS Dossier_NBR
        FROM   _SMDBA_.[Incident] 
        WHERE  [Open Date] >= CURRENT_TIMESTAMP
        GROUP BY [Closed GROUP:]
UNION ALL
SELECT [Closed GROUP:], COUNT([Work ORDER #]) AS Order_NBR
FROM   _SMDBA_.[Work Orders] 
WHERE  [Open Date] >= CURRENT_TIMESTAMP
GROUP BY [Closed GROUP:]) T1

[sasuma]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
SELECT [Closed GROUP:] AS Groupe,(sum([Incident #])) AS Dossier_NBR
FROM 
(SELECT [Closed GROUP:], COUNT([Incident #]) AS Dossier_NBR
        FROM   _SMDBA_.[Incident] 
        WHERE  [Open Date] >= CURRENT_TIMESTAMP
        GROUP BY [Closed GROUP:]
UNION ALL
SELECT [Closed GROUP:], COUNT([Work ORDER #]) AS Order_NBR
FROM   _SMDBA_.[Work Orders] 
WHERE  [Open Date] >= CURRENT_TIMESTAMP
GROUP BY [Closed GROUP:]) T1

là ce ne me reconnait plus ma colonne [Incident #] elle existe bien dans la base
de plus le T1 dans la dernière ligne c'est pour renommer [Incident #]?? je l'ai déjà recnommé à la ligne 1????
comprend plus(((

[Chtulus]

Effectivement, comme tu as déjà renommé dans la table dérivée T1, tu dois reprendre cet ALIAS.

Donc

Code :

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

SUM(Dossier_NBR)

[sasuma]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
SELECT sum(Dossier_NBR) 
FROM 
(SELECT [Closed Group:], COUNT([Incident #]) AS Dossier_NBR
        FROM   _SMDBA_.[Incident] 
        WHERE  [Open Date] >=  convert(DATETIME,Convert(varchar(10), getDate(),103),103)
        GROUP BY [Closed GROUP:]
UNION ALL
SELECT [Closed Group:] AS Groupe, COUNT([Work ORDER #]) AS Order_NBR
FROM   _SMDBA_.[Work Orders] 
WHERE  [Open Date] >=  convert(DATETIME,Convert(varchar(10), getDate(),103),103)
GROUP BY [Closed Group:]) as T1

et ça marcheeee))
merci et bon weekend