utilisation SET SCOPE_IDENTITY()

Version imprimable

Bonjour,

Je n'ai pas encore beaucoup d'expérience en SQL Server

J'analyse la syntaxe d'un SET SCOPE_IDENTITY() pour recuperer un ID autoincrement

Dans la commande ci dessous, le POINT VIRGULE qui separe l'Insert du SET @returnid = SCOPE_IDENTITY();";

Est-il nécessaire, supperflu ou incorrect !

D'une maniere plus genérale, cette syntaxe est elle bonne ?

Merci de votre aide.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
        command.CommandText = @"INSERT INTO Alarms VALUES (" 
                              + "@DeviceBearerId, "
                              + "@time,"
                              + "@position); " 
                              + " SET @returnid = SCOPE_IDENTITY();";
        command.Parameters.Add(new SqlParameter("@DeviceBearerId", memopasAlarm.DeviceBearerId));
        command.Parameters.Add(new SqlParameter("@time", memopasAlarm.Time));
        command.Parameters.Add(new SqlParameter("@position", memopasAlarm.Position));
 
 
        SqlParameter returnId = new SqlParameter("@returnid", SqlDbType.Int);
        returnId.Direction = ParameterDirection.Output;
        command.Parameters.Add(returnId);
        command.ExecuteNonQuery();

Vous passez deux ordres SQL en un seul, ceci n'est pas correct. Vous devez d'abord lancer la commande suivante :

Code:

1
2
3
4
5
6
7
8
command.CommandText = @"INSERT INTO Alarms VALUES (" 
                              + "@DeviceBearerId, "
                              + "@time,"
                              + "@position)";
        command.Parameters.ADD(new SqlParameter("@DeviceBearerId", memopasAlarm.DeviceBearerId));
        command.Parameters.ADD(new SqlParameter("@time", memopasAlarm.Time));
        command.Parameters.ADD(new SqlParameter("@position", memopasAlarm.Position));
        command.ExecuteQuery();

suivie de :

Code:

1
2
3
 
command.CommandText = @"SELECT SCOPE_IDENTITY() AS NEW_ID;";
        command.OpenQuery();

Mais comme cela fait près de 10 ans que je n'ai pas fait de langage itératif, il y a sans doute des fautes, d'autant que vous ne précisez pas le langage utilisé !

A +

25/11/2008, 17h16
olibara

Merci beaucoup pour votre réponse

Mais j'ai quand meme une objection !

Il me semble tout a fait correct de passer plusieurs commandes sql en une seule chaine tout en separant les commandes par un POINT VIRGULE

C'est en tout cas ce que je fais en Mysql et en SyBase

Ce qui m'avait interpellé dans la syntaxe ci-dessus c'est le SET sans nouveau SELECT
et l'absence de Point virgule (que j'avais ajouté moi meme) avant le SET

Je suis en SQL SErver 2008
25/11/2008, 17h23
SQLpro

Tout dépend de l'objet que vous avez utilisé. L'objet commandtext le permet, mais il n'est pas bon de l'utiliser dans ce sens pour des raisons de performances. Il doit en principe être réservé à des batch pas à des traitements fonctionnels. Utilisez de préférence des objets TQuery ou nmieux, des procédures stockées qui incorporent toute la logique

A +

Merci

En fait je reprends une application fonctionelle mais qui presente justement un probleme dans cette commande
Le problème etait avant tout l'assignation d'un parametre mais au passage j'essaye d'analyser le reste

Donc dans ce contexte, quelle est la meilleure syntaxe :

Avec ;

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
        command.CommandText = @"INSERT INTO Alarms VALUES (" 
                              + "@DeviceBearerId, "
                              + "@time,"
                              + "@position); " 
                              + " SET @returnid = SCOPE_IDENTITY();";
        command.Parameters.Add(new SqlParameter("@DeviceBearerId", memopasAlarm.DeviceBearerId));
        command.Parameters.Add(new SqlParameter("@time", memopasAlarm.Time));
        command.Parameters.Add(new SqlParameter("@position", memopasAlarm.Position));
 
 
        SqlParameter returnId = new SqlParameter("@returnid", SqlDbType.Int);
        returnId.Direction = ParameterDirection.Output;
        command.Parameters.Add(returnId);
        command.ExecuteNonQuery();

Ou sans

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
       command.CommandText = @"INSERT INTO Alarms VALUES (" 
                              + "@DeviceBearerId, "
                              + "@time,"
                              + "@position) " 
                              + " SET @returnid = SCOPE_IDENTITY();";
        command.Parameters.Add(new SqlParameter("@DeviceBearerId", memopasAlarm.DeviceBearerId));
        command.Parameters.Add(new SqlParameter("@time", memopasAlarm.Time));
        command.Parameters.Add(new SqlParameter("@position", memopasAlarm.Position));
 
 
        SqlParameter returnId = new SqlParameter("@returnid", SqlDbType.Int);
        returnId.Direction = ParameterDirection.Output;
        command.Parameters.Add(returnId);
        command.ExecuteNonQuery();

25/11/2008, 19h35
elsuket

Bonjour,

Rien ne vous oblige à mettre des point-virgules dans un instruction T-SQL, mais je crois que cela va devenir un standard.

@++ ;)