Discussion :

[randriano]

Bonjour,

Faudrait que je m'y fasse au fait que sous Oracle, un "select field into" avec aucune donnée trouvée génère une exception NO_DATA_FOUND contrairement à SQL Server où on aurait juste un retour NULL !!
Comment éviter cette exception alors ?
Je me propose de faire un "select count(field)" et ne faire le "select field into" que si le comptage supérieur à 0:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
select count(SC.Name) into v_CountClassName from TClasses SC, TEntity OE,   TCategories OC 
where OE.IDCa = OC.ID and 
OC.IDClass = SC.ID and 
OE.ID = v_ID;
 
 
if v_CountClassName = 0 then
   v_ClassName := null;
else
   select SC.Name into v_ClassName  from TClasses SC, TEntity OE,   TCategories OC 
   where OE.IDCa = OC.ID and 
   OC.IDClass = SC.ID and 
   OE.ID = v_ID; 
end if;

Est-ce que cette technique est bonne ou y a-t-il une autre technique ?

[miloux32]

Si tu es dans une procédure , pourquoi ne pas pas faire un bloc d'exeption WHEN_NO_DATA_FOUND ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
DECLARE
 
BEGIN
 
 
SELECT SC.Name INTO v_ClassName  FROM TClasses SC, TEntity OE,   TCategories OC 
   WHERE OE.IDCa = OC.ID AND 
   OC.IDClass = SC.ID AND 
   OE.ID = v_ID; 
EXCEPTION
   When NO_DATA_FOUND then
         -- Ne rien faire
END ;

[randriano]

En effet, je l'utilise dans une procédure mais je ne veux pas considérer le fait qu'il n'y a pas de donnée trouvée comme une erreur, il y a d'autres instructions qui suivent le SELECT !! Sous Access et Sql Server, c'est pas un problème donc je veux faire pareil
Il semble aussi qu'un select brut c-à-d pas dans une procédure génère aussi une exception quand il n'y a pas de donnée trouvée ??

[miloux32]

En effet, je l'utilise dans une procédure mais je ne veux pas considérer le fait qu'il n'y a pas de donnée trouvée comme une erreur, il y a d'autres instructions qui suivent le SELECT !! Sous Access et Sql Server, c'est pas un problème donc je veux faire pareil
Il semble aussi qu'un select brut c-à-d pas dans une procédure génère aussi une exception quand il n'y a pas de donnée trouvée ??

Ben c'est tout l'astuce du code que je t'ai donné !

Apres le select et avant le bloc "EXCEPTION" tu peux mettre le traitement que tu veux ( qui ne se déclenchera donc pas quand il y'a pas de données)

[randriano]

Les codes qui suivent le select doivent être après lui car ils vont utiliser les enregistrements resultants. C'est pour ça que je fais ce "select count" pour affecter une chaine vide au résultat si no data found

[miloux32]

Je redonne mon explication car a priori tu n'as pas compris ....

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
DECLARE
BEGIN
SELECT SC.Name INTO v_ClassName  FROM TClasses SC, TEntity OE,   TCategories OC 
   WHERE OE.IDCa = OC.ID AND 
   OC.IDClass = SC.ID AND 
   OE.ID = v_ID; 

ECRIRE ICI TON TRAITEMENT

EXCEPTION
   When NO_DATA_FOUND then
         -- TON TRAITEMENT DANS LE CAS OU PAS DE DONNEES
END ;

Le traitement sera donc le suivant :
Si ton "select ...into" se passe correctement , la partie du traitement que tu auras ecrit se fera.
Sinon => l'execution de ton traitement sautera il fera ce que tu as ecrit dans le "WHEN NO_DATA_FOUND"

Ca revient à ce que tu as fait mais en plus propre, sans la requete de count en plus et te permettant de bien gerer l'exception comme tu le veux .

[fatati]

Bonjour,

je pense qu'une instruction NVL peut te régler le problème
donc tu vas mettre :

Code :

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

1
2
3
 
 
select nvl(ton_champ,0) into var frop table

Bonne journée

[plaineR]

Un petit exemple pour illustrer ce que t'a expliqué miloux32 :

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
SQL> declare
  2    v_res number(1);
  3  begin
  4    dbms_output.put_line ('Avant le select');
  5    begin
  6      select 1 into v_res from dual where 1 = 1;
  7      dbms_output.put_line ('Apres le select');
  8    exception
  9       when no_data_found then
 10          dbms_output.put_line ('Aucune donnee');
 11    end;
 12    dbms_output.put_line ('Fin du traitement');
 13  end;
 14  /
Avant le select
Apres le select
Fin du traitement
 
Procédure PL/SQL terminée avec succès.
 
SQL> declare
  2    v_res number(1);
  3  begin
  4    dbms_output.put_line ('Avant le select');
  5    begin
  6      select 1 into v_res from dual where 1 = 0;
  7      dbms_output.put_line ('Apres le select');
  8    exception
  9       when no_data_found then
 10          dbms_output.put_line ('Aucune donnee');
 11    end;
 12    dbms_output.put_line ('Fin du traitement');
 13  end;
 14  /
Avant le select
Aucune donnee
Fin du traitement
 
Procédure PL/SQL terminée avec succès.

Après à toi de placer le code où tu souhaites en fonction de ce que tu veux faire.

NB : le count(*) est à proscrire pour des raisons de performance.

[miloux32]

Bonjour,

je pense qu'une instruction NVL peut te régler le problème
donc tu vas mettre :

Code :

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

1
2
3
 
 
select nvl(ton_champ,0) into var frop table

Bonne journée

et dans le cas ou la valeur vaut vraiment 0 ?
et pourquoi faire un traitement sur une variable qui n'existe pas ?
Et ca ne resoud pas le problème ,je crois... je ne suis pas sur qu'un nvl retourne une valeur quand la requete ne ramene rien ...

exempl simple :

Code :

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

select nvl(1,2) from dual where 1=2

Code :

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

1
2
Résultat : 
no rows returned ...

[mnitu]

Bon si tu y tiens vraiment

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
 
declare
  l_ind_trouve number(1);
Begin
  Select *
    Into l_ind_trouve
  From (  
    Select 1
      From dual
     Where Exists (Select Null
                     From  scott.emp e
                    Where e.empno = '7369'
                  )
    Union ALL             
    Select 0
      From dual
     Where Not Exists (Select Null
                     From  scott.emp e
                    Where e.empno = '7369'
                  )
       );
  dbms_output.put_line(to_char(l_ind_trouve));
end;
/

[plaineR]

Bon si tu y tiens vraiment

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
 
declare
  l_ind_trouve number(1);
Begin
  Select *
    Into l_ind_trouve
  From (  
    Select 1
      From dual
     Where Exists (Select Null
                     From  scott.emp e
                    Where e.empno = '7369'
                  )
    Union ALL             
    Select 0
      From dual
     Where Not Exists (Select Null
                     From  scott.emp e
                    Where e.empno = '7369'
                  )
       );
  dbms_output.put_line(to_char(l_ind_trouve));
end;
/

Pour faire simple quand on peut faire compliqué
En plus les perf

[Vincent Rogier]

La solution de miloux32 EST la façon la plus simple et la plus performante.

Je ne vois pas du tout pourquoi tu ne veux pas la mettre en œuvre !

[randriano]

Désolé pour le retard de ma réponse mais j'ai étudié la proposition de miloux32!
Le catch d'exception n'est-il pas unique pour chaque couple begin-end car en fait, si j'utilises un catch exception pour le SELECT INTO où vais-je placer le catch exception général ??
Mon soucis avec votre méthode c'est que la longueur du code va doubler car je devrais tout copier le code après SELECT dans le "When NO_DATA_FOUND then" pour le cas où aucune réponse !
En fait, un select hors procédure crée-t-il une exception ?

[c.langlet]

j'ai lu en biais les reponse , mais il me semble qu'il ne faut pas recopier tout
le code dans l'exception, il suffit de refaire un bloc begin end pour le select

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
 
begin
--traitement avant
 
    begin
 
       SELECT SC.Name INTO v_ClassName  FROM TClasses SC, TEntity OE,    TCategories OC 
       WHERE OE.IDCa = OC.ID AND 
       OC.IDClass = SC.ID AND 
       OE.ID = v_ID; 
    exception
         when no_data_found then
                   v_ClassName := NULL;
      end;
 
--traitement apres avec test de v_classname
end;

[laurentschneider]

éventuellement

Code :

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

1
2
3
4
select (SELECT SC.Name  FROM TClasses SC, TEntity OE,   TCategories OC 
   WHERE OE.IDCa = OC.ID AND 
   OC.IDClass = SC.ID AND 
   OE.ID = v_ID) INTO v_ClassName from dual;

[randriano]

J'avais pensé à votre code c.langlet, donc si ça marche et ça marche, c'est la solution.
Pour votre code laurentschneider, c'est un autre moyen de faire comme sous Access, retourner une chaine vide lorsqu'aucun objet n'est trouvé. C'est aussi une autre solution !!!

[mnitu]

Pour faire simple quand on peut faire compliqué
En plus les perf

J'avais jamais penser à l'utiliser ! Mais si c'était tellement important de faire comme le SQL Server ...

[mnitu]

Il existe toujours une autre technique pour éviter l'exception No_DATA_FOUND qui consiste toute simplement à remplacer le SELECT ... INTO ... FROM par un curseur explicite

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
  Cursor crs Is 
    Select ... 
      From ... 
     Where ..
  l_var crs%RowType;
Begin
  Open crs;
  Fetch crs Into l_var;
  If crs%NOTFOUND THEN
    -- Traitement à faire quand le curseur ne ramène rien
  End If;
  Close crs;
End;

Il y avait une époque ou on disait que c'était plus performant d'utiliser le curseur explicite (c'est un mythe).

A partir de cette idée il est possible d'utiliser la variante suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Declare
  l_ename scott.emp.ename%Type;
Begin
  For crs In (Select ename 
                From scott.emp
               Where empno = 7369)
  Loop
    l_ename := crs.ename;
  End Loop;
  dbms_output.put_line('Name = '||l_ename);
End;
/

que je dois avouer que j'hésiterais avant de l'employer. Et dans le même contexte il pourrait être intéressant d'examiner aussi:

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
 
Declare
  Type refcursor Is ref cursor;
  Cursor crs Is 
    Select cursor(Select ename 
                    From scott.emp 
                   Where empno = 73691)
      From dual;
  --
  l_crs refcursor;
  l_ename scott.emp.ename%Type;  
Begin
  Open crs;
  Fetch crs Into l_crs;
  Fetch l_crs Into l_ename;
  If l_crs%NotFound Then
    l_ename := Null;
  End If;
  Close crs;
  dbms_output.put_line('Name = '||l_ename);
End;
/

[randriano]

merci mnitu pour cette autre technique qu'est l'emploi de curseur explicite, c'est vraiment un autre moyen de contourner le traitement de l'exception no_data_found