在java中尝试/ catch vs null检查

Question

有时我面对我必须写一段这样的代码(通常它有更多的嵌套if和更复杂的结构,但是对于这个例子来说)

public void printIt(Object1 a){
  if (a!=null){
     SubObject b= a.getB();
     if (b!=null){
         SubObject2 c=b.getC();
         if(c!=null){
             c.print();
         }
     }
  }
}

当我不需要知道什么是失败的,如果什么是空的什么都不做,一种方法可能是

public void printIt(Object1 a){
    try{
      a.getB().getC().print();
    }catch (NullPointerException e) {
    }
}

第二种形式如性能或其他类型的问题是否有问题？

谢谢

Answer 1

异常版本(类似于使用Groovy安全导航操作符的链?.)使得很容易采用Demeter法则(或者我称之为Demeter的强烈建议)并使其成为夜晚的玩物.

类似地,深度嵌套的if语句导致难以阅读的代码,并且在其下面,存在相同的"违规",并且这种方法的圈复杂度很高.

public void printIt(Object1 a) {
    if (null == a) {
        return;
    }

    SubObject b = a.getB();
    if (null == b) {
        return;
    }

    SubObject2 c = b.getC();
    if (null == c) {
        return;
    }

    c.print();
}

我宁愿在适当的地方看到LAM(小辅助方法)来封装检查,并且几乎完全消除了对问题的需求.

Answer 2

是.第二个版本将有糟糕的表现.

不要将异常用于正常控制流程.有效的Java项目57:仅在特殊情况下使用异常.

== ==更新

甚至忽略了性能问题(根据我的基准测试,异常比以前更快,但不像简单的if检查那么快),对于像这样的标准程序流使用异常真的是代码异常.JVM字节码具有特殊的优化功能,即使在if语句中也可以进行空检查.第一个代码示例是非常优选的.

Answer 3

第二个版本的最大部分是当NPE发生在内部时getB(),getC()它将被默默地忽略.如前所述,例外情况适用于特殊情况.

Answer 4

public void printIt(Object1 a){
    if(a==null){
        throw new IllegalArgumentException("a was null, but this is not allowed here."),
    }
    [...]

快速失败并努力失败.如果shoud不为null,则抛出异常.这将使您的代码更稳定可靠.

所以,如果我必须在你的a)和你的b)之间做出决定,我会选择a).但是如果a不能为空,那么你会隐藏错误情况.