Java 理论与实践: 平衡测试，第 2 部分

　 这一简短系列的 第 1 部分 介绍了如何进行有效测试，它构建了 FindBugs 插件来查找一个简单的 bug 模式（只需调用 System.gc() 即可）。Bug 模式会标识有问题的编码实践，它们经常位于 bug 所在的区域。当然，并不是所有出现 bug 模式的地方都一定出现 bug，但这并不能抹杀 bug 模式检测器的巨大作用。一个有效 bug 模式检测器的主要功能是发现更高百分比的可疑代码，使该模式具有更大的使用价值。创建 bug 模式检测器可以提高使用价值；创建检测器之后，无论是现在还是将来，您都可以在您需要的任何代码上运行它，并且您可能对发现的问题感到惊奇。例如，第 1 部分 中的简单检测器显示了对 System.gc() 的调用，在 JDK 1.4.2 中，它隐藏在 JPEG 图像 I/O 库中。

　  编写检测器可以查找对特定静态方法的调用，这并不困难，但是大多数的 bug 检测器都包含相当多的分析和实现。在这一期的文章中，您将开发一个称为 RuntimeException capture 的更小 bug 模式的检测器（目前，FindBugs 发行版中已包含此 bug 检测器。）

RuntimeException 捕捉

用 java™ 语言进行异常处理的一个优点是：异常是一些对象，try-catch 机制了解异常类型的分层结构，并在客户机如何处理错误处理方面提供实际灵活性。例如，假如不能找到文件，则 FileInputStream 构造函数会抛出 FileNotFoundException，该异常是 IOException 的一个子类。此传统用法答应客户机处理未发现文件的条件，这些条件是从其他与文件相关的条件中分离出来的（假如他们喜欢单独捕捉 FileNotFoundException）。但是，他们还可以使用捕捉 IOException 的方法处理所有与文件相关的错误条件。

　  另一方面，异常处理的主要缺陷是：在正确使用异常时，易于建立带有三行或四行业务逻辑以及 20 或 30 行异常处理的方法。因为错误恢复代码在测试时轻易出现错误并且执行困难，使一部分专门用于异常处理的代码无所适从并轻易出错。这种情况的典型示例如清单所示，其中带有两行 “真的” 代码的方法需要三个独立的捕捉块，每个捕捉块都执行完全相同的操作 —— 记录该异常：

public void addInstance(String className) {
    try {
        Class clazz = Class.forName(className);
        objectSet.add(clazz.newInstance());
    }
   catch (IllegalaccessException e) {
        logger.log("Exception in addInstance", e);
    }
    catch (InstantiationException e) {
        logger.log("Exception in addInstance", e);
    }
    catch (ClassNotFoundException e) {
        logger.log("Exception in addInstance", e);
    }
}
      

　  请参见清单 1，您可能尝试将三个捕捉块合并成捕捉 Exception 的单独捕捉块，因为每个捕捉块的捕捉恢复操作是相同的。乍一看，该策略似乎是一个好方法 —— 但代码副本有错误，所以整合这些复制路径应该是一种改进。不过，此 “改进” 经常会带来意想不到的结果。因为 RuntimeException 扩展了 Exception，将三个捕捉块合并成一个捕捉块（如清单 2 所示），所以这会更改语义，现在，未经检查的异常将被记录（而不传播）。此 bug 模式（其中 RuntimeException 轻易被超大捕捉块捕捉）也称为 RuntimeException 捕捉。

public void addInstance(String className) {
    try {
        Class clazz = Class.forName(className);
        objectSet.add(clazz.newInstance());
    }
    catch (Exception e) {
        logger.log("Exception in newInstance", e);
    }
}
      

　  bug 模式通常源自语言的模糊功能或类库；出现此 bug 模式是因为 RuntimeException 扩展了 Exception，这稍微有点违反常理。对 RuntimeException 捕捉的修复非常轻易 —— 您需要了解以下问题：首先捕捉 RuntimeException，并在捕捉 Exception 之前重新将其抛出，如清单 3 所示。不过，即使知道 bug 模式及其修复方法，在代码审查过程中也很轻易忘记执行它或忽略它，并且编译器也不会通知您。这是引入 bug 模式的原因，帮助您避免违犯 “您已较好地了解” 之类的错误。

public void addInstance(String className) {
    try {
        Class clazz = Class.forName(className);
        objectSet.add(clazz.newInstance());
    }
    catch (RuntimeException e) {
        throw e;
    }
    catch (Exception e) {
        logger.log("Exception in newInstance", e);
    }
}