Problème d'insert en SQL dynamique

**greg75** · 22/08/2007, 11h40

Bonjour,

Je crée une table via une procédure en SQL dynamique (ca fonctionne).

J'insère des données via la commande suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
request := 'insert into TBL_DIAG_CODE_' || tblRef ||' (ncDiagCode, ncMotif, ncSousMotif, ';
request := request || 'ncActionType, swDateCreated, ncDiagCodeId) values ( ';
request := request || '''' || t_ncDiagCode(i) || ''', ';
request := request || '''' || t_ncMotif(i) || ''', ';
request := request || '''' || t_ncSousMotif(i) || ''', ';
request := request || '''' || t_ncActionType(i) || ''', ';
request := request || '''' || t_swDateCreated(i) || ''', ';
request := request || t_ncDiagCodeId(i) || ')';
 
execute immediate request;

Lorsque ce code est exécuté, la table reste vide. Lorsque je débugge la procédure et que je fais un copier/coller de la valeur contenue dans la variable "'request" et que je l'exécute manuellement, cela fonctionne.

Par avance merci de votre aide.

Greg

**orafrance** · 22/08/2007, 11h54

t'a pensé au COMMIT ?

**greg75** · 22/08/2007, 14h03

Oui. J'avais rajouté un "execute immediate 'commit';" mais ca ne change rien.

**Garuda** · 22/08/2007, 14h08

Envoyé par greg75

Oui. J'avais rajouté un "execute immediate 'commit';" mais ca ne change rien.

Il ne faut pas faire "execute immediate 'commit';"
Mais simplement

Code :

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

1
2
3
4
 
...
COMMIT;
....

**orafrance** · 22/08/2007, 14h14

l'affichage de la requête via DBMS_OUTPUT est souvent très utile

**rbaraer** · 22/08/2007, 16h46

Alerte aux variables de lien !

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
request := 'insert into TBL_DIAG_CODE_' || tblRef ||' (ncDiagCode, ncMotif, ncSousMotif, ';
request := request || 'ncActionType, swDateCreated, ncDiagCodeId) values ( ';
request := request || '''' || :ncDiagCode || ''', ';
request := request || '''' || :ncMotif || ''', ';
request := request || '''' || :ncSousMotif || ''', ';
request := request || '''' || :ncActionType || ''', ';
request := request || '''' || :swDateCreated || ''', ';
request := request || :ncDiagCodeId || ')';
 
execute immediate request
using t_ncDiagCode(i),
    t_ncMotif(i),
    t_ncSousMotif(i),
    t_ncActionType(i),
    t_swDateCreated(i),
    t_ncDiagCodeId(i);

Sans variable de lien vos requêtes ne restent pas dans le SHARED POOL ce qui entraine de lourdes dégradations de perfs, et en plus elles font sortir du shared pool les requêtes "bien écrites"

.

http://asktom.oracle.com/pls/asktom/f?p=100:11:0:::

11_QUESTION_ID:2588723819082
http://asktom.oracle.com/pls/asktom/f?p=100:11:0:::

11_QUESTION_ID:104931400346587790

De plus, pas de variable de lien = risque d'injection SQL

rbaraer

**Garuda** · 22/08/2007, 16h52

Attention, si utilisation de variables de lien (bind variable) modifier la requête (pas de quotes) !

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
request := 'insert into TBL_DIAG_CODE_' || tblRef ||' (ncDiagCode, ncMotif, ncSousMotif, ';
request := request || 'ncActionType, swDateCreated, ncDiagCodeId) values ( ';
request := request || ':ncDiagCode , :ncMotif ,:ncSousMotif , ';
request := request || ':ncActionType ,:swDateCreated , :ncDiagCodeId )';
execute immediate request
using t_ncDiagCode(i),
    t_ncMotif(i),
    t_ncSousMotif(i),
    t_ncActionType(i),
    t_swDateCreated(i),
    t_ncDiagCodeId(i);
COMMIT;