java三元运算符的空指针NPE问题

周中发现一个问题，下面的代码会抛出空指针异常。

JSONObject timeZone = getTimeZone();
Long sign = null == timeZone? 8L : getSign(timeZone);

上述功能比较简单，根据传入的JSONObject获取UTC时区的偏移量，如果没有获取到，就返回北京时间默认的东八区偏移8L；

在getSign方法内部，会对timeZone进行处理，核心代码可以简短描述为

Long getSign(JSONObject timeZone) {
  // 当满足某些场景时，直接返回偏移量
  if (timeZone.containsKey("sign")){
    return timeZone.getLong("sign");
  }
  // 其他情况下，直接返回null
  return null;
}

问题出现在三元运算符，当三元运算符的右侧，发现8L:getSign两个操作时，会统一将其转为基本类型，也就是long。这就会涉及到基本类型的拆箱操作，如果getSign的返回结果为null，就会触发NPE。

测试代码如下:

public static void main(String[] args) {
  Long result = null == args? 8L : getSign(args);
  System.out.println(result);
}

private static Long getSign(String[] args) {
  if (null == args || args.length == 0) {
    return null;
  }
  return Long.valueOf(args[0]);
}

通过查看class文件 javap -c ，如下图

image-20210228221019707

标红的代码也就是拆箱操作，所以会触发NPE。

针对该问题，最简单的可以采用以下方式:

JSONObject timeZone = getTimeZone();
Long defaultSign = 8L;
Long sign = null == timeZone? defaultSign : getSign(timeZone);

所以，使用三元运算符时，一定要注意类似的问题。