这一简短系列的 第 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); }}
新闻热点
疑难解答