SCOPE_IDENTITY 在 asp.net 中不起作用？

Question

我正在尝试插入一条记录并通过一个一个执行两个查询来获取其新生成的ID，但不知道为什么它给我以下错误。

Object cannot be cast from DBNull to other types

我的代码如下：（我不想使用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();
            }
        }
    }

如果可能，请检查上述代码是否编码良好，或者我们可以对其进行更多优化以提高性能？

谢谢

Answer 1

对 SCOPE_IDENTITY() 的调用不会被视为与您正在执行的 INSERT 命令在同一“范围”内。

基本上，你需要做的就是换行：

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

到：

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

然后调用

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

而不是 .ExecuteNonQuery 和“//获取最后插入的视频 ID”注释之后的代码块。

Answer 2

SCOPE_IDENTITY() 应从第一个命令（SELECT、RETURN 或 OUT）中提取并传递到下一个命令。我的意思是 SELECT_IDENTITY() 应该在 first 命令的末尾。在 SQL 2008 中，作为INSERT 的一部分，有额外的语法可以将值带回，这使得这更简单。

或者更有效：将命令组合成一个以避免往返。