Discussion :

[alex181]

bonjour,

Je voudrais écrire un script SQL qui puisse être exécuté plusieurs fois sans erreurs. A la fin du script, on supprime une colonne.
Si on exécute une 2eme fois, ca envoie une erreur indiquant que la colonne n'existe pas. Normal ..... sauf que j'ai mis un test "if exists (SELECT * FROM INFORMATION_SCHEMA.COLUMNS where COLUMN_NAME="macolonne")"

Et là, je ne comprends plus

Code :

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

1
2
3
4
5
6
7
8
9
10
if exists(SELECT * FROM INFORMATION_SCHEMA.COLUMNS where TABLE_CATALOG='cat1' and
 TABLE_SCHEMA='schema1' and TABLE_NAME='table1' and COLUMN_NAME='NiveauPyramide')
BEGIN
    --traitement utilisant la colonne 'NiveauPyramide'
    update table2 set col1= ... where table1.NiveauPyramide
 
    --suppression de la colonne
    alter table schema1.table1 drop column NiveauPyramide
END
GO

a l'exécution l'erreur suivante remonte:
Msg*207, Niveau*16, État*1, Ligne*17
Nom de colonne non valide*: 'NiveauPyramide'.

[zinzineti]

Qu'est ce qui vous empêche de nous présenter correctement vos requêtes ? vous nous balancer des requêtes tronquées avec des points de suspension ... en espérant que nous allons deviner ce que vous avez réellement écrit !

[Rei Ichido]

1. L'erreur correspond à laquelle des deux instructions ?
2. L'update porte sur table2, c'est normal ?

[Donpi]

Et comme ça, ça marche ?

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
IF EXISTS(
				SELECT C.object_id
				FROM sys.Columns C
				INNER JOIN sys.Tables T on C.object_id=T.object_id
				inner join sys.schemas S ON S.schema_id = T.schema_id
				WHERE S.name='schema1'
				AND T.Name ='table1'
				AND C.Name = 'NiveauPyramide'
			)
BEGIN
    --traitement utilisant la colonne 'NiveauPyramide'
    UPDATE table2 SET col1= ... WHERE table1.NiveauPyramide
 
    --suppression de la colonne
    ALTER TABLE schema1.table1 DROP COLUMN NiveauPyramide
END
GO

[zinzineti]

Non il y a encore des points de suspension dans ta requête ! on ne veut pas voir les points de suspension !

[Donpi]

Non il y a encore des points de suspension dans ta requête ! on ne veut pas voir les points de suspension !

Pfff... on est pas sur jeuxvideo.com les gars

On commence par lire ... tout ... y compris le pseudo...

[alex181]

Désolé pour les points de suspension, je voulais simplifier le problème
Je vous représente le problème de manière simplifiée mais complète cette fois-ci.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
if exists(
	SELECT * 
	FROM INFORMATION_SCHEMA.COLUMNS
	where TABLE_CATALOG='cat1' 
	and TABLE_SCHEMA='schema1' 
	and TABLE_NAME='table1' 
	and COLUMN_NAME='NiveauPyramide'
	)
		--print 'existe'
		select t.NiveauPyramide from cat1.schema1.table1 t;
else
		print 'existe pas'

Les 2 instructions 'print' fournissent le résultat attendu (selon que la colonne existe ou non)

Pourquoi le select t.NiveauPyramide renvoit une erreur ??? (si la colonne n'existe pas, ces instructions ne devraient, à mon sens, même pas être exécutées, non?

[Donpi]

Désolé pour les points de suspension, je voulais simplifier le problème
Je vous représente le problème de manière simplifiée mais complète cette fois-ci.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
if exists(
	SELECT * 
	FROM INFORMATION_SCHEMA.COLUMNS
	where TABLE_CATALOG='cat1' 
	and TABLE_SCHEMA='schema1' 
	and TABLE_NAME='table1' 
	and COLUMN_NAME='NiveauPyramide'
	)
		--print 'existe'
		select t.NiveauPyramide from cat1.schema1.table1 t;
else
		print 'existe pas'

Les 2 instructions 'print' fournissent le résultat attendu (selon que la colonne existe ou non)

Pourquoi le select t.NiveauPyramide renvoit une erreur ??? (si la colonne n'existe pas, ces instructions ne devraient, à mon sens, même pas être exécutées, non?

Chez moi ce code requête fonctionne bien.
C'est avec quel version de SQL ?

[alex181]

il faut tester dans le cas où la colonne n'existe pas !!

Je suis sur SQL Server 2008 R2 (Express)

[Donpi]

Votre requête n'existe pas chez moi (forcément) et je n'ai pas d'erreur.

J'ai pris exactement le même code pour tester avec une table et une requête a moi, ça marche aussi.

Sur la même version que vous.

Vous utilisé Management Studio ?

[SQLpro]

Vous mélangez du Transact SQL (IF....) et du SQL (SELECT...). L'un est transactionné l'autre pas. Entre temps (du IF au reste) les choes peuvent avoir bougé.
De plus si c'est dans une procédure il faudrait forcer la recompilation.
Autrement dit, transaction nexplicte au niveau SERIALIZABLE + recompilation....

Cela dit, ajouter et supprimer en permanence des colonnes à une table est de la plus haute stupidité. Il faut rechercher la stabilité du schéma...

SI vous nous parliez de votre besoin fonctionnel ? C'est quoi la nécessité d'ajouter supprimer une colonne ?

A +

[alex181]

merci pour vos réponses!
Oui, c'est sous Management Studio

D'accord pour le mélange T-SQL et SQL "classique".

Ce n'est pas une procédure, je suis bien d'accord qu'ajouter/supprimer une colonne n'est une chose à faire tout le temps!

C'est un script qui devrait être exécuté une seule fois.
Je voulais le rendre "plus propre" et faire en sorte qu'il n'y ait pas d'erreur s'il est exécuté plusieurs fois.

Je fais ça d'habitude lorsque j'ajoute une colonne (pour éviter le message indiquant que la colonne existe déjà) et ça marche bien.

Dans le cas présent, c'est plus compliqué parce qu'une colonne est supprimée.

[SQLpro]

Si c'est un batch, il faut en séparer les éléments par des GO, car l'analyse concerne tout le lot sans GO.....

A +

[iberserk]

Si c'est un batch, il faut en séparer les éléments par des GO, car l'analyse concerne tout le lot sans GO.....

A +

Dans votre cas ce n'est pas la solution.
En effet votre script tente d'effectuer une opération si une colonne existe encore avant de supprimer cette dernière.

Celà ne compilera jamais, il faut passer par sp_executeSql dont le texte de requète ne sera pas compilé:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
IF EXISTS(SELECT * FROM INFORMATION_SCHEMA.COLUMNS WHERE TABLE_CATALOG='cat1' AND
 TABLE_SCHEMA='schema1' AND TABLE_NAME='table1' AND COLUMN_NAME='NiveauPyramide')
BEGIN
    --traitement utilisant la colonne 'NiveauPyramide'
    EXEC sp_executeSql N'UPDATE table2 SET col1= ... WHERE table1.NiveauPyramide'
 
    --suppression de la colonne
    ALTER TABLE schema1.table1 DROP COLUMN NiveauPyramide
END
GO