Discussion :

[tway91]

bonjour je veux faire une selection dynamique et j'ai cette erreur :
ORA-00917: missing comma. je ne vois pas ou est ce que je dois rajouté une virgule, help me please !!

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
EXECUTE IMMEDIATE ( 'INSERT
                      INTO '|| lv_instance_name ||'@db-link ( stat_id,
                                                                  type,
                                                                  stat_date,
                                                                  db_name,
                                                                  value_1,
                                                                  value_2,
                                                                  value_3,
                                                                  value_4,
                                                                  value_5
                                                                 )
                                      VALUES ('||n_stat_id||','''
                                              ||l_type||''','''
                                              ||to_char(n_stat_date)||''','
                                              ||lv_database_name||','
                                              ||n_value_1||','    
                                              ||n_value_2||','
                                              ||n_value_3||','
                                              ||n_value_4||','
                                              ||n_value_5||')');

[Sunchaser]

Bonjour,
Il me semble que tu as un "value_5", juste avant VALUES, qui se termine avec une virgule mais rien derrière.

[tway91]

Oui c'est une erreur de ma part quand j'ai copier coller le code mais cependant j'ai toujours l'erreur..

[al1_24]

Il manque une virgule à la fin des lignes 5 et 6

[tway91]

modifier a l'instant, mais ce n'est pas le probleme ... il y a toujours cette erreur

[skuatamad]

1/ Il suffit d'afficher la requête générée afin de pouvoir la tester syntaxiquement...
2/ Utiliser des requêtes paramétrées afin de simplifier l'écriture de la requête, limiter le parse des requêtes générées et éviter les injections sql.

Un exemple :

Code :

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

1
2
3
4
5
6
l_query : = 'INSERT INTO '|| lv_instance_name ||'@db-link ( stat_id, type, stat_date, db_name, value_1, value_2, value_3, value_4, value_5)
             VALUES (:1, :2, to_char(:3), :4, :5, :6, :7, :8, :9)';
--
dbms_output.put_line (l_query);
--
EXECUTE IMMEDIATE l_query using n_stat_id, l_type, n_stat_date, lv_database_name, n_value_1, n_value_2, n_value_3, n_value_4, n_value_5;

PS : Le to_char sur une colonne qui pourrait être une date sans préciser le moindre format semble douteux

[tway91]

merci skuatamad je l'ai testé comme tu me l'a dis mais j'ai comme erreur cette fois :ORA-06502: PL/SQL: numeric or value error: character string buffer too small

[mnitu]

A priori une variable mal taillée.

[tway91]

j'ai résolu le problème, j'avais oublié un quote, merci pour vous réponses