Discussion :

[weed]

Bonjour, je suis en train de tester les procédure et je viens de realiser celle ci :

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
create or replace procedure RAS 
is
  cursor cnb is select nb_emprunt from exemplaires;
begin
  for vnb in cnb
  loop
    case 
      when vnb.nb_emprunt<=10                        then update exemplaires set etat='NE';
      when vnb.nb_emprunt>10 and vnb.nb_emprunt<=25  then update exemplaires set etat='BO';
      when vnb.nb_emprunt>26 and vnb.nb_emprunt<=40  then update exemplaires set etat='MO';
      when vnb.nb_emprunt>40                         then update exemplaires set etat='MA';
      else 0;
    end case;  
  end loop;
end;

Au moment de l'execution, j'obiens ceci :

Connecting to the database oracle.
ORA-06592: CASE introuvable lors de l'exécution de l'instruction CASE
ORA-06512: à "HR.RAS", ligne 7
ORA-06512: à ligne 2
Process exited.
Disconnecting from the database oracle.

Et lorsque regarde l'erreur 06592

ORA-06592: CASE not found while executing CASE statement
Cause: A CASE statement must either list all possible cases or have an else clause.
Action: Add all missing cases or an else clause.

http://ora-06592.ora-code.com/

Il faudrait soit indiquer tous les cas possibles dans le cas, soit mettre un else. Bref je ne vois pas ou ils veulent en venir.

Savez pourquoi j'ai une erreur ?
est ce que vous arrivez à executer la procédure ?

[laurentschneider]

Code :

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

else null;

[McM]

Attention, tu updates toutes les lignes dans tes conditions !

Mieux vaut inclure le CASE dans un update directement, genre

Code :

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

1
2
3
4
5
6
7
8
9
UPDATE exemplaires
SET etat = (CASE 
      WHEN nb_emprunt<=10   THEN 'NE'
      WHEN nb_emprunt>10 AND nb_emprunt<=25  THEN 'BO'
      WHEN nb_emprunt>26 AND nb_emprunt<=40  THEN 'MO'
      WHEN nb_emprunt>40   THEN 'MA'
      ELSE NULL
    END)
WHERE nb_emprunt IS NOT NULL

[laurentschneider]

soit

Code :

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

1
2
3
4
5
6
7
UPDATE exemplaires
SET etat = CASE 
      WHEN nb_emprunt<=10 THEN 'NE'
      WHEN nb_emprunt<=25 THEN 'BO'
      WHEN nb_emprunt<=40 THEN 'MO'
      WHEN nb_emprunt>40  THEN 'MA'
      ELSE 0 end;

soit (équivalent à la requête de McM, écrit différement)

Code :

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

1
2
3
4
5
6
7
8
UPDATE exemplaires
SET etat = CASE 
      WHEN nb_emprunt<=10  THEN 'NE'
      WHEN nb_emprunt<=25  THEN 'BO'
      WHEN nb_emprunt<=40  THEN 'MO'
      ELSE 'MA'
    END
WHERE nb_emprunt IS NOT NULL

[McM]

Yop, pas vu cette amélioration du case de ne traiter que l'inférieur.

[weed]

Merci à vous tous, en effet cela faisait une mise à jour sur toute la table ce qui n'etait pas l'effet voulu bien sur mais je voulais avant tout faire des tests.

Avant de voire vos réponses, j'ai cependant changé ma procédure pour faire proprement la mise à jour sur la ligne concerné.

L'idée d'utiliser la clause nb_emprunt IS NOT NULL n'est pas une bonne option à mon sens. Tu vas empecher de tout mettre à jour certe mais pas la bonne ligne.


le mieux est je pense d'utiliser :
- where current of cnb; (le nom du curseur)
- for update of etat; (pour indiquer la table que l'on est train de lister va etre mise à jour sur le champs "etat")


Par contre l'idée de faire un case directement dans un update est assez séduisante en effet.


Voici ma nouvelle procédure :

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
create or replace procedure RAS 
is
  cursor cnb is select nb_emprunt from exemplaires for update of etat;
begin
  for vnb in cnb
  loop
    case 
      when vnb.nb_emprunt<=10                        then update exemplaires set etat='NE' where current of cnb;
      when vnb.nb_emprunt>10 and vnb.nb_emprunt<=25  then update exemplaires set etat='BO' where current of cnb;
      when vnb.nb_emprunt>25 and vnb.nb_emprunt<=40  then update exemplaires set etat='MO' where current of cnb;
      else update exemplaires set etat='MA' where current of cnb;
 
    end case;  
  end loop;
end;

Elle fonctionne très bien mais par contre je ne comprends pas trop pourquoi il faut absolument utiliser ELSE dans un CASE.


J'utilise comme doc : Oracle PL/SQL Programming (O'Reilly)

et on peux voire que le else est optionnel

This CASE statement has an explicit ELSE clause; however, the ELSE is optional. When you do not explicitly specify an ELSE clause of your own, PL/SQL implicitly uses the following:

ELSE
RAISE CASE_NOT_FOUND;

et on peux y voire un de ces exemples sans else :

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
 
    CASE
    WHEN salary >= 10000 THEN
       CASE
       WHEN salary <= 20000 THEN
          give_bonus(employee_id, 1500);
       WHEN salary > 40000 THEN
          give_bonus(employee_id, 500);
       WHEN salary > 20000 THEN
          give_bonus(employee_id, 1000);
       END CASE 
;
    WHEN salary < 10000 THEN
       give_bonus(employee_id,0);
    END CASE;

[McM]

Pour le Else en plsql, je ne sais pas.

par contre ta procédure fait la même chose que cette requete simple

Code :

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

1
2
3
4
5
6
7
UPDATE exemplaires
SET etat = CASE 
      WHEN nb_emprunt<=10  THEN 'NE'
      WHEN nb_emprunt<=25  THEN 'BO'
      WHEN nb_emprunt<=40  THEN 'MO'
      ELSE 'MA'
    END

Sauf que si ta table fait 10000 lignes, dans ta procédure tu vas faire 10000 updates.
Ma condition where était pour gérer le cas de nb_emprunt NULL que tu gérais par un ELSE 0; (corrigé par un ELSE NULL; ) qui ne faisait pas d'update.

Dans ta nouvelle procédure, dans le cas d'un nb_emprunt NULL, tu mets 'MA'

[degio1er]

Salut,

merci McM pour ta réponse, grâce au

Code :

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

else null;

j'ai fini ma requête.