SCOPE_IDENTITY no funciona en asp.net?

Question

Estoy tratando de insertar un registro y obtener su ID recién generado mediante la ejecución de dos consultas una por una, pero no sé por qué me da el siguiente error.

Object cannot be cast from DBNull to other types 

Mi código es la siguiente: (No quiero utilizar procedimientos almacenados SQL)

SqlParameter sqlParam; 
    int lastInsertedVideoId = 0; 

    using (SqlConnection Conn = new SqlConnection(ObjUtils._ConnString)) 
    { 
     Conn.Open(); 
     using (SqlCommand sqlCmd = Conn.CreateCommand()) 
     { 
      string sqlInsertValues = "@Name,@Slug"; 
      string sqlColumnNames = "[Name],[Slug]"; 
      string sqlQuery = "INSERT INTO videos(" + sqlColumnNames + ") VALUES(" + sqlInsertValues + ");"; 
      sqlCmd.CommandText = sqlQuery; 
      sqlCmd.CommandType = CommandType.Text; 

      sqlParam = sqlCmd.Parameters.Add("@Name", SqlDbType.VarChar); 
      sqlParam.Value = txtName.Text.Trim(); 

      sqlParam = sqlCmd.Parameters.Add("@Slug", SqlDbType.VarChar); 
      sqlParam.Value = txtSlug.Text.Trim(); 


      sqlCmd.ExecuteNonQuery(); 

      //getting last inserted video id 
      sqlCmd.CommandText = "SELECT SCOPE_IDENTITY() AS [lastInsertedVideoId]"; 
      using (SqlDataReader sqlDr = sqlCmd.ExecuteReader()) 
      { 
       sqlDr.Read(); 
       lastInsertedVideoId = Convert.ToInt32(sqlDr["lastInsertedVideoId"]); 
      } 
     } 
    } 

    //tags insertion into tag table 
    if (txtTags.Text.Trim().Length > 0 && lastInsertedVideoId > 0) 
    { 
     string sqlBulkTagInsert = ""; 
     string[] tags = txtTags.Text.Split(new string[] { "," }, StringSplitOptions.RemoveEmptyEntries); 
     foreach (string tag in tags) 
     { 
      sqlBulkTagInsert += "INSERT INTO tags(VideoId, Tag) VALUES(" + lastInsertedVideoId + ", " + tag.Trim().ToLowerInvariant()+ "); "; 
     } 

     using (SqlConnection Conn = new SqlConnection(ObjUtils._ConnString)) 
     { 
      Conn.Open(); 
      using (SqlCommand sqlCmd = Conn.CreateCommand()) 
      { 
       string sqlQuery = sqlBulkTagInsert; 
       sqlCmd.CommandText = sqlQuery; 
       sqlCmd.CommandType = CommandType.Text; 

       sqlCmd.ExecuteNonQuery(); 
      } 
     } 
    } 

Y también, si es posible, por favor verifica el código anterior es codificada así o podemos optimizarlo más para mejorar el rendimiento?

Gracias

Answer 1

La llamada a SCOPE_IDENTITY() no se está tratando como si estuviera en el mismo "alcance" que el comando INSERT que está ejecutando.

En esencia, lo que hay que hacer es cambiar la línea:

string sqlQuery = "INSERT INTO videos(" + sqlColumnNames + ") VALUES(" + sqlInsertValues + ");"; 

a:

string sqlQuery = "INSERT INTO videos(" + sqlColumnNames + ") VALUES(" + sqlInsertValues + "); SELECT SCOPE_IDENTITY() AS [lastInsertedVideoId]"; 

y luego llamar

int lastVideoInsertedId = Convert.ToInt32(sqlCmd.ExecuteScalar()); 

en lugar de .ExecuteNonQuery y el bloque de código siguiendo el comentario "// obteniendo el último identificador de video insertado".

Answer 2

El SCOPE_IDENTITY() deben ser extraídos de la primera orden (SELECT, RETURN o OUT) y se pasaron en el siguiente comando. Con eso, quiero decir que el SELECT_IDENTITY() debe estar al final del primer comando . En SQL 2008 hay una sintaxis adicional para devolver los valores como parte del INSERT, lo que hace esto más simple.

O más eficientemente: combine los comandos en uno para evitar viajes de ida y vuelta.