Problema Procedure SQL

[partygirl85]

Salve a tutti...sono nuova nel forum...ho un problema cn questa procedure sql:


create or replace procedure "RIPRISTINA_REAGENTI" (cod_reazione NUMBER, fattore NUMBER) as
begin
DECLARE
cod REAGENTE.COD_SOSTANZA%TYPE;
moli REAGENTE.NUM_MOLI%TYPE;

CURSOR c1 IS
SELECT COD_SOSTANZA, NUM_MOLI
FROM REAGENTE
WHERE COD_REAZIONE = cod_reazione;

BEGIN OPEN c1;
LOOP
FETCH c1 INTO cod, moli;
EXIT WHEN c1%NOTFOUND;
UPDATE SOSTANZA SET QUANTITA = QUANTITA + fattore*moli WHERE CODICE = cod;
END LOOP;
CLOSE CURSORE;
END;
end;

Il problema è che l'update non avviene solo dove "CODICE = cod" ma viene aggiornato "QUANTITA" su tutte le tutte le righe della tabella.

Spero che qualcuno mi sappia dire qualcosa....GRAZIE

[standard]

Originariamente inviato da partygirl85
CURSOR c1 IS
SELECT COD_SOSTANZA, NUM_MOLI
FROM REAGENTE
WHERE COD_REAZIONE = cod_reazione;

A mio avviso il problema è nella where condition che usi in questo cursore. Utilizzando per la variabile lo stesso nome del campo, alcuni motori DB interpretano in maniera di identità i campi della where condition. L'effetto è una completa full-scan della tabella e, di conseguenza, l'aggiornamento di tutte le sue righe.

Prova ad utilizzare un nome diverso per la variabile.
A presto,
ciao

[YuYevon]

Aggiungerei anche che questo non mi sembra corretto

...
begin
DECLARE 
...

la sezione delle dichiarazioni dovrebbe essere separata da quella del codice, quindi teoricamente dovresti scrivere

DECLARE
..
--dichiarazioni
..
BEGIN
..
--codice
..

E ancora

...
LOOP
FETCH c1 INTO cod, moli; 
...

Sicuro che questa scrittura sia corretta?

[partygirl85]

Originariamente inviato da standard
A mio avviso il problema è nella where condition che usi in questo cursore. Utilizzando per la variabile lo stesso nome del campo, alcuni motori DB interpretano in maniera di identità i campi della where condition. L'effetto è una completa full-scan della tabella e, di conseguenza, l'aggiornamento di tutte le sue righe.

Prova ad utilizzare un nome diverso per la variabile.
A presto,
ciao

GRAZIE MILLE !!! Era proprio questo il problema...

ciaoooo