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