这俩天忙着上线的任务,可谓是提心吊胆啊,忙到俩三点,还好,总算上线成功了,庆祝一番!但是,上线的过程中却有着一桩血案,且听我娓娓道来。上代码:
try {
List<NumPlateInfo> numPlateInfos = numPlateInfoMapper.getAllPlateNums();
List list = new ArrayList();
LongAndLat longAndLat;
for (NumPlateInfo n : numPlateInfos) {
longAndLat = new LongAndLat();
longAndLat.setLongitude(n.getLongitude());
longAndLat.setLatitude(n.getLatitude());
list.add(longAndLat);
}
result.setData(list);
} catch (Exception e) {
logger.error("查询门牌错误", e);
}
return result;
}
这是业务中的一段查询的代码,乍一眼看,没有多大的问题是吧?NO NO NO ,就是这段代码引起的血案!!!
注意看这段代码 :longAndLat.setLongitude(n.getLongitude());
longAndLat.setLatitude(n.getLatitude());
这段代码错在哪里呢?(敲黑板!!!)没有判空!!!数据库里,这个字段的值有可能是空的,那么在空的情况下,这个异常被try.....catch 捕获,却没有在控制台打印。为什么呢?看catch块里,打印了呀,对,没错,那只是在错误日志里打印的,没在控制台打印!!!上线的时候,由于存在空值,接口返回的也是空,没有打印报错信息(错误日志由于没有权限无法拉取)各种找问题,最后找到了这里!夜里俩点中,我的组员都急哭了(没错,是个妹子,哈哈),好在我还是比较机灵,俩分钟帮她改了(起初第一眼我也没看出来)结果还是搞到了四点(因为上线流程比较麻烦,还得审批啥的)。
正确代码:
try {
List<NumPlateInfo> numPlateInfos = numPlateInfoMapper.getAllPlateNums();
List list = new ArrayList();
LongAndLat longAndLat;
for (NumPlateInfo n : numPlateInfos) {
longAndLat = new LongAndLat();
if (StringUtils.isNotEmpty(n.getLatitude()) && StringUtils.isNotEmpty(n.getLatitude())) {
longAndLat.setLongitude(n.getLongitude());
longAndLat.setLatitude(n.getLatitude());
list.add(longAndLat);
}
}
result.setData(list);
} catch (Exception e) {
e.printStackTrace();
logger.error("查询门牌错误", e);
}
return result;
}
大家在开发过程中一定不要忘记判断空值哦,哪怕确定数据不会存在空值也要判空,判空判空判空!!!说三遍!