Discussion :

[siro1]

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 :

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
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 :

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

[FMJ]

Salut

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

Code :

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

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 :

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

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.

[elsuket]

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 :

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
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 :

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

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 :

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

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 :

IDVille nomVille
1 LYON

Si par la suite nous exécutons :

Code :

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

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 :

IDVille nomVille
1 LYON
2 TOULOUSE

@++

[siro1]

Salut

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

Code :

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

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 :

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

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 .

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

tout simplement

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