ORA-00904 invalid identifier pour un update

Version imprimable

bonjour,
J'ai un problème lors de l'execution d'un update depuis une procedure.

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
 
CREATE TABLE OUTPUT(
ID varchar2(10),
ACTION varchar2(20),
TYPE varchar2(5),
REQ_TYPE NUMBER,
REQ_OBJ NUMBER,
IMSI1 varchar2(50)
);
 
CREATE TABLE CONVERT(
ACTION VARCHAR2(20),
TYPE   VARCHAR2(5),
SET_STMT VARCHAR2(4000)
);
 
insert into OUTPUT values ('x1','CRE_AUC','NT',NULL,NULL,'1234');
 
insert into convert values ('CRE_AUC','NT','REQ_TYPE=1,REQ_OBJ=1,IMSI1=''60503'' || cur.IMSI1');
 
CREATE OR REPLACE PROCEDURE CONVERTING_TEST 
IS
    CURSOR tasks IS SELECT * FROM OUTPUT;
    set_expression varchar2(4000);
    sql_stmt varchar2(4000);
BEGIN
 
    FOR cur IN tasks 
    LOOP
       SELECT SET_STMT
       INTO set_expression
       FROM CONVERT
       WHERE ACTION = cur.ACTION
       AND TYPE = cur.TYPE;
 
        sql_stmt:=  'UPDATE OUTPUT SET ' || set_expression || ' WHERE ID = '''
                || cur.id
                || ''' AND ACTION = '''
                || cur.action
                || '''';
 
        execute immediate sql_stmt;
 
    END LOOP;
 
 
   EXCEPTION
     WHEN NO_DATA_FOUND THEN
       NULL;
     WHEN OTHERS THEN
       -- Consider logging the error and then re-raise
       RAISE;
END CONVERTING_TEST;
/

J'ai le message d'erreur : ORA-00904: "cur.IMSI1": invalid identifier

Est ce que quelqu'un pourrait m'aider?
Je bloquuuuue dessus :cry:

Merci

Ajouter un dbms_output.put_line dans votre procédure avant le execute immediate pour voir le instruction sql généré

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
Connected to Oracle Database 10g Express Edition Release 10.2.0.1.0 
Connected as mni
 
SQL> set serveroutput on
SQL> 
SQL> begin
  2    converting_test;
  3  end;
  4  /
 
UPDATE OUTPUT SET REQ_TYPE=1,REQ_OBJ=1,IMSI1='60503' || cur.IMSI1 WHERE ID = 'x1' AND ACTION = 'CRE_AUC'
 
begin
  converting_test;
end;
 
ORA-00904: "CUR"."IMSI1" : identificateur non valide
ORA-06512: à "MNI.CONVERTING_TEST", ligne 34
ORA-06512: à ligne 3
 
SQL>

De plus je n'aime pas trop votre gestion des anomalies

Code:

1
2
3
4
5
6
7
 
   EXCEPTION
     WHEN NO_DATA_FOUND THEN
       NULL;
     WHEN OTHERS THEN
       -- Consider logging the error and then re-raise
       RAISE;

08/06/2010, 15h08
sassmine82

Merci mnitu pour ta réponse.
Pour la gestion des erreurs, en fait, ce n'est pas la procédure que j'utilise vraiment. C'est un petit exemple pour spécifier mon problème.

Vu que je ne suis pas admin sur la base, je n'ai pas les droits pour : SET serveroutput ON.

mais j'ai mis qd meme le resultat dans une table et il est correct :
UPDATE OUTPUT SET REQ_TYPE=1,REQ_OBJ=1,IMSI1='60503' || cur.IMSI1 WHERE ID = 'x1' AND ACTION = 'CRE_AUC'

J'ai toujours l'erreur ORA-00904: "CUR"."IMSI1": invalid identifier

Tu as une idée ? Merci
08/06/2010, 15h26
xdescamp

Bonjour,

Je ne pense pas qu'on ai besoin des droits d'admin pour faire un "SET serveroutput ON" : pour moi il s'agit d'un fonctionnalité du client Sql*Plus. Tu as un message d'erreur ?

Sinon, l'ordre que tu obtient n'est pas correct :

Code:

UPDATE OUTPUT SET REQ_TYPE=1,REQ_OBJ=1,IMSI1='60503' || cur.IMSI1 WHERE ID = 'x1' AND ACTION = 'CRE_AUC'

Il n'y a aucune table dans ta requête qui s'appelle "cur", donc "cur.IMSI1" ne peut pas être évalué.

Merci xdescamp pour ta réponse.

En fait, dans ma procédure (dans mon 1er post), j'ai un curseur cur :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
FOR cur IN tasks 
    LOOP
       SELECT SET_STMT
       INTO set_expression
       FROM CONVERT
       WHERE ACTION = cur.ACTION
       AND TYPE = cur.TYPE;
 
        sql_stmt:=  'UPDATE OUTPUT SET ' || set_expression || ' WHERE ID = '''
                || cur.id
                || ''' AND ACTION = '''
                || cur.action
                || '''';
 
        execute immediate sql_stmt;

08/06/2010, 15h31
mnitu

Set serveroutput on c'est une commande sqlplus rien avoir avec le DBA.

C'est quoi le ||cur.IMS1 dans ton cas? Testez la commande sous sqlplus.
08/06/2010, 15h37
sassmine82

Citation:

Envoyé par mnitu

C'est quoi le ||cur.IMS1 dans ton cas? Testez la commande sous sqlplus.

c'est un curseur sur la table OUTPUT.
Avec sql plus, la commande marche.

c'est un problème de visibilité, j'ai l'impression. Comme si le curseur n'est pas vu dans "set_expression"
08/06/2010, 15h49
xdescamp

L'ordre que tu récupères ne peux pas fontionner sous Sql*Plus : "cur" n'existe pas. Si tu prends la requête d'UPDATE générée et que tu l'exécutes sous Sql*Plus, elle ne fonctionne pas.
Et c'est ce qui est fait dans ton programme :
- tu construis une chaine de caractères
- puis cette chaine de caractères est exécutée seule, telle quelle, indépendamment (d'un point de vue de la syntaxe) du bloc PL/SQL dans lequel elle a été construite.
08/06/2010, 16h02
sassmine82

Citation:

Envoyé par xdescamp

L'ordre que tu récupères ne peux pas fontionner sous Sql*Plus : "cur" n'existe pas. Si tu prends la requête d'UPDATE générée et que tu l'exécutes sous Sql*Plus, elle ne fonctionne pas.
Et c'est ce qui est fait dans ton programme :
- tu construis une chaine de caractères
- puis cette chaine de caractères est exécutée seule, telle quelle, indépendamment (d'un point de vue de la syntaxe) du bloc PL/SQL dans lequel elle a été construite.

Il n'y a donc pas un moyen de stocker l'expression SET dans une table?
ça ne peut pas marcher par un autre moyen?
08/06/2010, 16h09
xdescamp
Citation:

Envoyé par sassmine82

Il n'y a donc pas un moyen de stocker l'expression SET dans une table?
ça ne peut pas marcher par un autre moyen?

Pas en faisant référence à un morceau du bloc PL/SQL.
Éventuellement, tu peux utiliser une bind variable.
Quelque chose du style :
- tu stockes dans ta table non pas "cur.IMSI1" mais ":cur_imsi1" par ex.
- pour exécuter tu fais :
Code:

1 2 3 ... execute immediate sql_stmt using cur.IMSI1 ; ...
08/06/2010, 16h34
mnitu

Peut être que vous devez nous expliquer un peu ce que vous voulez faire. Certaines solutions sont recommandées dans certains cas et pas dans les autres …