Discussion :

[olibara]

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 :

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

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();

[SQLpro]

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

Code :

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

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 :

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

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 +

[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

[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 +

[olibara]

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 :

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

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 :

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

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();

[elsuket]

Bonjour,

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

@++