Variable CHAR dans EXECUTE IMMEDIATE

Version imprimable

Bonjour,

J'arrive à créer correctement la procédure stockée suivante :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 CREATE OR REPLACE PROCEDURE PH_SEQ5 ( MyPeriode CHAR ) IS -- Déclaration variables locales stSql VARCHAR2(3000); BEGIN stSql:='CREATE OR REPLACE VIEW PH_VUE_TEMP AS SELECT * FROM PH_EMI_AG WHERE TRTPER_PNU='||MyPeriode; EXECUTE IMMEDIATE (stSql); END; /
Mais à l'exécution avec une variable alphanumérique :

Code:

CALL PH_SEQ5('2012H38');

J'obtiens l'erreur :

Citation:

CALL PH_SEQ5('2012H38')
Error at line 33
ORA-00933: SQL command not properly ended
ORA-06512: at "A263.PH_SEQ5", line 17

Si j'exécute le code suivant ça marche :

Code:

CALL PH_SEQ5(4);

Dans la table PH_EMI_AG, le champ TRTPER_PNU est de type VARCHAR2(7).

J'ai essayé différentes combinaisons avec les quotes sans succès.

Si quelqu'un a une idée, je suis preneur.

Merci.

Comme bien souvent, afficher la chaîne à exécuter vous mettra la puce à l'oreille :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
CREATE OR REPLACE PROCEDURE PH_SEQ5 (MyPeriode CHAR)
IS
    -- Déclaration variables locales
    stSql VARCHAR2(3000);
 
    BEGIN
        dbms_output.enable;
        stSql:='CREATE OR REPLACE VIEW PH_VUE_TEMP AS SELECT * FROM PH_EMI_AG WHERE TRTPER_PNU='||MyPeriode;
        dbms_output.put_line (stSql);
        -- EXECUTE IMMEDIATE (stSql);
    END;
/

21/08/2013, 15h58
Filippo

Merci Waldar, j'y ai bien pensé mais je ne vois pas le résultat du

Code:

dbms_output.put_line (stSql);

s'afficher.

J'ai essayé de mettre

Code:

serveroutput on;

mais je n'ai pas réussi à voir la valeur de stSql.
21/08/2013, 15h59
Waldar

Même avec le dbms_output.enable dans la procédure stockée ?
21/08/2013, 16h00
Filippo

Je viens de voir la valeur de stSql.

Il manque des quotes avant et après.

Enfin sinon je vous le fais le dbms_output :

Code:

1
2
3
4
5
6
7
CALL PH_SEQ5('2012H38');
 
CREATE OR REPLACE VIEW PH_VUE_TEMP AS SELECT * FROM PH_EMI_AG WHERE TRTPER_PNU=2012H38
 
CALL PH_SEQ5(4);
 
CREATE OR REPLACE VIEW PH_VUE_TEMP AS SELECT * FROM PH_EMI_AG WHERE TRTPER_PNU=4

21/08/2013, 16h02
Filippo

C'est bon,
ça marche avec :

Code:

stSql:='CREATE OR REPLACE VIEW PH_VUE_TEMP AS SELECT * FROM PH_EMI_AG WHERE TRTPER_PNU='''||MyPeriode||'''';

Merci.

;)
21/08/2013, 16h09
Waldar

Maintenant que le problème de départ est résolu, j'attire votre attention sur la relative inutilité de cette procédure, je ne comprends pas vraiment la nécessité de créer un objet Oracle simplement pour appliquer un filtre.
21/08/2013, 19h05
dariyoosh

Et plus généralement, mettre des DDL (CREATE TABLE, CREATE VIEW, etc.) à l'intérieur de EXECUTE IMMEDIATE n'est pas considérée comme une bonne approche de programmation.
21/08/2013, 19h55
Filippo

J'ai complété la procédure avec d'autres actions.

Je dois automatiser des requêtes pour que des personnes qui n'y connaissent presque rien en sql les utilisent sous toad sans toucher au code.

Je ne suis pas informaticien. Je suis conscient que mon code n'est pas optimal.
Je tiens néanmoins compte de vos remarques.
Merci pour vos conseils.