Discussion :

[pinocchio]

Bonjour,
Lorsque je crée la procédure ci-dessous :

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
CREATE OR REPLACE  PROCEDURE "VERIF_ETAT"  (
  NewEtat in Varchar2
) is
  ErrCode number;
begin
     if(NewEtat='XX' or NewEtat='YY' or NewEtat is null or NewEtat='') then
        ErrCode :=0;
		    Raise_application_error(-20000,'OK');
     else
        ErrCode :=1;
		    Raise_application_error(-20000,'Etat non correct');
     end if;
exception
when others then
        ErrCode :=1;
		Raise_application_error(-20000,'non modifié');
end;

Quelque soit NewEtat, je me retrouve à passer dans le exception when others et avec une erreur sur le Raise_application_error.
Pour faire des tests, j'utilise DBMS_OUTPUT.PUT_LINE et là aucun souci, je passe bien dans le if ou le else selon la valeur de NewEtat.
Auriez-vous une explication à cette erreur, s'il vous plait?
Cordialement
Pinocchio

[Rei Ichido]

Il est normal quand tu lève une exception avec raise_application_error de passer ensuite dans le "when other", qui intercepte toutes les exceptions ...

[pinocchio]

Merci, je n'y avais pas pensé.
Est-il possible de ne pas aller plus loin dans la procédure à partir du moment où l'on a rencontré un raise_application_error?

[Rams7s]

La vraie question, c'est est-ce que le when other est necessaire?
On va soit dans le if, soit dans le else. Je ne vois pas de raison d'aller ailleurs. [Mais je me trompe souvent ]

Par contre, le test NewEtat is null OR NewEtat='' c'est blanc bonnet et
bonnet blanc.

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
TMP@minilhc >declare
  2     a varchar2(20);
  3     procedure test_nullite(p_vc varchar2) is
  4     begin
  5        if(p_vc is null) then
  6           dbms_output.put_line('parameter is null');
  7        else
  8           dbms_output.put_line('parameter is not null ['||p_vc||']');
  9        end if;
 10     end;
 11  begin
 12     test_nullite(a);
 13
 14     a:='a';
 15     test_nullite(a);
 16
 17     a:='';
 18     test_nullite(a);
 19
 20     a:='a';
 21     test_nullite(a);
 22
 23     a:=null;
 24     test_nullite(a);
 25
 26
 27  end;
 28  /
parameter is null
parameter is not null [a]
parameter is null
parameter is not null [a]
parameter is null
 
PL/SQL procedure successfully completed.
 
Elapsed: 00:00:00.03

[pinocchio]

Merci pour le null ou '' , j'ai toujours traité cela de manière différente et dans ce cas là à tort.

Pour le if et le else, j'étais parti du principe, et encore une fois à tort, que si NewEtat était autre chose qu'une chaine de caractère, ce serait ni dans le if ni dans le else et donc que je le récupèrerai dans le when others.

Merci pour toutes ces précisions.


Ma question sur la sortie prématurée reste valable car je vais avoir d'autres procédures avec plus de test et par sécurité, je mettrai peut-être le when others

[al1_24]

Merci pour le null ou '' , j'ai toujours traité cela de manière différente et dans ce cas là à tort.

C'est toutefois une bonne habitude à conserver car tous les SGBD ne considèrent pas comme Oracle que '' IS NULL.