Encore une procédure

Version imprimable

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
34
35
36
 
CREATE OR REPLACE PROCEDURE proc_logem_dimin_pourc  
(p_pourc IN NUMBER,
 p_nomquart IN Quartier.nomquart%TYPE,
 p_nomcom IN Commune.nomcom%TYPE)
 
IS
 
CURSOR Cur_Logements_pourc IS
 
SELECT
L.idlog, L.numero, L.loyer, Q.nomquart, C.nomcommune
FROM
Commune C, Quartier Q, Logement L
WHERE
C.idcom = Q.idcom
AND
Q.idquart = L.idquart
AND
p_nomquart = Q.nomquart
AND
p_nomcom = C.nomcom
FOR UPDATE OF L.loyer;
 
BEGIN
 
FOR enreg IN Cur_Logements_pourc
LOOP
UPDATE Logement SET
L.loyer = (1 - p_pourc/100) * L.loyer
WHERE CURRENT OF Cur_Logements_pourc;
END LOOP;
 
END proc_logem_dimin_pourc ;
 
/

pourquoi me met il cette erreur ????????

ERREUR à la ligne 23 : PL/SQL: SQL Statement ignored 0,00 secondes

26/03/2012, 10h53
Z3phur

bonjour,

pourquoi dans ton curseur tu fais FOR UPDATE?

Peux-tu l'enlever pour voir.

N'oublie pas de faire un commit dans ta procédure.
26/03/2012, 14h02
mnitu

Citation:

Envoyé par zell61

...pourquoi me met il cette erreur ????????

ERREUR à la ligne 23 : PL/SQL: SQL Statement ignored 0,00 secondes

Vous voulez de l'aide ? Fournissez un jeu d'essai complét: création des tables, insertion des données, code des procédures, messages d'erreur, etc.
29/03/2012, 14h42
Bibeleuh

Quand je copie colle ton code j'ai un avertissement :

A cursor for loop that contains DML statements should be refactored to use BULK COLLECT and FORALL

Je te conseil donc au lieu de faire un curseur de faire un bulck collect de ton select directement dans un tableau contenant tes 4 champs.

Ensuite tu pourra faire un
FORALL enreg IN tontableau
Update.....

Désolé pour cette syntaxe très approximative :P
29/03/2012, 16h25
mnitu

Citation:

Envoyé par Bibeleuh

Un truc dans le genre ?
...

La syntaxe proposée est très approximative et ajouter un commit dans la procédure n’est pas une bonne idée non plus.

C'est ton update qui a un alias de table inconnu

Code:

1
2
3
 UPDATE Logement 
  SET loyer = (1 - p_pourc/100) * loyer
  WHERE CURRENT OF Cur_Logements_pourc;

mais bien sur faire une boucle pour un update, n'est pas optimisé.
Un simple update suffit

Code:

1
2
3
4
5
6
7
8
9
10
UPDATE Logement L
SET loyer = (1 - p_pourc/100) * loyer
WHERE EXISTS ( 
    SELECT 1
    FROM Commune C, Quartier Q
    WHERE C.idcom = Q.idcom
    AND Q.idquart = L.idquart
    AND Q.nomquart = p_nomquart
    AND C.nomcom = p_nomcom
    )