Discussion :

[yonnel]

Bonjour,

Basé sur l'existence ou non de tables, je souhaite insérer des enregistrements dans une nouvelle table.

Tout fonctionne si je fais:

Code :

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

1
2
3
4
5
6
7
8
9
IF OBJECT_ID('[Table1]') IS NOT NULL 
		SELECT *
		INTO NewTable 
		FROM Table1
 
ELSE IF OBJECT_ID('[Table2]') IS NOT NULL 
		SELECT *
		INTO NewTable 
		FROM Table2

Par contre, j'ai une erreur si je cherche à créer une table temporaire:

Code :

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

1
2
3
4
5
6
7
8
9
IF OBJECT_ID('[Table1]') IS NOT NULL 
		SELECT *
		INTO ##NewTable 
		FROM Table1
 
ELSE IF OBJECT_ID('[Table2]') IS NOT NULL 
		SELECT *
		INTO ##NewTable 
		FROM Table2

Il me dit que "il existe déjà un objet nommé '##NewTable' dans la base de données"... (et la deuxième apparition de ##NewTable est soulignée en rouge)

J'ai du mal à comprendre pourquoi j'ai cette erreur
Est ce que vous savez à quoi cela peut être du?

Merci par avance pour vos réponses

Petite précision: avant de lancer la requête je m'assure que les tables sont effacées!

[darkelend]

Petite précision: avant de lancer la requête je m'assure que les tables sont effacées!

Effacées ou supprimées ? Il faut faire un DROP TABLE avant.

[yonnel]

Supprimé completement avec un DROP TABLE.

En fait mon code ressemble plutot à ca:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
IF OBJECT_ID('tempdb..[##NewTable]') IS NOT NULL DROP TABLE [##NewTable]
 
IF OBJECT_ID('[Table1]') IS NOT NULL 
		SELECT *
		INTO ##NewTable 
		FROM Table1
 
ELSE IF OBJECT_ID('[Table2]') IS NOT NULL 
		SELECT *
		INTO ##NewTable 
		FROM Table2

[darkelend]

Pourquoi ne pas faire ceci plutot ? C'est vrai que je n'aime pas trop le select into par rapport à l'insert into...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
IF OBJECT_ID('tempdb..[##NewTable]') IS NULL
CREATE TABLE [##NewTable] (col1 int, col2 varchar(50), ...)
 
IF OBJECT_ID('[Table1]') IS NOT NULL 
		INSERT INTO ##NewTable
		SELECT *
		FROM Table1
 
ELSE IF OBJECT_ID('[Table2]') IS NOT NULL 
		INSERT INTO ##NewTable
		SELECT *
		FROM Table2

[yonnel]

Salut Darkelend,

Merci beaucoup pour ta réponse.
C'est effectivement une solution mais je voulais l'éviter car le type de mes colonnes peut changer d'une table source à l'autre. La solution INSERT INTO me permet donc de gérer les choses bien plus facilement...

Car la situation réelle est plus complexe que le bout de code que je montre ici... Mais le problème que je rencontre vient de là!

Avant de partir vers une solution alternative, j'aimerais bien comprendre ce qui ne va pas dans mon bout de code

Bug SQL server? Problème de conception? ...

[aieeeuuuuu]

Bonjour,

Lorsque SQL Server parse votre code, il détecte deux INSERT INTO vers la même table, et considère donc que la deuxième exécution n'est pas correcte : il ne "voit" pas le branchement avec le IF qui empêche que ces deux instructions s’exécutent.

de la même façon, si dans un meme lot vous ajoutez une colonne a une table puis vous faites une requete dessus, vous aurez un message d'erreur indiquant que la colonne n'existe pas (alors qu'elle vient d'etre ajoutée ! )

Code :

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

1
2
3
4
5
6
7
 
 
ALTER TABLE UneTable
	ADD UneColonne varchar(30) NULL
 
SELECT UneColonne 
FROM UneTable

Pour contourner le problème vous pouvez mettre vos SELECT INTO dans un EXEC, les commandes seront alors parsées au moment de l'execution :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
IF OBJECT_ID('tempdb..[##NewTable]') IS NOT NULL DROP TABLE [##NewTable]
 
IF OBJECT_ID('[Table1]') IS NOT NULL 
		EXEC('SELECT *
		INTO ##NewTable 
		FROM Table1')
 
ELSE IF OBJECT_ID('[Table2]') IS NOT NULL 
		EXEC('SELECT *
		INTO ##NewTable 
		FROM Table2')

[yonnel]

Merci beaucoup pour l'explication!