Case dans une clause where avec plusiuers valeurs à évaluer

**Oncle_Pete** · 20/06/2018, 15h26

Bonjour le forum,

Je me bat avec une syntaxe que je n'arrive pas à trouver (peut etre cela n'est il pas possible d'ailleurs)

Je suis dans ma clause Where et je souhaite évaluer tout un ensemble de machines en fonction d'une ligne

Code :

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

1
2
3
4
5
6
7
8
9
 
Where ...
AND CASE
	WHEN  :MaLigne =  'A' THEN uniteProd IN ('A1', 'A2', 'A5')
	WHEN  :MaLigne =  'B' THEN uniteProd IN ('B2', 'B10')
	WHEN  :MaLigne =  'C' THEN uniteProd IN ('C1', 'C2', 'C5')
	WHEN  :MaLigne =  'G' THEN uniteProd IN ('G4', 'G45')
	WHEN  :MaLigne =  'Toutes' THEN  uniteProd IN ('A1', 'A2', 'A5', 'B2', 'B10', 'C1', 'C2', 'C5', 'G4', 'G45')
    END

J'ai l'erreur Missing keyword ou bien Missing right parenthese selon mes essais.

Merci pour vos conseils

Oncle pete

**skuatamad** · 20/06/2018, 16h19

Il faut obligatoirement comparé le CASE après le END et non dans le THEN, ce qui oblige à écrire du code moins intuitif :

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
SQL> var Maligne varchar2(10);
SQL> exec :Maligne := 'A';
 
Procedure PL/SQL terminee avec succes.
 
SQL> with t as (
select 'A1' as uniteProd from dual union all
select 'A2' as uniteProd from dual union all
select 'A5' as uniteProd from dual union all
select 'B2' as uniteProd from dual union all
select 'B10' as uniteProd from dual union all
select 'C1' as uniteProd from dual union all
select 'C2' as uniteProd from dual union all
select 'C5' as uniteProd from dual union all
select 'G4' as uniteProd from dual union all
select 'G45' as uniteProd from dual
)
select *
  from t
 where CASE WHEN :MaLigne = 'A' and uniteProd IN ('A1', 'A2', 'A5')
            then 1
            WHEN :MaLigne = 'B' and uniteProd IN ('B2', 'B10')
            then 1
            WHEN :MaLigne = 'C' and uniteProd IN ('C1', 'C2', 'C5')
            then 1
            WHEN :MaLigne = 'G' and uniteProd IN ('G4', 'G45')
            then 1
            WHEN :MaLigne = 'Toutes' and  uniteProd IN ('A1', 'A2', 'A5', 'B2', 'B10', 'C1', 'C2', 'C5', 'G4', 'G45')
            then 1
        END = 1;
 
UNI
---
A1
A2
A5
 
SQL>  exec :Maligne := 'G';
 
Procedure PL/SQL terminee avec succes.
 
SQL> /
 
UNI
---
G4
G45
 
SQL> exec :Maligne := 'Toutes';
 
Procedure PL/SQL terminee avec succes.
 
SQL> /
 
UNI
---
A1
A2
A5
B2
B10
C1
C2
C5
G4
G45
 
10 ligne(s) selectionnee(s).
 
SQL>

Ou comme ça, peut être un peu moins bizarre :

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
SQL> with t as (
select 'A1' as uniteProd from dual union all
select 'A2' as uniteProd from dual union all
select 'A5' as uniteProd from dual union all
select 'B2' as uniteProd from dual union all
select 'B10' as uniteProd from dual union all
select 'C1' as uniteProd from dual union all
select 'C2' as uniteProd from dual union all
select 'C5' as uniteProd from dual union all
select 'G4' as uniteProd from dual union all
select 'G45' as uniteProd from dual
)
select *
  from t
 where CASE WHEN :MaLigne = 'A' and uniteProd IN ('A1', 'A2', 'A5')
            then 'A'
            WHEN :MaLigne = 'B' and uniteProd IN ('B2', 'B10')
            then 'B'
            WHEN :MaLigne = 'C' and uniteProd IN ('C1', 'C2', 'C5')
            then 'C'
            WHEN :MaLigne = 'G' and uniteProd IN ('G4', 'G45')
            then 'G'
            WHEN :MaLigne = 'Toutes' and  uniteProd IN ('A1', 'A2', 'A5', 'B2', 'B10', 'C1', 'C2', 'C5', 'G4', 'G45')
            then 'Toutes'
        END = :MaLigne;
 
UNI
---
A1
A2
A5
B2
B10
C1
C2
C5
G4
G45
 
10 ligne(s) selectionnee(s).
 
SQL> exec :Maligne := 'G';
 
Procedure PL/SQL terminee avec succes.
 
SQL> /
 
UNI
---
G4
G45
 
SQL>

**al1_24** · 20/06/2018, 18h40

Quand on commence à mettre un CASE dans la clause WHERE, c'est qu'il y a une question de logique qui a été mal posée...
Pourquoi ne pas écrire simplement cela ?

Code :

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

1
2
3
4
5
6
where   (:MaLigne = 'A' and uniteProd IN ('A1', 'A2', 'A5'))
    or  (:MaLigne = 'B' and uniteProd IN ('B2', 'B10'))
    or  (:MaLigne = 'C' and uniteProd IN ('C1', 'C2', 'C5'))
    or  (:MaLigne = 'G' and uniteProd IN ('G4', 'G45'))
    or  (:MaLigne = 'Toutes' and  uniteProd IN ('A1', 'A2', 'A5', 'B2', 'B10', 'C1', 'C2', 'C5', 'G4', 'G45'))
;

**Oncle_Pete** · 20/06/2018, 19h32

Merci al1_24 et skuatamad pour vos réponses

Bien sur j'avais imaginé utiliser le OR, mais je me l'interdit.

Un OR mal placé ou sans les parenthèses bien placé c'est susceptible de mettre à la ramasse un serveur.
La personne qui assure la gestion de la DB nous a tenu des discours assez dramatique sur les conséquences possibles.

J'ai pas cherché à le contredire. J'ai pas la culture pour ca.

Demain j'analyse la réponse de al1_24 qui me semble compliquée (mais pourquoi pas ?).
Merci encore

OP

**Waldar** · 21/06/2018, 11h03

Envoyé par Oncle_Pete

Un OR mal placé ou sans les parenthèses bien placé c'est susceptible de mettre à la ramasse un serveur.

Tout comme une jointure, et pourtant je suis sûr que vous les utilisez.
Votre interlocuteur prône un nivellement par le bas - c'est potentiellement dangereux alors on ne fait pas - au lieu d'un nivellement pas le haut - c'est potentiellement dangereux, voici ce qu'il faut comprendre pour éviter les écueils.

Envoyé par Oncle_Pete

Demain j'analyse la réponse de al1_24 qui me semble compliquée (mais pourquoi pas ?).

C'est pourtant la plus logique et la plus lisible.

**Oncle_Pete** · 21/06/2018, 11h50

Bonjour Waldar et merci pour ton support

En parlant d'analyse, je voulais en effet parler de la solution de skuatamad. Je me suis emmelé les pinceaux (désolé).

Je suis d'accord avec vous, une mauvaise liaison entre 2 tables est aussi capable de gros dégâts.
La différence fondamentale que je vois est que les jointures sont indispensables pour accéder aux données de plusieurs tables.
Pour le OR il doit etre possible de ne pas l'utiliser donc de faire autrement (de faire mieux) pour obtenir ce qu'on cherche
Je suis dans cette option.

Merci encore

Oncle pete

**skuatamad** · 21/06/2018, 12h05

Ce que vous pouvez aussi faire c'est générer la requête qui va bien en fonction du paramètre :Maligne, plutôt que gérer tous les cas dans une seule requête.

Donc au final, si :Maligne = 'A', seul le filtre uniteProd IN ('A1', 'A2', 'A5') sera généré, la requête sera plus optimale dans le cas où certains critères sont très filtrant et d'autres très peu (index vs full scan)

**Oncle_Pete** · 21/06/2018, 12h27

Bonjour skuatamad,

Pas bête ! en effet. Pas exploré cette piste.
Super j'ai de quoi me remettre dans les tests.

Merci tous pour vos avis / aides

Oncle Pete