异常处理的反模式

2018-07-07 本文已影响0人码代码的陈同学

欢迎访问陈同学博客原文
 Exception-Handling Antipatterns Blog from Oracle Community
Exception management antipatterns from Java Peanuts
《反模式－危机中软件、架构和项目的重构》by 宋锐等译

异常设计实践中有位读者评论：又记录日志又抛异常反模式设计。其实我并不知道反模式，本文是对于反模式的学习整理，数据都来自参考资料。

先岔开主题，BB两个小点：

读者的一个评论，有时却能让笔者受益良多，这也是交流带来的好处，打破认知的局限。
对于知识的学习经常是写小学作文的 总－分－总 模式。先知晓概念，然后碎片学习，最后整体总结，通过一个个小知识点的积累，最终不断丰富与深化自己知识框架。

关于反模式

设计模式的风靡，反映了软件从业人员对改善行业质量与标准的强烈愿望，因使用和创建可复用的设计模式而获得成功的项目不断增长，设计模式体验了巨大的价值。

设计模式通俗来说就是套路，来源于实践，反哺给实践。但实践中，也常因将设计模式应用于不适当的上下文环境而导致许多问题。

反模式 利用实践经验来定义经常发生的错误，让你留意开发过程中潜在的各种陷阱与风险。反模式描述了这些易错场景的基本形式、可能带来的负面影响，提供了补救措施，并给这些场景定义了名称。

更多关于反模式的知识，可以阅读《反模式－危机中软件、架构和项目的重构》。

异常处理反模式

此处仅挑选几个，更多请参考 Exception-Handling Antipatterns Blog

Log and Throw

下面是三个讨厌的反模式，既打印日志又抛出异常，这对于运维工程师来说简直就是噩梦，因为一个问题产生了多种log信息。

例1:

catch (NoSuchMethodException e) { LOG.error("Blah", e); throw e; }

例2:

catch (NoSuchMethodException e) { 
    LOG.error("Blah", e); throw new MyServiceException("Blah", e); 
}

例3:

catch (NoSuchMethodException e) { 
    e.printStackTrace(); throw new MyServiceException("Blah", e); 
}

Throwing Exception

public void foo() throws Exception {}

直接抛出 异常基类 完全打破了 checked exeption 的目的，这样做就是在向你的调用者宣布：我可能会干点坏事，你悠着点。

Catching Exception

try { foo(); } catch (Exception e) { LOG.error("Foo failed", e); }

这是"私吞"异常的例子，也是异常处理中不可饶恕的恶行，异常从此凭空消失，只有这条淹没在日志汪洋中的傻乎乎的日志是唯一线索。

Destructive Wrapping

catch (NoSuchMethodException e) { throw new MyServiceException("Blah: " + e.getMessage()); }

这种破坏性包装导致原始异常的stack trace丢失，出现异常时大家一起来抓瞎，再抓个实习生顶锅。

Return null

下面三个例子都返回null值，除非特殊的业务场景需要返回null值，否则最好抛出异常，让调用者进行处理。

catch (NoSuchMethodException e) { LOG.error("Blah", e); return null; }
catch (NoSuchMethodException e) { e.printStackTrace(); return null; } 
catch (NoSuchMethodException e) { return null; }

Throw from Within Finally

try { blah(); } finally { cleanUp(); }

如果 cleanUp() 永远不会抛异常倒还好，因为如果 blah() 出错，将进入 finnaly，可一旦 cleanUp() 出错，将丢失 blah() 中抛出的异常，同时需要处理 finally 中的异常。

Relying on getCause()

catch (MyException e) { if (e.getCause() instanceof FooException) {}

依赖 getCause() 的代码非常脆弱，这种方式不一定能捕获到真正的异常。推荐使用 Apache的 commons-lang 中的 ExceptionUtils.getRootCause() 来代替。