Discussion :

[Sylv_62]

Bonjour,

J'ai encore un souci avec une requête qui va servir à la génération d'un rapport (MS Report builder).

Les parties SELECT et FROM sont OK, j'ai les bons noms de tables et les colonnes correspondant aux informations que je dois récupérer.

Le point qui m'ennuie est une des clauses dans le WHERE.
J'ai un des paramètres en entrée, que j'ai appelé @TypeTravail, qui pourra prendre une (seule et unique) valeur parmi ces 3 : Routage, Autres, Tous.
Dans le cas où @TypeTravail est Routage, je dois vérifier la condition B1.ResourceType IN ('RO', 'RP', 'RU');
Si @TypeTravail est Autres, cette fois je dois vérifier B1.ResourceType NOT IN ('RO', 'RP', 'RU');
Et si @TypeTravail est Tous, je n'ai pas besoin de filtrer B1.ResourceType, donc je pensais juste vérifier une clause B1.ResourceType <> '' pour contrôler que la colonne ne soit pas vide (l'ERP ne permet pas d'avoir une valeur vide pour cette colonne).

Mon code actuel est rédigé de cette façon :

Code :

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

1
2
3
4
 
AND (CASE WHEN @TypeTravail = 'Routage' THEN B1. ResourceType IN ('RO', 'RP', 'RU')
		 ELSE WHEN  @TypeTravail = 'Autres' THEN B1. ResourceType NOT IN ('RO', 'RP', 'RU')
		 ELSE B1.ResourceType <> '' END)

Mais SQL Server Management Studio me reproche une syntaxe incorrecte : Incorrect syntax near the keyword 'IN'

J'ai fait un 2eme essai pour retirer la partie "IN" :

Code :

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

1
2
3
AND (CASE WHEN @TypeTravail = 'Routage' THEN (B1. ResourceType = 'RO'OR B1.ResourceType = 'RP' OR B1.ResourceType =  'RU')
		 ELSE WHEN  @TypeTravail = 'Autres' THEN (B1.ResourceType <> 'RO' AND B1.ResourceType <> 'RP'AND B1.ResourceType <>  'RU')
		 ELSE B1.ResourceType <> '' END)

Réponse de SSMS : Incorrect syntax near '='

Dans les deux cas, la ligne concernée est celle où mon CASE commence.

Quelle méthode pourrais-je utiliser pour pouvoir vérifier ces conditions?
Merci d'avance pour vos lumières

[tatayo]

Bonjour,
Tu peux remplacer le CASE par une simple "combinaison):

Code SQL :

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

1
2
3
4
5
 
...
AND ((@TypeTravail = 'Routage' AND B1. ResourceType IN ('RO', 'RP', 'RU'))
OR (@TypeTravail = 'Autres' AND B1. ResourceType NOT IN ('RO', 'RP', 'RU'))
OR (@TypeTravail not in ('Routage','Autres') and B1.ResourceType <> ''))

Tatayo.

[Sylv_62]

Bonjour Tatayo,

Merci, cette fois la requête passe

Je passe la question en résolu, je n'ai plus qu'à vérifier que les résultats sont bien ceux qu'il me faut.

[SQLpro]

En fait le CASE est plus efficace si c'est un CASE valué et non un CASE généralisé :

Code :

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

1
2
3
4
5
6
7
8
SELECT ...
FROM   ...
WHERE  CASE @TypeTravail 
          WHEN 'Tous' THEN 1
          WHEN 'Routage' AND B1.ResourceType IN ('RO', 'RP', 'RU') THEN 1
          WHEN 'Autres' AND B1.ResourceType NOT IN ('RO', 'RP', 'RU') THEN 1
          ELSE 0
       END = 1

À lire : https://sqlpro.developpez.com/cours/sqlaz/select/#L7


A +

[Michel.Priori]

Mon code actuel est rédigé de cette façon :

Code :

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

1
2
3
AND (CASE WHEN @TypeTravail = 'Routage' THEN B1. ResourceType IN ('RO', 'RP', 'RU')
		 ELSE WHEN  @TypeTravail = 'Autres' THEN B1. ResourceType NOT IN ('RO', 'RP', 'RU')
		 ELSE B1.ResourceType <> '' END)

L'expression CASE n'admet seulement une expression (valeur calculée ou pas) après THEN
https://learn.microsoft.com/fr-fr/sq...ult_expression

Ici l'opérateur IN ou NOT IN est une conditionnelle, pas une expression ; d'où le message d'erreur

La réponse de SQLpro, qui tient compte du fait que seules 3 valeurs sont admises (Routage, Autres, Tous), est élégante

[Sylv_62]

Code :

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

1
2
3
4
5
6
7
8
SELECT ...
FROM   ...
WHERE  CASE @TypeTravail 
          WHEN 'Tous' THEN 1
          WHEN 'Routage' AND B1.ResourceType IN ('RO', 'RP', 'RU') THEN 1
          WHEN 'Autres' AND B1.ResourceType NOT IN ('RO', 'RP', 'RU') THEN 1
          ELSE 0
       END = 1

A +

Merci pour cette lecture

Etrangement, votre forme du case me renvoyait une erreur (je me demande si ça n'est pas simplement dû à SQL Server qui "arrange" un peu les choses à sa sauce).
J'ai adapté en complétant ma requête de cette façon :

Code :

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

1
2
3
4
5
6
 AND CASE 
          WHEN @TypeTravail ='Tous' THEN 1
          WHEN @TypeTravail ='Routage' AND B1.ResourceType IN ('RO', 'RP', 'RU') THEN 1
          WHEN @TypeTravail ='Autres' AND B1.ResourceType NOT IN ('RO', 'RP', 'RU') THEN 1
          ELSE 0
       END = 1

Avec la solution donnée par Tatayo, j'ai mis un peu de temps avant de m'apercevoir que les résultats contenaient des lignes qui auraient dû être exclues.

Avec ce case, il semble maintenant que les résultats soient plus en phase avec l'attendu.
Le nombre de résultats renvoyés (pour les mêmes combinaisons de paramètres) a également baissé, ce qui m'a l'air d'être un bon signe.

[Séb.]

Perso je l'aurais écrite comme ça :

AND CASE @TypeTravail
    WHEN 'Tous' THEN TRUE
    WHEN 'Routage' THEN B1.ResourceType IN ('RO', 'RP', 'RU')
    WHEN 'Autres' THEN B1.ResourceType NOT IN ('RO', 'RP', 'RU')
    ELSE FALSE
END

(j'aime bien remettre des pièces)

[Michel.Priori]

(j'aime bien remettre des pièces)

Ok pour le fun
Mais la proposition ne fonctionne pas ; exactement pour la même raison évoquée précédement

[escartefigue]

La plupart des SGBD émettent un message d'erreur si on utilise la syntaxe CASE ma_colonne avec un deuxième argument.

Et bizarrement, non seulement MySQL accepte cette syntaxe, mais surtout, renvoie un résultat faux :

Exemple :

Code SQL :

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
create table T1
      (  T1ID   integer primary key
       , T1C1   char(4) not null
       , T1C2   char(2) not null
      )
;
insert into T1
       (T1ID, T1C1, T1C2)
values (1, 'ABCD', 'AA')
     , (2, 'ABCD', 'DD')
     , (3, 'ABCD', 'AA')
     , (4, 'EFGH', 'XX')
     , (5, 'EFGH', 'DD')
     , (6, 'EFGH', 'DD')
     , (7, 'ZZZZ', 'YY')
;
select T1ID
     , case when T1C1 = 'ABCD' and T1C2='DD' then 'cas 1'
            when T1C1 = 'EFGH' and T1C2='XX' then 'cas 2'
            when T1C2 = 'YY'                 then 'cas 3'
            else 'autre'
       end as CASE01
     , case T1C1
            when 'ABCD' and T1C2='DD' then 'cas 1'
            when 'EFGH' and T1C2='XX' then 'cas 2'
            when T1C2 = 'YY'          then 'cas 3'
            else 'autre' 
       end as CASE02
from T1

Résultat avec MySQL (Postgre, SQL server et DB2 plantent, à raison) :

[Séb.]

case T1C1
    when 'ABCD' and T1C2='DD' then 'cas 1'
    when 'EFGH' and T1C2='XX' then 'cas 2'
    when T1C2 = 'YY'          then 'cas 3'
    else 'autre' 
end as CASE02

Cette construction n'a pas de sens. Pour le 1er WHEN elle revient à faire T1C1 = ('ABCD' and T1C2='DD')

MySQL accepte 2 constructions de CASE bien distinctes => https://dev.mysql.com/doc/refman/8.4/en/case.html

https://modern-sql.com/feature/case

[escartefigue]

Comme indiqué précédemment, le problème c'est que MySQL renvoie n'importe quoi ! Il devrait réagir comme les autres SGBD en émettant un message d'erreur bloquante.
MySQL est le seul SGBD qui réponde à la question "Quel âge avait Rimbaud ?"