背景
今早在排查生产日志的时候发现了一个空指针异常,遂跟踪了一下,最后发现报异常的代码如下
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’这种问题的,解决的思路都是一样的