Discussion :

[Filippo]

Bonjour,

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

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

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

CALL PH_SEQ5('2012H38');

J'obtiens l'erreur :

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 :

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

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.

[Waldar]

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

Code :

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

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

[Filippo]

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

Code :

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

dbms_output.put_line (stSql);

s'afficher.

J'ai essayé de mettre

Code :

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

serveroutput on;

mais je n'ai pas réussi à voir la valeur de stSql.

[Waldar]

Même avec le dbms_output.enable dans la procédure stockée ?

[Filippo]

Je viens de voir la valeur de stSql.

Il manque des quotes avant et après.

[Waldar]

Enfin sinon je vous le fais le dbms_output :

Code :

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

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

[Filippo]

C'est bon,
ça marche avec :

Code :

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

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

Merci.

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

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

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