Discussion :

[Louis-Guillaume Morand]

J'ai cette SP toute simple mais il ne veut pas de mon declare. j'ai pourtant besoin de cette variable pk_f (j'ai simplifié la SP, il manque du code qui justement, utilise cette variable)

Quelqu'un saurait comment dois-je mettre ce simple petit declare pour avoir une variable de type number que je puisse modifier. je n'ai pas besoin de la retourner au final (out)

merci d'avance

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
CREATE OR REPLACE PROCEDURE sp_createalertfile (
   p_id               OUT      NUMBER,
   p_usr              IN       VARCHAR2 DEFAULT '',
   p_global_comment   IN       VARCHAR2 DEFAULT '',
   p_file             IN       VARCHAR2
)
AS
DECLARE pk_file number;
BEGIN
   SELECT pk_file
     INTO pk_f
     FROM t_files
    WHERE file_name = p_file;
 
   INSERT INTO t_alerts
               (pk_alert, pk_file, global_comment, created_on, created_by
               )
        VALUES (sq_alert.NEXTVAL, pk_f, p_global_comment, SYSDATE, p_usr
               );
 
   SELECT sq_alert.CURRVAL
     INTO pk_id
     FROM DUAL;
END sp_createalertfile;
/

[phili_b]

Tu confonds entre SQL server et Oracle. On utilise IS sous Oracle pour débuter proprement dit la procédure. De plus, contrairement à du PL/SQL seul sans procédure, le mot clé DECLARE n'est pas utilisé. La déclaration de variable se fait entre le IS et le BEGIN.

[Louis-Guillaume Morand]

je sais que AS vient de MSSQL mais pourtant, sous TOAD, ca compile et je crois meme que ca marche :/

par contre (et donc avec le "IS")

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
CREATE OR REPLACE PROCEDURE sp_createalertfile (
   p_id               OUT      NUMBER,
   p_usr              IN       VARCHAR2 DEFAULT '',
   p_global_comment   IN       VARCHAR2 DEFAULT '',
   p_file             IN       VARCHAR2
)
IS
DECLARE pk_file number;
BEGIN
   SELECT pk_file
     INTO pk_f
     FROM t_files
    WHERE file_name = p_file;
 
   INSERT INTO t_alerts
               (pk_alert, pk_file, global_comment, created_on, created_by
               )
        VALUES (sq_alert.NEXTVAL, pk_f, p_global_comment, SYSDATE, p_usr
               );
 
   SELECT sq_alert.CURRVAL
     INTO pk_id
     FROM DUAL;
END sp_createalertfile;
/

ca marche pas plus pour moi exactement la même erreur qu'avec le AS

et si j'enleve le mot "DECLARE", ca plante plus loin en me disant "pk_f must be declared"

[phili_b]

Je ne pense pas me tromper car je vois que tu déclares pk_file mais tu cherches à remplir pk_f qui n'est déclaré nul part.

[Louis-Guillaume Morand]

Je ne pense pas me tromper car je vois que tu déclares pk_file mais tu cherches à remplir pk_f qui n'est déclaré nul part.

effectivement. c'était un mixte de coquille, plus mot clé DECLARE inutile(voire bloquant) et le mot AS

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
CREATE OR REPLACE PROCEDURE sp_createalertfile (
   p_id               OUT      NUMBER,
   p_usr              IN       VARCHAR2 DEFAULT '',
   p_global_comment   IN       VARCHAR2 DEFAULT '',
   p_file             IN       VARCHAR2
)
IS
  pk_f number;
BEGIN
   SELECT pk_file
     INTO pk_f
     FROM t_files
    WHERE file_name = p_file;
 
   INSERT INTO t_alerts
               (pk_alert, pk_file, global_comment, created_on, created_by
               )
        VALUES (sq_alert.NEXTVAL, pk_f, p_global_comment, SYSDATE, p_usr
               );
 
   SELECT sq_alert.CURRVAL
     INTO p_id
     FROM DUAL;
END sp_createalertfile;
/

merci

[Michel SALAIS]

Je crois comprendre que tu considère l'utilisatio du mot AS comme erronée ...
En fait AS ou IS sont utilisables dans ce cas sans aucune différence