c# - 异常处理中的异常处理

Question

在异常处理中异常处理的最佳实践是什么？

我发现自己在现有的 C#(Framework 4.0)系统上工作，该系统在 catch 中使用自定义对象，并最终阻塞整个系统的大部分应用服务器层。

考虑此代码库中方法的以下片段版本:

    public void DoSomeStuff(string sGUID)
    {
        try
        {
            // Foo
        }
        catch (Exception oEx)
        {
            oExceptions.Add(oEx);

            if (oDBConn.NumberOfActiveTrans > 0)
            {
                oDBConn.Rollback();
            }
        }
        finally
        {
            oDBConn.DeleteLocksByGUID(sGUID);
        }
    }

我可能过于偏执了，但我发现自己非常担心这些可能发生的未处理异常。

因此，像以下更新版本这样的做法是否可以接受，或者是否有更好的方法来完成同样的事情？

    public void DoSomeStuff(string sGUID)
    {
        try
        {
            // Foo
        }
        catch (Exception oEx)
        {
            oExceptions.Add(oEx);

            try
            {
                if (oDBConn.NumberOfActiveTrans > 0)
                {
                    oDBConn.Rollback();
                }
            }
            catch (Exception oEEx)
            {
                oExceptions.Add(oEEx);
            }
        }
        finally
        {
            try
            {
                oDBConn.DeleteLocksByGUID(sGUID);
            }
            catch (Exception oFEx)
            {
                oExceptions.Add(oFEx);
            }
        }
    }

Answer 1

我个人不会在finally里面加一个try catch block ，那它就可以是一个无穷无尽的链条了。通常你不应该在 finally 中有复杂的东西，并且在任何情况下都应该在调用者中捕获意外异常。

编辑:仔细观察代码，我不明白为什么 finally 中的代码不应该在 try block 中。