【转】编写高质量代码改善C#程序的157个建议——建议70：避免在调用栈较低的位置记录异常

 

建议70：避免在调用栈较低的位置记录异常

并不是所有的异常都要被记录到日志，一类情况是异常发生的场景需要被记录，还有一类就是未被捕获的异常。未被捕获的异常通常被视为一个Bug，所以，对于它的记录，应该被视为系统的一个重要组成部分。

最适合记录异常和报告的是应用程序的最上层，这通常是UI层。假设存在这样一个应用程序，它的BLL层可能被一个WinForm窗体调用，也可能被一个控制台应用程序调用，那么要在BLL模块向管理员报告异常的时候，你可能会不知道该使用MessageBox方法还是使用Console.Write方法。

 

如果异常在调用栈较低的位置被记录或报告，且还存在被包装后抛出的情况：

        internal void MethodLow()
        {
            try
            {
                //...
            }
            catch (SomeException)
            {
                Logger.Write("异常SomeException发生")
                throw;
            }
        }

然后在调用栈较高位置也捕获记录异常：

        internal void MethodHigh()
        {
            try
            {
                MethodLow();
            }
            catch (SomeException)
            {
                Logger.Write("异常SomeException发生")
            }
        }

这就会让记录重复出现。在调用栈较低的情况下，往往异常被捕获了也不能被完整的处理。所以，综合考虑，应用程序在设计初期，就应该为开发成员约定在何处记录和报告异常。

 

 

转自：《编写高质量代码改善C#程序的157个建议》陆敏技