SqlTransaction 导致应用程序因错误而崩溃？

Question

以下是一些背景信息： 我们的网站会定期崩溃，以至于必须重新启动 IIS；这几乎总是在修补 DLL 后的一个小时内发生（我们使用网站项目，而不是 Web 应用程序项目，因此每个 ASPX 页面都是一个单独的 DLL）。

在做一些研究时，我发现我们的自制 DAL 可以在调试时导致 Visual Studio 的内置 Web 服务器实际上停止工作并在存储过程中遇到 SQL 错误时关闭（我的意思是它不仅会抛出一个在浏览器中显示的异常，它实际上会说 Web 服务器遇到了错误并且需要关闭！）

在进一步挖掘中，该错误似乎与对所有内容（包括 Select 语句）使用事务有关达尔。似乎发生的情况是这样的：

1. 尝试执行存储过程，存储过程由于缺少/无效列或其他错误而失败。
2. 应用程序代码捕获错误并重新抛出它（不好，是的，但我没有写这个）。
3. 尽管发生异常，事务仍尝试提交，在 transaction.Commit() 行上获取 NullReferenceException（似乎位于 Connection 属性上，因为有是交易对象）。另外，这个 NullRef 似乎无法被捕获（我尝试了一个演示，该演示因无效的 Sproc 而强制崩溃，并且 NullRef 从未被捕获，即使输出错误的类型为 System.NullReferenceException ）
4. 事务抛出错误上面写着“交易已完成且不再可用”。
5. ???但是 VS Web 服务器崩溃了。调试这部分似乎挂在上面的异常上，永远不会离开该方法。

现在，我不知道这是否是导致 IIS 崩溃的原因，但它看起来很可疑，而且无论如何这都是一个明显的错误。

之前没有处理过事务并且只了解它们的基本概念，我的第一个问题是为什么事务在抛出异常后仍然尝试提交？我的第二个问题是如何修复失败的提交以及可能无限循环的异常，直到服务器崩溃。添加这样的内容（该方法采用名为 transaction 的 SqlTransaction 参数）是否有意义：

catch (SqlException se) 
{
    if (transaction != null)
    {
        transaction.Rollback();
    }
    throw;
}

这个小更改是否可以修复我认为导致 IIS 崩溃的持续异常循环？ DAL 本身非常脆弱，并且在数百个文件中具体使用，因此我无法正确地从头开始重写它。

编辑整个代码块是这样的（同样，遗留代码 - 使用旧的微软数据访问块助手）：

    public static DataSet ExecuteDatasetStoredProc(SqlConnection conn, String storedProcName, SqlTransaction transaction, params SqlParameter[] storedProcParms)
    {
        try
        {
            // Execute the stored proc
            if (transaction != null)
            {
                return SqlHelper.ExecuteDataset(transaction, CommandType.StoredProcedure, storedProcName, storedProcParms);
            }
            else
            {
                return SqlHelper.ExecuteDataset(conn, CommandType.StoredProcedure, storedProcName, storedProcParms);
            }
        }
        catch (SqlException se)
        {
            throw new ApplicationException("Error calling " + storedProcName + ". " + se.Message, se);
        }
    }

但是，如果catch块执行事务仍然尝试提交，这似乎会导致挂断。

Answer 1

如果您将事务代码包装在 try catch 中，也请更改您的

Try
{
// your code that you assign and execute the SQl

}
catch (SQLException sqlex)
{
  try 
  {
    //try to do the rollback here.. don't always assume the commit or rollback will work
  }
  catch (Your SQL Exception ex)
  {
  }
}