TransactionScope失败仍完成，C#事务结构及FTP异常问题求助

让我来帮你拆解并解决这个问题——你的代码结构里有几个关键的逻辑漏洞，导致了TransactionScope看起来“失败后仍处于完成状态”，同时FTP上传的异常也没被正确处理，我们一步步来修正：

核心问题拆解 #

• 连接与事务的绑定逻辑错误：你把SqlConnection放在外部using块并提前打开，之后在内部循环创建TransactionScope。这种情况下，已打开的连接不会自动加入新的事务上下文，会导致事务状态混乱——看起来调用了tran.Complete()，但实际数据库操作根本没在事务管控范围内，或者事务的回滚逻辑失效。
• FTP操作无法纳入ACID事务：TransactionScope只能管控数据库这类支持事务的资源，FTP上传属于网络/文件系统操作，不支持事务回滚。如果FTP上传失败，数据库事务即使回滚，已经上传的文件也不会自动删除；反过来，数据库操作失败时，已完成的FTP上传也无法自动撤销。
• 异常处理的边界问题：虽然你用了try-catch，但事务的生命周期和异常捕获的范围没有匹配好，加上连接的问题，导致你误以为事务“完成了”，实际事务逻辑已经失效。

修复后的代码方案 #

foreach(var file in files) { 
    try { 
        // 每个文件对应一个独立的事务
        using(var tran = new TransactionScope()) { 
            // 将连接创建和打开放到事务内部，确保连接自动加入当前事务
            using(var con = new SqlConnection(CONNECTIONSTRING)) { 
                con.Open(); 
                using(var cmd = new SqlCommand(CMDTEXT1, con)) { 
                    // 添加参数
                    // ExecuteScalar 或 ExecuteNonQuery
                } 
                // 执行其他数据库操作...
            }

            // 先确保所有数据库操作成功，再执行FTP上传
            // 执行FTP上传操作
            UploadFileToFtp(file);

            // 只有所有操作都成功，才提交事务
            tran.Complete(); 
        } 
    } catch(Exception ex) { 
        // 记录异常日志
        LogException(ex);

        // 补偿操作：如果FTP已经上传成功，手动删除服务器上的文件
        if(FtpFileExists(file)) {
            DeleteFtpFile(file);
        }
    } 
}

关键优化点说明 #

1. 调整连接与事务的层级：把SqlConnection的创建和打开移入TransactionScope内部，这样连接会自动注册到当前事务上下文，确保数据库操作完全受事务管控。
2. 分离事务与非事务操作顺序：先执行所有数据库操作（受事务保护），确认无误后再执行FTP上传。如果数据库操作失败，事务会自动回滚，不会触发FTP；如果FTP上传失败，事务会因为没调用tran.Complete()而自动回滚，同时我们需要手动删除已上传的FTP文件（补偿操作）。
3. 明确事务回滚的触发条件：只要tran.Complete()没有被调用，TransactionScope的Dispose()方法就会自动回滚事务，无需手动处理，但要确保异常能触发这个逻辑。

额外注意事项 #

• 检查你的数据库连接字符串，确保没有设置Enlist=false（这个参数会阻止连接加入事务）。
• 对于FTP的补偿操作，要考虑网络异常场景（比如上传到一半失败），需要额外判断FTP服务器上的文件状态再执行删除。
• 如果需要所有文件的操作要么全部成功要么全部回滚，把TransactionScope放到foreach循环外面即可，但要注意事务时长，避免长时间占用数据库连接。

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