线上代码发现一个问题,下面的代码会抛出空指针异常。
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 ,如下图
标红的代码也就是拆箱操作,所以会触发NPE。
针对该问题,最简单的可以采用以下方式:
JSONObject timeZone = getTimeZone();Long defaultSign = 8L;Long sign = null == timeZone? defaultSign : getSign(timeZone);所以,使用三元运算符时,一定要注意类似的问题。