Discussion :

[sassmine82]

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

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

Merci

[mnitu]

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

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

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

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;

[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

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

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

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

[sassmine82]

Merci xdescamp pour ta réponse.

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

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

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

[sassmine82]

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"

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

[sassmine82]

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?

[xdescamp]

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 :

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

1
2
3
...
execute immediate sql_stmt using cur.IMSI1 ;
...

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