java嵌套如果或单个如果

内容来源于 Stack Overflow,并遵循CC BY-SA 3.0许可协议进行翻译与使用

  • 回答 (2)
  • 关注 (0)
  • 查看 (17)

我对执行下面的代码块(示例)有一个基本的疑问:

String version = computer.getSoundcard().getUSB().getVersion();

NullPointerException如果Soundcard不存在,可能会抛出什么。

所以我有 ,

选项1 :

if(computer!=null && 
        computer.getSoundCard() !=null && 
                 computer.getSoundCard().getUSB()!=null) {
   version = computer.getSoundcard().getUSB().getVersion();
}

选项2:

if(computer !=null){
   SoundCard sc = computer.getSoundCard();
   if(sc!=null){
      USB usb = sc.getUSB();
      if(usb!=null){
         version = usb.getVersion();
      }
   }
}

根据我的理解,选项1将有额外的开销,因为它必须多次评估相同的表达,如computer.getSoundCard() 3次,computer.getSoundCard().getUSB()2次。

我的理解是否正确?

从选项2更改

version = computer.getSoundcard().getUSB().getVersion();
提问于
用户回答回答于

根据我的理解,选项1会有额外的开销,因为它必须多次评估相同的表达式

是的,这些电话会被多次拨打。但是,如果您将作业作为条件的一部分,则可以缩短它,如下所示:

SoundCard sc;
USB usb;
if(computer != null && (sc = computer.getSoundCard()) != null && (usb = sc.getUSB()) != null) {
    version = usb.getVersion();
}

请注意,要引用scusb表达内,里面的条件是安全的,因为&&评估是保证停止在到达第一false链。

用户回答回答于

更好的方法是将此USB版本获取代码提取到另一种方法中,比如说getComputerUsbVersion(),然后将超长if或嵌套if-else块压缩成几个简单if块:

public String getComputerUsbVersion(Computer computer) {

    if (computer == null)  return null; 

    SoundCard soundCard = computer.getSoundCard();
    if (soundCard == null) return null; 

    USB usb = soundCard.getUSB()
    if (usb == null) return null;

    return usb.getVersion();
}

正如你所看到的,代码更清晰易懂,超长if状态或嵌套if-else块也可以避免。你甚至可以很容易地在后面添加更多的条件检查代码。

扫码关注云+社区