SCOPE_IDENTITY non funziona in asp.net?

Question

Sto cercando di inserire un record e ottenere il suo id appena generato eseguendo due query una alla volta, ma non so perché mi sta dando il seguente errore.

Object cannot be cast from DBNull to other types 

Il mio codice è il seguente: (Non voglio usare stored procedure di 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(); 
      } 
     } 
    } 

E anche, se possibile, si prega di controllare è il codice di cui sopra codificato bene o siamo in grado di ottimizzare più per migliorare le prestazioni?

Grazie

Answer 1

La chiamata a SCOPE_IDENTITY() non viene considerata come nello stesso "ambito" del comando INSERT che si sta eseguendo.

In sostanza, ciò che devi fare è cambiare la linea:

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

a:

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

e quindi chiamare

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

invece di .ExecuteNonQuery e il blocco di codice seguendo il commento "// ottenendo l'ultimo ID video inserito".

Answer 2

Il SCOPE_IDENTITY() devono essere estratti dal primo comando (SELECT, RETURN o OUT) e passati nel comando successivo. Con ciò intendo che lo SELECT_IDENTITY() dovrebbe trovarsi alla fine del comando primo. In SQL 2008 è disponibile una sintassi aggiuntiva per riportare i valori come parte dello INSERT, il che rende questo più semplice.

O in modo più efficiente: combinare i comandi in uno per evitare round-trip.