Discussion :

[thief]

Bonjour tous !
j'ai des pbs avec des curseurs! j'explique: j'essaie de mettre à jour les valeurs d'une table avec les valeurs recuperées d'une autre autre table alors pour cela je fais comme suit:

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
17
18
19
20
21
22
23
24
25
26
 
CREATE OR REPLACE PROCEDURE UPDATE_BILL
IS
 
TYPE CALL_REC IS RECORD(id NUMBER,msis CHAR(20),tp CHAR(3), mnt NUMBER);
call CALL_REC;
mont NUMBER;
CURSOR c1 IS SELECT ID,MSISDN,TYPE,MONTANT FROM CALL_I20050801RESUME;
CURSOR c2(id IN CALL_I20050801RESUME.ID%TYPE, msi IN CALL_I20050801RESUME.MSISDN%TYPE, 
         typ IN CALL_I20050801RESUME.TYPE%TYPE)IS
   SELECT CHARGE FROM BILL_I20050801 WHERE SUBSCRIBER_ID=id AND TRIM(HANDSET)=TRIM(msi)
      AND TRIM(CHASERV)=TRIM(typ) FOR UPDATE;
BEGIN
 
   OPEN c1;
      LOOP
         FETCH c1 INTO call;
         EXIT WHEN c1%NOTFOUND;
         OPEN c2(call.id,call.msis,call.tp);
            FETCH c2 INTO mont;
            UPDATE BILL_I20050801 SET CHARGE=call.mnt WHERE CURRENT OF C2;
         CLOSE c2;
      END LOOP;
   CLOSE c1;
 
END;

Après execution j'ai ce message d'erreur:

Code :

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

1
2
3
4
5
 
ERREUR à la ligne 1 :
ORA-01410: invalid ROWID
ORA-06512: at "UPDATE_BILL", line 20
ORA-06512: at line 1

j'ai passé tout le week end la dessus!
Merci pour votre aide d'avance!

[orafrance]

j'ai trouvé une note : http://metalink.oracle.com/metalink/plsql/ml2_documents.showDocument?p_database_id=NOT&p_id=18815.1

This error is raised when an operation refers to a ROWID in a table for which there is no such row.

The reference to a ROWID may be implicit from a WHERE CURRENT OF clause or directly from a WHERE ROWID=... clause.

ORA 1410 indicates the ROWID is for a BLOCK that is not part of this
table. If you update a rowid where the BLOCK is valid but the slot
within the block is invalid you just get ZERO rows processed/returned.
This can be a source of confusion as it is inconsistent.

If you have a cursor that selects from more than one table and
has 'for update' at the end - and then you are trying to do delete/
update WHERE CURRENT OF <cursor_name>, then change the 'for update' to 'for update OF <column_on_table_that doing_dml_on>' -
see Bug 502671 (Closed - not a bug.)

Essaye :

Code :

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

1
2
3
4
CURSOR c2(id IN CALL_I20050801RESUME.ID%TYPE, msi IN CALL_I20050801RESUME.MSISDN%TYPE,
         typ IN CALL_I20050801RESUME.TYPE%TYPE)IS
   SELECT CHARGE FROM BILL_I20050801 WHERE SUBSCRIBER_ID=id AND TRIM(HANDSET)=TRIM(msi)
      AND TRIM(CHASERV)=TRIM(typ) FOR UPDATE OF CHARGE;

[nuke_y]

Comme ça à l'oeil je dirais que tu devrais essayer de ramener toutes les colonnes dans C2.

A mon avis pour faire WHERE CURRENT OF, il a besoin des ROWID et je ne suis pas sûr qu'il les ramène dans ton curseur C2. En même temps ça me paraît pas logique... donc je sais pas trop. Et en évitant le WHERE CURRENT OF ?

[thief]

orafrance
J'ai essayé ta proposition mais la meme erreur!!
je desespère un peu!

[Jaouad]

Est ce que tu analyse tes objets fréquemment , reconstruire les index ?

[orafrance]

t'es sûr que la ligne existe pour les paramétres que tu mets ? Tu serais bien inspirer d'ajouter un test

[McM]

Je ne sais pas si ça diffère, mais essaye avec des FOR IN plutôt qu'avec des FETCH

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
17
18
19
20
21
22
CREATE OR REPLACE PROCEDURE UPDATE_BILL 
IS 
CURSOR c1 IS SELECT ID,MSISDN,TYPE,MONTANT FROM CALL_I20050801RESUME; 
CURSOR c2(ID IN CALL_I20050801RESUME.ID%TYPE, msi IN CALL_I20050801RESUME.MSISDN%TYPE, 
         typ IN CALL_I20050801RESUME.TYPE%TYPE)IS 
   	SELECT CHARGE 
	FROM BILL_I20050801 
	WHERE SUBSCRIBER_ID = ID 
	AND TRIM(HANDSET) = TRIM(msi) 
    AND TRIM(CHASERV) = TRIM(typ) 
	FOR UPDATE; 
BEGIN 
   FOR r1 IN c1 
   LOOP 
      FOR r2 IN c2(r1.ID, r1.msis, r1.tp) 
      LOOP
         UPDATE BILL_I20050801 
			SET CHARGE=r1.montant 
			WHERE CURRENT OF c2; 
	  END LOOP; 
   END LOOP; 
END;

[plaineR]

Essaie (j'ai ajouté la condition de sortie si c2 ne ramène rien) :

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
17
18
19
20
21
22
23
24
25
26
CREATE OR REPLACE PROCEDURE UPDATE_BILL 
IS 
 
TYPE CALL_REC IS RECORD(id NUMBER,msis CHAR(20),tp CHAR(3), mnt NUMBER); 
call CALL_REC; 
mont NUMBER; 
CURSOR c1 IS SELECT ID,MSISDN,TYPE,MONTANT FROM CALL_I20050801RESUME; 
CURSOR c2(id IN CALL_I20050801RESUME.ID%TYPE, msi IN CALL_I20050801RESUME.MSISDN%TYPE, 
         typ IN CALL_I20050801RESUME.TYPE%TYPE)IS 
   SELECT CHARGE FROM BILL_I20050801 WHERE SUBSCRIBER_ID=id AND TRIM(HANDSET)=TRIM(msi) 
      AND TRIM(CHASERV)=TRIM(typ) FOR UPDATE; 
BEGIN 
 
   OPEN c1; 
      LOOP 
         FETCH c1 INTO call; 
         EXIT WHEN c1%NOTFOUND; 
         OPEN c2(call.id,call.msis,call.tp); 
            FETCH c2 INTO mont; 
          exit when c2%notfound;
            UPDATE BILL_I20050801 SET CHARGE=call.mnt WHERE CURRENT OF C2; 
         CLOSE c2; 
      END LOOP; 
   CLOSE c1; 
 
END;

[thief]

Merci à vous tous!
j'ai essayé les deux propositions!
Les deux tournent cependant c'est celui de Mcn qui a resolu mon pb!
vive la communauté!!!

[McM]

J'ai corrigé mon précédent code (mauvaise valeur de l'update) mais tu avais du le voir.. et suppresssion des variables inutiles.
Par contre, si tu ne fais que ça, le mieux est de faire un update de masse, c'est plus rapide qu'un update dans un curseur.

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
CREATE OR REPLACE PROCEDURE UPDATE_BILL 
IS 
CURSOR c1 IS 
SELECT ID, MSISDN, TYPE, MONTANT 
FROM CALL_I20050801RESUME; 
 
BEGIN 
   FOR r1 IN c1 
   LOOP 
		UPDATE BILL_I20050801 
		SET CHARGE = r1.montant
		WHERE SUBSCRIBER_ID = r1.ID 
		AND TRIM(HANDSET) = TRIM(r1.msis) 
		AND TRIM(CHASERV) = TRIM(r1.tp);
   END LOOP; 
END;