在 C# 中回滚 linq2sql 插入事务的问题

Question

我正在尝试使用 linq2SQL 将 CSV 文件的内容插入到数据库表中。

我希望能够在任何插入失败时回滚事务，但是当我尝试使用此代码时，我会收到以下错误 - db.Transaction.Commit()

System.InvalidOperationException 未处理：此 SqlTransaction 已完成；它不再可用。

有人知道我做错了什么吗？

using (DataContext db = new DataContext())
{
    db.Connection.Open();
    db.Transaction = db.Connection.BeginTransaction();

    try
    {
        foreach (string entry in entries)
        {
            XXX xxx = new XXX()
            {
                P1 = "something",
                P2 = "something"
            };

            db.XXXX.InsertOnSubmit(xxx);
            db.SubmitChanges();
        }
    }
    catch (Exception)
    {
        db.Transaction.Rollback();
    }
    finally
    {
        db.Connection.Close();
    }

    db.Transaction.Commit();
}

Answer 1

好吧，顺序是错误的 - 您在整个大块之后调用 db.Transaction.Commit() ，因此即使发生异常并且您已经调用 db.Transaction.Commit() ，它也会被调用。 Transaction.Rollback();

将代码更改为：

using (DataContext db = new DataContext())
{
    db.Connection.Open();
    db.Transaction = db.Connection.BeginTransaction();

    try
    {
        foreach (string entry in entries)
        {
            ....
            db.XXXX.InsertOnSubmit(xxx);
            db.SubmitChanges();
        }

        db.Transaction.Commit(); <== CALL HERE !!
    }
    catch (Exception)
    {
        db.Transaction.Rollback();
    }
    finally
    {
        db.Connection.Close();
    }
}

在这种情况下，您的 Commit 在 foreach 之后调用，但如果遇到异常并执行回滚，则不会调用它。

马克

Answer 2

是因为你在回滚之后才进行提交吗？

您应该将提交放在 try 块的最后，以便调用回滚或提交。永远不会两者...

更新：正如彼得在他的回答中提到的那样，我希望 Close 或 Rollback 语句都不是必需的，因为 using 块将处理（因此也关闭）连接，并且未提交的事务应该自动被回滚。

Answer 3

基于“使用 datacontext”将确保当前事务和连接将被关闭的事实，我将假设以下块应该足够了：

01.    using (DataContext db = new DataContext())
02.    {    
03.        db.Connection.Open();    
04.        db.Transaction = db.Connection.BeginTransaction();    
05.
06.        foreach (string entry in entries)        
07.        {                
08.            XXX xxx = new XXX()                
09.            {                        
10.                P1 = "something",                        
11.                P2 = "something"                
12.            };                
13.            db.XXXX.InsertOnSubmit(xxx);                
14.        }    
15.        db.SubmitChanges();        
16.
17.        db.Transaction.Commit();
18.    }

如果在第 05 行和第 16 行之间发生异常，则事务将永远不会被标记为 Commit 和因此，一旦事务和连接在第 18 行完成，就会回滚。

注意：这里的行为有所不同，我不确定是否是有意为之：除了回滚事务之外，您的 catch 块还会吞掉异常从而隐藏了发生错误的事实。

更新：我还将 SubmitChanges 调用移出内部循环。您应该能够首先进行插入，然后针对所有更改提交一次更改。

Answer 4

关于“Peter Lillevold”的答案中发布的代码：
我可以看到，如果第 15 行发生错误，例如 db.SubmitChanges()，它不会关闭数据库连接。因此，正确的解决方案是：

enter code here
      using (DataContext db = new DataContext())
        {
            // The dispose method of DbConnection will close any open connection
            // and will rollback any uncommitted transactions
            using (DbConnection dbConnection = db.Connection)
            {
                dbConnection.Open();
                db.Transaction = dbConnection.BeginTransaction();
                foreach (string entry in entries)
                {
                    XXX xxx = new XXX()
                    {
                        P1 = "something",
                        P2 = "something"
                    };
                    db.XXXX.InsertOnSubmit(xxx);
                }
                db.SubmitChanges();
                db.Transaction.Commit();
            }
        } 

PS: 如需进一步说明，请参考 http: //msdn.microsoft.com/en-us/library/bb292288.aspx，其中显示“如果您提供打开的连接，DataContext 将不会关闭它。”