Discussion :

[superbobo]

Bonsoir,
j'ai la procédure stockée suivante :

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
23
24
 
 
set ANSI_NULLS ON
set QUOTED_IDENTIFIER ON
GO
 
ALTER PROCEDURE CREATEINVIT
@email varchar(50),
@idstatut int,
@idcompte int
 
AS
BEGIN	
	SET NOCOUNT ON;   	
	if(select count(idcompte) from compte where email= @email) =0
	BEGIN
		declare @iduserExt int;
		insert into USEREXT(email)
		values(@email);
		select @iduserExt = @@identity;
		insert into temp(IDUSEREXT,IDSTATUT)
		values(@iduserExt,@idstatut);
	END
END

lorsque je fais select @iduserExt = @@identity;
Est ce que je suis absolument certain de récupérer l'identifiant de l'insertion juste avant, ou bien il est possible qu'entre l'insertion et le select, une autre insertion se soit faite entre temps, et que du coup je ne récupère pas le bon id ? J'ai vu qu'on pouvaut aussi utiliser @@SCOPEIDENTITY, mais je ne suis pas sur d'avoir bien compris.
Merci de vos eclaircissements :-)

[SQLpro]

@@identity est propre à la session en cours. Même di des millions d'utilisateurs font des insertions simultanées dans les mêmes tables, l'@@identity que vous aurez est relatif à votre dernière insertion. Cepandant si la table visée par l'insertion première comporte un trigger d'insertion qui insère dans une autre table, alors @@identity aura la valeur du dernier autoincrément de cette autre table. C'est pourquoi certains préfère SCOPE_IDENTITY qui lui porte non seulement sur la session mais la routine en cours.

Au passage votre procédure est très mal écrite, voici comment il faudrait l'écrire :

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
ALTER PROCEDURE CREATEINVIT
@email varchar(50),
@idstatut int,
@idcompte int
 
AS
BEGIN	
 
   SET NOCOUNT ON;   	
 
   INSERT INTO USEREXT(email)
   SELECT @email
   FROM compte 
   WHERE email= @email;
 
   IF @@ROWCOUNT > 0
      INSERT INTO temp(IDUSEREXT,IDSTATUT)
      VALUES(@@identity, @idstatut);
 
END

En effet entre la première requête qui compte si l'email existe, et la seconde qui fait l'insertion, il se peut qu'un autre utilisateur ait déjà compté et fasse cette même insertion avant la votre !

A +

[superbobo]

Merci pour votre aide SQLPro, je comprends mieux à présent l'utilité de @@scopeidentity.
Pour la procédure mal écrite, je m'en suis aperçu en revenant dessus, mais je ne suis pas revenu l'éditer sur le post car ce n'était pas l'objet de ma question
Merci bcp en tout cas.