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 ,如下图

java三元运算符的空指针NPE问题_第1张图片

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

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

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

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

你可能感兴趣的:(java,java)