Discussion :

[zell61]

Code sql :

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
 
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

[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.

[mnitu]

...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.

[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

[mnitu]

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.

[McM]

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

Code :

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

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 :

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

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
    )