In the interests of completeness, there are three issues with your code sample.
1) You are executing your query twice by calling ExecuteNonQuery
and ExecuteScalar
. As a result, you will be inserting two records into your table each time this function runs. Your SQL, while being two distinct statements, will run together and therefore you only need the call to ExecuteScalar
.
2) Scope_Identity()
returns a decimal. You can either use Convert.ToInt32
on the result of your query, or you can cast the return value to decimal and then to int.
3) Be sure to wrap your connection and command objects in using
statements so they are properly disposed.
using (SqlConnection connection = new SqlConnection(connectionString))
{
using (SqlCommand command = new SqlCommand(sql, connection))
{
connection.Open();
int tenderId = (int)(decimal)command.ExecuteScalar();
}
}
与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…