Discussion :

[drinkmilk]

Bonjour,

Je souhaite récupérer la valeur de retour de la procédure stockée suivante (dont je ne peux pas modifier le code):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
create procedure sp_nextkey @table VARCHAR(30) as 
set nocount on 
DECLARE @tablename VARCHAR(30)
 
IF @table = 'SYSTEMKEY' BEGIN 
 INSERT INTO DOCSADM.MyTable (FIELD) VALUES (0) 
 DELETE FROM DOCSADM.MyTable WHERE LASTKEY = SCOPE_IDENTITY() 
 SELECT SCOPE_IDENTITY() 
END 
 
GO

Est-ce possible ? Si oui, comment ?

J'ai essayé la syntaxe suivante sans succès (au sein d'une autre procédure stockée):

Code :

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

1
2
Declare @LASTKEY integer
 @LASTKEY = exec DOCSADM.SP_NEXTKEY 'SYSTEMKEY'

[rudib]

Bonjour,

La syntaxe pour la valeur de retour est

Code :

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

1
2
3
 
Declare @LASTKEY integer
exec @LASTKEY = DOCSADM.SP_NEXTKEY 'SYSTEMKEY'

Par contre, tu ne récupéreras pas ton SCOPE_IDENTITY de cette façon, puisqu'il est renvoyé par un SELECT et non pas un RETURN.
Tu dois le récupérer dans un recordset. Si tu l'appelles depuis du SQL :

Code :

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

1
2
3
4
 
CREATE TABLE #tmp (id int)
INSERT INTO #tmp
EXEC DOCSADM.SP_NEXTKEY 'SYSTEMKEY'

Evite de préfixer tes procédures par sp_, pour des raisons de performance

[drinkmilk]

Bonjour,

merci pour cette réponse très complète !

J'ai également vu qu'il était possible dans ce cas d'utiliser la fonction IDENT_CURRENT('Nom_Table') qui permet de récupérer le dernier identifiant insérer dans une table:

Code :

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

SELECT @LASTKEY = IDENT_CURRENT('MyTable')

Pourquoi les procédures stockées préfixées par sp_ sont elles moins performantes ?

Merci encore !

[rudib]

IDENT_CURRENT() est moins approprié, parce qu'il te retournera une valeur même si elle a été insérée par une autre connexion.

Quand SQL Server voit une procédure préfixée par sp_, il la considère comme une procédure système, et va d'abord la chercher dans la base Master. S'il ne la trouve pas, il revient dans la base courante pour la trouver. Tu perds donc le temps de cet aller-retour.

[ZERS]

Par contre, tu ne récupéreras pas ton SCOPE_IDENTITY de cette façon, puisqu'il est renvoyé par un SELECT et non pas un RETURN.
Tu dois le récupérer dans un recordset. Si tu l'appelles depuis du SQL :

+1

Evite de préfixer tes procédures par sp_, pour des raisons de performance

Ah ces professionnels ! Sont pointilleux quand même

L'aller-retour master/base user est quand même pas ce qui consomme le plus de temps
Sauf bien sur si un vrai DBA a fait un vrai travail de placement des bases sur des partitions différentes sur serveurs différents, réseau pourri et des 10aine milliers de procédures dans master

Je taquine Rudi bien sur

[drinkmilk]

Ok, merci pour ces précisions.

Chez developpez, on leur demande ça, ils nous donnent CA !