Recuperer le resultat d'une requete dans une variable

Bonjour,

J'essai dans la requete ci-dessous de recuperer la valeur maximum d'un identifiant (idVille) afin de pouvoir inserer une nouvelle ville avec un ID incrémenté de +1.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

BEGIN TRANSACTION insertion_ville DECLARE @id INT DECLARE @error INT DECLARE @libelleVille VARCHAR(50)   SET @error = 0 SET @libelleVille = 'Lyon'   SELECT INTO @id MAX(idVille) FROM dbo.Ville --le pb est dans cette ligne   IF @id = NULL SET @id = 0 ELSE SET @id = @id +1   INSERT INTO dbo.Ville VALUES (@id,@libelleVille) SET @error = @error + @@error   IF @id = NULL OR @error <> 0 ROLLBACK TRANSACTION insertion_ville ELSE COMMIT TRANSACTION insertion_ville

---

Et j'obtient l'erreur suivante :

Citation:

---

Msg*156, Niveau*15, État*1, Ligne*7
Syntaxe incorrecte vers le mot clé 'INTO'.

---

Ou est mon erreur ? La syntaxe est-elle bonne ?

Merci

Salut

Alors si je ne gourre pas, la syntaxe pour affecter la valeur d'une requête à une variable est la suivante :

Code:

---

SELECT @id=MAX(idVille) FROM dbo.Ville

---

Donc sans INTO et avec un "="

De plus, pourquoi tu ne fais pas simplement cette requête :

Code:

---

1 2	INSERT INTO dbo.Ville SELECT TOP 1 ISNULL((SELECT MAX(idVille) FROM dbo.Ville),0) + 1 AS ID, @libelleVille FROM dbo.Ville

---

Comme ça, pas besoin de variable.

Bonjour,

Si vous êtes sous SQL SErver 2005 ou 2008, vous pouvez gérer le cas d'exception à l'aide du contrôle TRY ... CATCH

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

BEGIN TRY BEGIN TRANSACTION insertion_ville   INSERT INTO dbo.Ville ( colonnes ) SELECT CASE ISNULL(MAX(idVille), 0) WHEN 0 THEN 0 ELSE ISNULL(MAX(idVille), 0) + 1 END, 'Lyon' FROM dbo.Ville --le pb est dans cette ligne END TRY BEGIN CATCH ROLLBACK TRANSACTION RAISERROR('une erreur', 16, 1) END CATCH

---

A quoi sert de mettre TOP 1 puisque vous agrégez toute la table ?
MAX() ne vous retournera forcément qu'une valeur ;)

A quoi sert également de mettre tout cela dans une transaction ?
Il y a lieu d'utiliser une déclaration explicite de transaction lorsque qu'on souhaite voir s'exécuter au moins deux instructions de modifications de données (INSERT, UPDATE ou DELETE) comme une seule.
Donc pas dans votre cas, où vous ne faites qu'un seul INSERT ;)
Les instructions de modification de données sous SQL Server sont par défaut automatiquement transactionnées ;)

Il aurait tout de même été bien plus simple de créer votre table dbo.Ville avec une valeur de clé primaire auto-incrémentée :

Code:

---

1 2 3 4 5

CREATE TABLE dbo.TbVille ( IDVille SMALLINT IDENTITY NOT NULL CONSTRAINT PK_TbVille PRIMARY KEY, nomVille VARCHAR(32) NOT NULL CONSTRAINT UQ_TbVille_nomVille UNIQUE )

---

Vous n'avez ensuite plus qu'à écrire :

Code:

---

1 2 3 4 5 6

BEGIN TRY INSERT INTO dbo.TbVille (nomVille) VALUES ('LYON') END TRY BEGIN CATCH RAISERROR('une erreur', 16, 1) END CATCH

---

Et en interrogeant le table TbVilles (SELECT * FROM dbo.TbVilles), vous auriez :

Citation:

---

IDVille nomVille
1 LYON

---

Si par la suite nous exécutons :

Code:

---

1 2 3 4 5 6

BEGIN TRY INSERT INTO dbo.TbVille (nomVille) VALUES ('TOULOUSE') END TRY BEGIN CATCH RAISERROR('une erreur', 16, 1) END CATCH

---

Et que nous réinterrogeons la table TbVilles :

Citation:

---

IDVille nomVille
1 LYON
2 TOULOUSE

---

@++ ;)

:ccool:

Citation:

---

Envoyé par FMJ

Salut

Alors si je ne gourre pas, la syntaxe pour affecter la valeur d'une requête à une variable est la suivante :

Code:

---

SELECT @id=MAX(idVille) FROM dbo.Ville

---

Donc sans INTO et avec un "="

De plus, pourquoi tu ne fais pas simplement cette requête :

Code:

---

1 2	INSERT INTO dbo.Ville SELECT TOP 1 ISNULL((SELECT MAX(idVille) FROM dbo.Ville),0) + 1 AS ID, @libelleVille FROM dbo.Ville

---

Comme ça, pas besoin de variable.

---

Merci pour ton poste FMJ ! ta solution est correcte . :mouarf:

sinon j'ai trouver aussi ==> SET @id = (SELECT MAX(idVille) FROM dbo.Ville)

tout simplement :D

elsuket, Merci a toit aussi pour ces précisions . Je vais faire plus ample recherche sur l'autoincrément