Discussion :

[Jinkas99]

Bonjour,

Je voudrais récupérer un numéro de séquence et l'insérer dans une table dans un champs de type varchar. Je voudrais savoir quelle serait la plus appropriée parmi les 2 solutions proposées ci-dessous.

Code :

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

1
2
3
4
5
6
7
CREATE OR REPLACE FUNCTION GET_SEQ_GROUP_ENT(SEQ_NAME IN VARCHAR2) AS
DECLARE
    seq_out NUMBER;
BEGIN
    EXECUTE IMMEDIATE 'SELECT ' || SEQ_NAME || '.NEXTVAL from dual' INTO seq_out;
    RETURN seq_out;
END;

ou

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
create or replace procedure next_val (p_sequence_name varchar2)
as
 
v_nextval integer;
v_select varchar2(100);
 
begin
 
v_select := 'select '||p_sequence_name||'.nextval from dual';
 
execute immediate v_select into v_nextval;
 
dbms_output.put_line('Nextval is: '||TO_CHAR(v_nextval));
 
end;

Merci,

[mnitu]

Aucune. La meilleur solution c'est d'utiliser la séquence lors de l'insert

Code :

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

1
2
 
Insert into tab (id, val) values (seq.nextval, 'VALEUR')

[Jinkas99]

Bonjour,

Je vous remercie pour votre réponse.

Je suis d'accord avec votre proposition. Sauf que j'utilise un framework spécifique qui fait appel à des procédures stockées et donc, pour garder une cohérence dans le code, je souhaiterais créer une procédure stockée.

Merci,

[mnitu]

Faites une fonction qui logiquement réponde à la demande: "donne-moi la prochaine valeur de la séquence".

Mais j'ai toujours mal à comprendre les approchent totalitaristes des développement surtout quand cela se base sur des mauvaise approches.

Puis je n'aime pas l'utilisation abusive des requêtes dynamiques. Mais si vous y tenez à les utiliser prenez soin de lire "How to write SQL injection proof PL/SQL" de Bryn LLewellyn pour apprendre à faire ça correctement.

[Jinkas99]

Merci encore pour vos conseils.

Donc cela devrait suffire :

Code :

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

1
2
3
4
5
6
7
CREATE OR REPLACE FUNCTION GET_SEQ_GROUP_ENT(SEQ_NAME IN VARCHAR2) AS
DECLARE
    seq_out NUMBER;
BEGIN
    EXECUTE IMMEDIATE 'SELECT ' || SEQ_NAME || '.NEXTVAL from dual' INTO seq_out;
    RETURN seq_out;
END;

Merci,

[mnitu]

Non cette fonction ne passera pas un test de sécurité du code.