Discussion :

[Papy214]

Bonjour,

J'utilise une procédure stockée pour insérer une ligne dans une table et renvoyer l'id de la nouvelle ligne. Après mon "INSERT INTO" je fais un

Code :

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

SELECT @id = IDENT_CURRENT('fichiers')

Mais ça ne fonctionne pas. Je dois rajouter une ligne:

Code :

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

1
2
3
 
	SELECT @id = IDENT_CURRENT('fichiers')
	SELECT @id

Est-ce normal ?

Papy

[zinzineti]

avez-vous défini le paramètre @id comme paramètre de sorti (OUTPUT) ?

A+

[Papy214]

En fait, ma procédure (simplifiée au niveau de la requête) est structurée comme suit:

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
 
CREATE PROCEDURE [dbo].[AjouteFichier] 
	@champ1 varchar(255), @champ2 varchar(255)
AS
BEGIN
	DECLARE @id integer	
	 INSERT
	 INTO   fichiers
			(
				   champ1,
				   champ2
			)
			VALUES
			(
				   @champ1,
				   @champ2
			)
	SELECT @id = IDENT_CURRENT('fichiers')
	SELECT @id 
END

Et dans mon code pascal, je récupère le champ dans le recordset obtenu.

J'ai peut-être fait tout de travers ? :-(

[elsuket]

Bonjour,

C'est plutôt pas mal, mais il y a quelques petites erreurs : n'utilisez pas IDENT_CURRENT() pour récupérer l'identifiant d'une ligne que vous venez d'insérer.
Vous préférerez SCOPE_IDENTITY() qui retourne l'identifiant de la ligne insérée par la transaction juste avant son appel.
Si vous utilisez IDENT_CURRENT() et que de nombreuses lignes sont insérées en même temps, il est possible que vous n'obteniez pas l'identifiant qui correspond à la ligne que vous venez d'insérer.
En effet cette fonction n'a pas de limite de portée, tandis que SCOPE_IDENTITY() est limitée à la portée du module dans laquelle elle est appelée.

D'autre part, veillez à toujours qualifier le nom des objets que vous manipulez par le nom du schéma auquel ils appartiennent : ici c'est dbo, le schéma par défaut.

Enfin vous pouvez utiliser la clause OUTPUT pour vous retourner la valeur de l'identifiant, ce qui donne :

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
CREATE PROCEDURE [dbo].[AjouteFichier] 
	@champ1 varchar(255)
	, @champ2 varchar(255)
	, @id int = NULL OUTPUT
AS
BEGIN
	SET NOCOUNT ON
 
	INSERT	INTO dbo.fichiers
	(
		champ1
		, champ2
	)
	VALUES
	(
		@champ1
		, @champ2
	)
 
	SELECT @id = SCOPE_IDENTITY()
END
GO

Et à l'utilisation :

Code :

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

1
2
3
DECLARE @id_fichier int
EXEC dbo.AjouteFichier 'a', 'b', @id_fichier OUTPUT
SELECT @id_fichier

Je ne sais pas si Pascal gère ce type de paramètres.
Si ce n'est pas le cas, vous pouvez conserver le code que vous avez écrit, en le modifiant avec les recommandations que je vous ai faites plus haut

@++

[Papy214]

Merci pour cette aide précise et claire.

Dans mon code Delphi, j'ai pu récupérer la valeur renvoyée dans un paramètre défini comme "retour de valeur" et ça fonctionne très bien.

J'ai mis votre blog dans mes favoris, même si pour l'instant je n'ai pas vraiment le niveau requis pour tout comprendre.

[elsuket]

Avec plaisir !

Si vous ne comprenez pas, demandez sur le forum !

@++