Entity Framework Core事务异常未始终回滚问题排查

你提到已经有catch块处理回滚，但安全扫描仍报问题，大概率是以下几个容易忽略的场景没覆盖到：

1. Catch块未覆盖所有异常场景 #

如果你的catch只捕获了特定类型的异常（比如DbUpdateException），而非通用的异常兜底逻辑，那么像线程中断、内存不足这类未被捕获的异常触发时，事务就会处于未回滚状态。另外，有些CLR级别的异常（比如StackOverflowException）无法被catch，这时候必须靠finally块做兜底。

2. 事务/连接的初始化未包裹在try块中 #

整改要求里提到“用try块打开连接、开启事务”，如果你的BeginTransactionAsync调用在try块外面，比如：

// 错误示例：开启事务在try外
var transaction = await _context.Database.BeginTransactionAsync();
try
{
    // 操作数据库
    await transaction.CommitAsync();
}
catch
{
    await transaction.RollbackAsync();
}

一旦BeginTransactionAsync本身抛出异常（比如连接断开），transaction对象可能为null或处于无效状态，后续的catch块根本无法执行回滚，直接导致事务残留。

3. 缺少finally块的兜底回滚 #

即使你有catch块，也必须在finally里检查事务状态并回滚——因为catch可能漏过某些异常，或者代码在catch块执行前就崩溃。正确的做法是：

public async Task CommitTransactionAsync()
{
    IDbContextTransaction transaction = null;
    try
    {
        // 打开连接+开启事务都放在try里
        await _context.Database.OpenConnectionAsync();
        transaction = await _context.Database.BeginTransactionAsync();
        
        // 执行业务操作
        await _context.SaveChangesAsync();
        
        await transaction.CommitAsync();
    }
    catch
    {
        // catch里先回滚活跃事务
        if (transaction?.State == Microsoft.EntityFrameworkCore.Storage.TransactionState.Active)
        {
            await transaction.RollbackAsync();
        }
        throw; // 不要吞异常，向上抛出便于上层处理
    }
    finally
    {
        // finally兜底：确保未提交的事务一定回滚
        if (transaction?.State == Microsoft.EntityFrameworkCore.Storage.TransactionState.Active)
        {
            await transaction.RollbackAsync();
        }
        // 关闭连接，确保无未完成事务残留
        if (_context.Database.GetDbConnection().State == System.Data.ConnectionState.Open)
        {
            await _context.Database.CloseConnectionAsync();
        }
        transaction?.Dispose();
    }
}

4. 关闭连接前未检查事务状态 #

整改要求“关闭数据库前无未完成事务”，也就是说，不管操作成功还是失败，关闭连接前必须确保事务要么提交成功，要么已经回滚。如果你的代码在关闭连接前没有检查事务状态，就可能出现连接关闭但事务仍处于活跃状态的情况，数据库会一直持有锁资源。

5. Commit操作的异常未处理 #

CommitAsync本身也可能抛出异常（比如数据库宕机），这时候事务还是处于活跃状态，必须回滚。如果你的代码在Commit后没有处理异常的逻辑，就会遗漏这种场景。

内容的提问来源于stack exchange，提问作者One Developer