Question

我目前正在使用Microsoft Enterprise Library 5.0，我想知道下面的代码是否是一种可接受的处理事务的方式。

我已经简化了一下场景，但实质上是我想在同一个事务中的不同数据库中执行多次插入。

如果其中一个插入失败，则应回滚所有事务。

我看过TransactionScope，但我想知道如果没有它我是否可以管理。

public void InsertStuff_AcrossDbs()
{
    //Create a ref to 2 different Db's on the same server
    Database db_a = DatabaseFactory.CreateDatabase("Data Source=localhost;Initial Catalog=db_a"); 
    Database db_b = DatabaseFactory.CreateDatabase("Data Source=localhost;Initial Catalog=db_b");

    //Create Connections for the 2 db's
    using (DbConnection connection_db_a = db_a.CreateConnection())
    using (DbConnection connection_db_b = db_b.CreateConnection())
    {
        //Create DbTransactions for the 2 db's
        DbTransaction transaction_dbA = connection_db_a.BeginTransaction();
        DbTransaction transaction_dbB = connection_db_b.BeginTransaction();

        try
        {
            //Insert into DbA.Person and get the PK
            DbCommand dbCmd_dbA_insert = db_a.GetSqlStringCommand("Insert INTO Person(Name,Age)Values('test',23); SET @pkReturnId= SCOPE_IDENTITY() ");
            db_a.AddOutParameter(dbCmd_dbA_insert, "pkReturnId", DbType.Int32, 0);
            db_a.ExecuteNonQuery(dbCmd_dbA_insert, transaction_dbA);
            int personID = Convert.ToInt32(db_a.GetParameterValue(dbCmd_dbA_insert, "@pkReturnId"));

            //Insert 'personId' into dbB.Employee  (a different table in a different db)
            DbCommand dbCmd_dbB_delete = db_a.GetSqlStringCommand("Insert INTO Employee(personId) VALUES(" + personID + ")");
            db_a.ExecuteNonQuery(dbCmd_dbB_delete, transaction_dbB);

            //try to commit both transactions
            transaction_dbA.Commit();
            transaction_dbB.Commit();
        }
        catch (Exception ex)
        {
            //If either transactions fails, roll back both
            try
            {
                transaction_dbA.Rollback();
            }
            catch { }

            try
            {
                transaction_dbB.Rollback();
            }
            catch { }

            throw ex;
        }
        finally
        {
            connection_db_a.Close();
            connection_db_b.Close();
        }

    }

}

这段代码是否可以正常使用，或者有些情况下，如果其中一个事件发生错误，将不会回滚所有事务？

Answer 1

您的代码不确定。有些情况下两个数据库不一致。

在您的代码中，您创建了两个本地事务：一个在数据库A中，一个在数据库B中。这不是您想要的，因为您希望在一个事务中发生这两个操作以保持一致性。为此，您需要使用分布式事务，如您所述，TransactionScope是执行此操作的最佳方式。它还将使代码更具可读性。

e.g。

Database db_a = DatabaseFactory.CreateDatabase("Data Source=localhost;Initial Catalog=db_a"); 
Database db_b = DatabaseFactory.CreateDatabase("Data Source=localhost;Initial Catalog=db_b");

using (TransactionScope scope = new TransactionScope())
{
    using (DbConnection connA = db_a.CreateConnection())
    {
        // ...
    }

    using (DbConnection connB = db_b.CreateConnection())
    {
        // ...
    }

    scope.Complete();
}

我意识到您的代码是一个简化的场景，所以我不确定这些是否适用于您的实际代码，但这里还有其他一些注释：

DbCommand dbCmd_dbB_delete = 
    db_a.GetSqlStringCommand("Insert INTO Employee(personId) VALUES(" + personID + ")");

应避免使用动态SQL字符串。它们容易受到SQL注入攻击，并且性能也较低，因为数据库执行计划不会被重用于具有不同参数的其他请求。

finally
{
    connection_db_a.Close();
    connection_db_b.Close();
}

您不需要手动关闭连接，因为您通过using语句处理连接，而Close和Dispose在功能上是等效的。

企业库：回滚多个事务

1 个答案: