Discussion :

[dubitoph]

Bonjour,

Dans ma requête, j'ai certainement une erreur de logique que je ne parviens pas à détecter.

Dans une table GLIT_LITS qui liste tous les lits de la clinique, par unité de soins (NUMUS), je cherche à comptabiliser le nombre de lits qui ne sont pas fictifs (FICTIF = 0). Seulement, pour unité particulière (NUMUS = 26796), dans le comptage des lits, en plus des lits fictifs, je ne dois pas non plus comptabiliser les lits bébé (TYPE = 'BB').

Voici la requête que j'ai écrite :

Code :

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

1
2
3
4
5
6
SELECT NUMUS, count(*) as NBLIT1 FROM 
GLIT_LITS 
where ((NUMUS <> 26796) or (NUMUS = 26796 and TYPE <> 'BB'))  AND 
          FICTIF = 0 
group by NUMUS 
order by NUMUS;

Cette requête ne me donne pas le nombre de lits corrects par unité et de plus, l'unité 26796 n'est pas reprise dans le résultat.

Quelqu'un aurait une idée?

Merci d'avance!

[dubitoph]

J'ai modifié ma requête comme ceci :

Code :

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

1
2
3
4
5
6
SELECT NUMUS, count(*) as NBLIT1 FROM 
GLIT_LITS 
where ((NUMUS <> 26796) or (NUMUS = 26796 and (TYPE is null or TYPE <> 'BB')))  AND 
          FICTIF = 0 
group by NUMUS 
order by NUMUS;

Cela a résolu mon problème.

[tatayo]

Bonjour,
Je dirais que tu peux simplifier ta condition:

Code sql :

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

1
2
3
4
5
6
7
 
SELECT NUMUS, count(*) as NBLIT1 FROM 
GLIT_LITS 
where ((NUMUS <> 26796) or (TYPE is null or TYPE <> 'BB'))  AND 
          FICTIF = 0 
group by NUMUS 
order by NUMUS

En effet, comme tu ne testes que la colonne NUMUS dans la première partie du OR, il est inutile de faire le test inverse dans la deuxième partie.
Tatayo.

[dubitoph]

De fait, merci!

[Waldar]

En effet, comme tu ne testes que la colonne NUMUS dans la première partie du OR, il est inutile de faire le test inverse dans la deuxième partie.

N'oublions pas nos amis les nulls.

Prenons ce cas où j'ai combiné NUMUS sur l'ensemble {26796, 10, null}, avec TYPE sur l'ensemble {AA, BB, null} et FICTIF sur l'ensemble {0, 1} :

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
with glit_lits (NUMUS, TYPE, FICTIF) as
(
select 26796, 'AA', 0 from dual union all
select    10, 'AA', 0 from dual union all
select  null, 'AA', 0 from dual union all
select 26796, 'BB', 0 from dual union all
select    10, 'BB', 0 from dual union all
select  null, 'BB', 0 from dual union all
select 26796, null, 0 from dual union all
select    10, null, 0 from dual union all
select  null, null, 0 from dual union all
select 26796, 'AA', 1 from dual union all
select    10, 'AA', 1 from dual union all
select  null, 'AA', 1 from dual union all
select 26796, 'BB', 1 from dual union all
select    10, 'BB', 1 from dual union all
select  null, 'BB', 1 from dual union all
select 26796, null, 1 from dual union all
select    10, null, 1 from dual union all
select  null, null, 1 from dual
)

La requête originale de dubitoph :

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
  select numus
       , count(*) as nblit1
    from glit_lits 
   where (numus <> 26796
      or (numus  = 26796 
     and (type   is null
      or  type   <> 'BB')))
     and fictif  = 0 
group by numus
order by numus;
 
     NUMUS     NBLIT1
---------- ----------
        10          3
     26796          2

La requête de tatayo :

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
  select numus
       , count(*) as nblit1
    from glit_lits 
   where (numus <> 26796
      or type   is null
      or type   <> 'BB')
     and fictif  = 0 
group by numus
order by numus;
 
 
     NUMUS     NBLIT1
---------- ----------
        10          3
     26796          2
                    2

Attention à la simplification qui n'est pas forcément ISO.