Unboxing of ‘xxx‘ may produce ‘NullPointerException‘

背景

今早在排查生产日志的时候发现了一个空指针异常，遂跟踪了一下，最后发现报异常的代码如下

if (null != vo.getLongitude()) 
	{
     rsp.setDistance(Objects.isNull(map.get("distance")) ? null : (double) map.get("distance"));
     }
     //distance定义
     private double distance;

以上代码逻辑很简单，就是在经度不为空且计算的distance不为空的情况下，取出map中的distance字段

分析

编辑器在null的地方标上了黄线，并且提示Unboxing of 'null' may produce 'NullPointerException',原来是null这儿出的问题，那我们来分析一下，为什么会出现这个错误。

我使用了Objects.isNull方法对map中的参数进行判空，如果为空，则直接赋值null，如果不为空，则强转为double类型。

而因为我的distance字段是double类型，是个基本数据类型，所以在distance为空的时候，null会自动拆箱为double基本类型，这里就可能会产生空指针异常

解决

所以说，只需要避免null的自动拆箱就可以解决，这里给予两种方案

• 方案一：
  将distance的字段类型改成包装类型Double，那么原始的判断逻辑为空时赋值null就不会出现空指针问题了

• 方案二：
  既然在为空的时候，是给了null默认值报的错，那我们换成double类型的默认值，就不会报错了，比如

if (null != vo.getLongitude()) 
	{
     rsp.setDistance(Objects.isNull(map.get("distance")) ? 0.0 : (double) map.get("distance"));
     }
     //distance定义
     private double distance;

这个默认值需要根据业务来设置，比如我这里是计算的距离，那么默认值给0.0显然是不合适的

总结

总觉得空指针排查比较容易，但是今天遇到的这个问题还是挺有意思的，其实本质还是基本类型的装箱和拆箱问题。凡是遇到Unboxing of ‘xxx’ may produce 'NullPointerException’这种问题的，解决的思路都是一样的