Discussion :

[CROSS]

Bonjour,

Je dois creer un ficheir de commandes pl/sql qui permet de mettre dans la table courrier_empl: num_emp,nom,num_dept,salaire de l'employé si son salaire est superrieur à 2331.00

j'ai un leger probleme:

ORA-06550: Ligne 10, colonne 41 :
PL/SQL: ORA-00933: la commande SQL ne se termine pas correctement

voici mon code:

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
DECLARE 
CURSOR cp IS
SELECT num_emp,nom,num_dept,salaire FROM employes;

BEGIN 

OPEN cp;
LOOP
FETCH cp INTO num_emp,nom,num_dept,salaire;
INSERT INTO courrier_empl VALUES num_emp,nom,num_dept,salaire WHERE salaire>2331;
EXIT WHEN cp%NOTFOUND;
END LOOP;
CLOSE cp;
END;
/

la ligne de l'erreur est la ligne orange

[taska]

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
begin
     for rec in (SELECT num_emp,nom,num_dept,salaire 
                    FROM employes
                    WHERE salaire>2331 ) loop
         begin
         insert into courrier_empl(num_emp,nom,num_dept,salaire)
         values 
         ( rec.num_emp,
           rec.nom,
           rec.num_dept,
           rec.salaire);
          exception WHEN_OTHERS then
            dbms_output.put_line ( sqlerrm ) ;  //  ou sqlerrcode ? à vérifier
          end;
     end loop ;
end;

CDLT.

[CROSS]

heu ca ne serait pas plus simple avec des curseurs comme je l'ai fait ??

parceque la ca ne marche pas du tout j'ai modifier des truc mais bon ca n'a rien changer

[taska]

Je pense que pour la lisibilité ma méthode est meilleure. Néanmoins j'ai saisi le code à l'a

[taska]

Je pense que pour la lisibilité ma méthode est meilleure.
Je pense qu'il est préférable de déclarer des curseurs de façon implicite. Il est plus facile d'analyser ce type de curseurs lorsqu'il sont inclus dans un package de quelques milliers de lignes.
Néanmoins j'ai saisi le code à l'arrache et n'ayant pas tes structures, je ne l'ai pas compilé....

Si veux conserver ton code , 2 PBs sautent aux yeux :

1. La clause where doit être déclarée dans ton curseur et non pas dans l'ordre insert .

Code :

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

SELECT num_emp,nom,num_dept,salaire FROM employes;

à replacer par

Code :

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

SELECT num_emp,nom,num_dept,salaire FROM employes where salaire >2321 ;

2. Tu FETCH des valeurs dans des variables qui ne sont pas déclarées ( initialisées )

Code :

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

FETCH cp INTO num_emp,nom,num_dept,salaire;

CDLT.

[rphenix]

Je reste dans la logique première et propose ceci:

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
DECLARE 
CURSOR cp IS
SELECT num_emp,nom,num_dept,salaire FROM employes;
line cp%ROWTYPE;
BEGIN 
 
OPEN cp;
LOOP
FETCH cp INTO line;
INSERT INTO courrier_empl VALUES line.num_emp,line.nom,line.num_dept,salaire WHERE salaire>2331;
EXIT WHEN cp%NOTFOUND;
END LOOP;
CLOSE cp;
END;
/