Problème dans une procédure

Version imprimable

1 pièce(s) jointe(s)

bonjour tout le monde;
je vous serait très reconnaissant si vous voulez bien me dire ou est e problème.
En faite il s'agit d'une procédure en pl sql qui fait une augmentation salariale
je vous joint le modèle physique en pièce jointe.Pièce jointe 177838

L'énoncé est comme suit :
Automatisez à l’aide d’unité(s) de programmation PL/SQL l’augmentation salariale des employés ayant plus de 10 ans d’expérience pour un travail donné dans l’entreprise.
Une augmentation de salaire signifie passer d’un salaire de niveau n à un salaire de niveau n+1 (classe_salariale.niveau_sal). Les employés en niveau 5 (maximum de classe_salariale.niveau_sal) n’ont pas le droit à une augmentation de salaire.
Notez qu’il n’y a que 5 enregistrements dans la table CLASSE_SALARIALE (correspondant aux 5 niveaux de salaire).

le problème dans ma requête que j arrive pas a trouver comment updater la table travaille
Code:

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 create or replace procedure augmentation_salariale (nombre_années_experience in number) is Begin declare cursor cursor_1 is select emp_id,travaille.sal_id,dep_id , classe_salariale.niveau_sal,debut from travaille, classe_salariale where travaille.sal_id = classe_salariale.sal_id and classe_salariale.niveau_sal < 5 and to_char(sysdate,'yyyy')-to_char(travaille.debut,'yyyy') > nombre_années_experience; niveau_salaire classe_salariale.niveau_sal%type; identifiant_salaire classe_salariale.sal_id%type; identifiant_employe travaille.emp_id%type ; identifiant_dep travaille.dep_id%type; begin open cursor_1; loop fetch cursor_1 into identifiant_employe ,identifiant_dep, identifiant_salaire, niveau_salaire, date_debut ; exit when cursor_1%notfound ; update travaille set emp_id= identifiant_employe, sal_id =(select sal_id from classe_salariale where classe_salariale.niveau_sal= niveau_salaire + 1), dep_id = identifiant_dep, where travaille.debut= date_debut; end loop; close cursor_1; end; end; execute augmentation_salariale(10);

Bonjour,

Quelques petites astuces :
1/ Pas obligé de mettre un DECLARE dans une proc, elle se situe entre le IS et le BEGIN
2/ Eviter si possible les OPEN CURSOR, préférer les FOR LOOP plus simples.

Ensuite pour ton souci d'update, tu y es presque. Tu dois mettre la PK de la table Travaille dans la condition Where car tu mets à jour ligne par ligne.

Code:

set emp_id= identifiant_employe

il ne faut pas, tu ne modifies pas cette colonne.

Je te donne juste une petite correction et la méthode avec le FOR LOOP, à toi de continuer
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 CREATE OR REPLACE PROCEDURE augmentation_salariale (nombre_années_experience IN number) IS CURSOR cursor_1 IS SELECT emp_id,travaille.sal_id,dep_id , classe_salariale.niveau_sal,debut FROM travaille, classe_salariale WHERE travaille.sal_id = classe_salariale.sal_id AND classe_salariale.niveau_sal < 5 AND TO_CHAR(SYSDATE,'yyyy')-TO_CHAR(travaille.debut,'yyyy') > nombre_années_experience; BEGIN FOR res1 IN cursor_1; LOOP UPDATE travaille set sal_id =(SELECT sal_id FROM classe_salariale WHERE classe_salariale.niveau_sal= res1.niveau_sal + 1) WHERE ... pk = cursor.pk_de_travaille.. END LOOP; END;
Bien sur, il est encore optimisable. Le SQL est fait pour traiter ce genre de procédure en une seule requête, mais ce n'est pas le but de ton TP je pense.

problème dans une procédure

Bonjour McM,
Merci infiniment pour tes corrections, ça m'a servis d'exécuter la procédure mais sans aucun effet sur la table travaille .
et effet la procédure s'exécute sans mettre a jour le SAL_ID dans la table travaille;
et je ne sais pas ou est le problème .
je vous joint le nouveau code sql ci-dessous

Code:

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
create or replace procedure augmentation_salariale (nombre_années_experience in number)  is 
 
          cursor cursor_1 is 
               select   emp_id,travaille.sal_id,dep_id , classe_salariale.niveau_sal from   travaille, classe_salariale 
                where  travaille.sal_id = classe_salariale.sal_id 
                 and    classe_salariale.niveau_sal < 5
                  and    to_char(sysdate,'yyyy')-to_char(travaille.debut,'yyyy') > nombre_années_experience; 
 
      niveau_salaire     classe_salariale.niveau_sal%type; 
      identifiant_salaire  classe_salariale.sal_id%type;
      identifiant_employe  travaille.emp_id%type ;
      identifiant_dep    travaille.dep_id%type;
 
      begin 
      for res in cursor_1 loop
 
 
              fetch cursor_1 into identifiant_employe ,identifiant_dep, identifiant_salaire, niveau_salaire ;
               exit when cursor_1%notfound ;
                update travaille  set  sal_id =(select sal_id from classe_salariale where classe_salariale.niveau_sal= niveau_salaire + 1)
                where 
                travaille.emp_id=identifiant_employe and travaille.dep_id=identifiant_dep and travaille.sal_id=identifiant_salaire;
                  commit;
 
      end loop;
 
 
 
end;
 
show errors,
 
execute augmentation_salariale(10);

20/05/2015, 11h15
McM

Et bien voila une seconde raison pour éviter les fetch et préférer le FOR LOOP :)

Code:

select emp_id, travaille.sal_id, dep_id, classe_salariale.niveau_sal from travaille, classe_salariale

Code:

fetch cursor_1 into identifiant_employe ,identifiant_dep, identifiant_salaire, niveau_salaire ;

L'ordre des colonnes n'est pas le bon.
20/05/2015, 11h16
McM

J'ajouterai que pour débugguer, tu peux utiliser du DBMS_OUTPUT.PUT_LINE si tu lances ta requête avec sql+ ou un sqldev, afin d'avoir des traces, ou de mettre des infos dans une table de trace.

Bonjour Mcm;

Merci infiniment pour votre aide très précieuse, voilà j'ai pu résoudre mon problème en se basant bien sur sur vos conseils concernant l'utilisation de boucle à la place de fetch et aussi l'inversion de l'ordre des identifiants.
je mets ci-dessous le code corrigé.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
create or replace procedure augmentation_salariale (nombre_années_experience in number)  is 
 
          cursor cursor_1 is 
               select   emp_id,dep_id ,travaille.sal_id, classe_salariale.niveau_sal from   travaille, classe_salariale 
                where  travaille.sal_id = classe_salariale.sal_id 
                 and    classe_salariale.niveau_sal < 5
                  and    to_char(sysdate,'yyyy')-to_char(travaille.debut,'yyyy') > nombre_années_experience; 
      begin 
      for res in cursor_1 loop
                update travaille  set  sal_id =(select sal_id from classe_salariale where classe_salariale.niveau_sal= res.niveau_sal + 1)
                where 
                travaille.emp_id=res.emp_id and travaille.dep_id=res.dep_id and travaille.sal_id=res.sal_id;
                commit;
      end loop;
end;
 
execute augmentation_salariale(10);

[/CODE]