Discussion :

[madevilts]

Bonjour à tous !

Je suis en train de devenir fou... je ne comprends pas ce qu'il se passe dans mon code, c'est très bizarre. Je suis sur une 10.2g

Voici le dit code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
create or replace
FUNCTION get_comid(cp VARCHAR2,ville VARCHAR2,pays VARCHAR2)
RETURN number
AS result number;
BEGIN
select champ_a into result from matable...
if result is not null then
  result := 10;
else
  result := 20;
end if;
RETURN result;
END;

Jusque la tout va bien. Ma requete me retourne soit une ligne, soit rien.
Et niveau résultat, j'ai soit 10 soit null... alors que je devrais logiquement avoir 10 ou 20 et dans aucun cas je devrais avoir null.

Le problème n'est pas vraiment de retourner 20 mais dans le cas ou je n'ai pas de résultat a la première requete, il faut que j'en fasse une autre mais j'ai remarqué que je ne passais jamais dans le "else"

Je patoge... Merci de votre aide !

[mnitu]

...
Jusque la tout va bien. Ma requete me retourne soit une ligne, soit rien.
Et niveau résultat, j'ai soit 10 soit null... alors que je devrais logiquement avoir 10 ou 20 et dans aucun cas je devrais avoir null.
...

Quand ta requête ne retourne rien alors il y l’exception NO_DATA_FOUND qui est levée.

[madevilts]

J'ai bien vu cela mais je n'arrive pas à la mettre en place.

En fait, ça sort a cause de l'exception et du coup le if n'est pas exécuté c'est ça ?

[mnitu]

...En fait, ça sort a cause de l'exception et du coup le if n'est pas exécuté c'est ça ?

Voilà.

[rvfranck]

Tu dois traiter les erreurs (dans le bloc EXCEPTION) pour que ton programme sache quoi faire en cas d'erreur, sinon ton programme est interrompu.
regardes ici ou là.

[PonGe]

Bonjour

Je débute en PL/SQL donc ne m'en veuillez pas si je dis nimp mais si j'ai bien compris ton problème il te suffit d'utiliser les exceptions comme le dis rvfranck de la façon suivante:

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
 
CREATE OR REPLACE
FUNCTION get_comid(cp VARCHAR2,ville VARCHAR2,pays VARCHAR2)
RETURN number
AS result number;
BEGIN
SELECT champ_a INTO result FROM matable...
IF result IS NOT NULL then
  result := 10;
RETURN result;
end IF;
EXCEPTION
When NO_DATA_FOUND  then 
  result :=20;
RETURN result;
END;

J'espere que ca pourra t'aider..

[Garuda]

Mieux avec un seul RETURN :

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
 
CREATE OR REPLACE FUNCTION get_comid (cp      IN VARCHAR2,
                                      ville    IN VARCHAR2,
                                      pays     IN VARCHAR2)
   RETURN number
AS
   result   number;
BEGIN
   .....
   BEGIN
      SELECT   10 INTO result FROM matable...;
   EXCEPTION
      WHEN NO_DATA_FOUND
      THEN
         result := 20;
   END;
   RETURN result;
END;