20k招的工程师刚加入团队,4行代码写3个NPE异常,服了!

共 5172字,需浏览 11分钟

 ·

2022-09-04 20:00






点击关注公众号,Java干货

及时送达



👇





e2e9bc4c995023849caac0582bdcd956.webp



文章来源:https://juejin.cn/post/7031445206152577061
















一、前言












Hello 大家好
,今天带来一个真实案例,让大家更深刻的理解


空指针异常







8a4dc09d8dabae9a96739522cc2d75c2.webp

公司刚入职了一名中级Java开发,经过一个星期的适应学习,各方面表现还不错,于是分配了一个小的迭代给新人做。


需求很简单,把从第三方拉取的数据匹配到自身公司后台设置的渠道后,聚合到一个列表中,批量入库。


然而就在匹配的逻辑中,上线后报了个NPE,这是作为一名中级开发不应犯的简单错误,新人被我狠狠的训了,记生产事故一次。


3d09c201fcb9a792127fab371c8ce999.webp















二、事故重现

















伪代码






说明

:伪代码并非真实线上代码,只是为了更方便,更形象的重现事故现场而编写的;真实的业务场景往往更加复杂,NPE的漏洞隐藏在更深处,不易code view出来,也不易测试出来;生产环境NPE是较常见的异常,希望大家不要纠结为什么测试没测出来,关键还是通过这样一个案例了解NPE的原因和解决方案。





// 后台设置的渠道
String channelNo = channelDao.getOne().getChannelNo();
// 第三方拉取的数据
List<ThirdData> thirdDataList = httpClientUtils.getThirdDatas(DateUtils.today());
// 匹配过滤
thirdDataList.stream().filter(o ->channelNo.equals(o.getChannelNo())).collect(Collectors.toList());
// 批量入库
thirdDataDao.saveAll(thirdDataList);











分析与解决




有经验、技术扎实的同学看到这里应该或多或少能发现问题了。
其实啊,这四段代码是作者精心设计的,可谓是
卧龙凤雏
😀😀。






短短


四行代码


居然凑齐了3个NPE,我枯了/(ㄒoㄒ)/~~







我们逐行分析







第一行分析




channelDao.getOne()如果返回为null,那么调用getChannelNo()会报


NPE








解决办法


1、使用防御性编程,提前返回(需根据具体业务场景而定)




// 如果channelNo是方法逻辑执行的必须元素,推荐用此方法
Channel channel = channelDao.getOne();
if (channel == null) {
    return;
}





2、使用三目运算,返回空字符串("")




// 返回兜底的空字符串
String channelNo = channelDao.getOne() == null ? "" : channelDao.getOne().getChannelNo();





3、使用Optional函数,返回空字符串("")




String channelNo = Optional.ofNullable(channelDao.getOne()).orElse("");








第三行分析(1)







thirdDataList如果为null,那么调用stream()会报


NPE








通过下面的源码截图就能知道原因:






8c5d364458f3cf03db3d599643cc8680.webp






0c1ab4743629d8a416757d781a85083f.webp






解决办法


1、使用防御性编程,提前返回(推荐)




// 推荐使用集合工具类判空
if (CollectionUtils.isEmpty(thirdDataList)) {
    return;
}





2、使用if条件语句包裹(不推荐)




if (CollectionUtils.isNotEmpty(thirdDataList)) {
    // 执行后面的逻辑
}









第三行分析(2)







channelNo如果返回为null,那么执行channelNo.equals(o.getChannelNo())会报


NPE








我们知道,按Java的规范String的equals()方法的调用,要求左边是确定值,就是为了避免调用方为null的情况。然而这里调用方和equals的入参都是变量,这种情况该怎么办呢?





1、再加一句判断:




channelNo != null && channelNo.equals(o.getChannelNo())




2、其实可以用java.uti包下的Objects类的equals方法




Objects.equals(channelNoo.getChannelNo())





看源码一目了然,该方法对左边的对象做了非空判断






99d69c36dfc959e46acca726e63a86a7.webp





3、用其他开源的工具类库或者自己实现




如:
org.apache.commons.lang3.StringUtils
cn.hutool.core.util.StrUtil;














最后












在这里
要推荐一款IDEA的插件:




SonarLint




3a47db6af2d41126b230e64f94629655.webp






能动态的帮您检查代码漏洞,像NPE这种代码风险都会给于相应的提示。




SonarLint还有一个大名鼎鼎的服务端叫


SonarQube








1. 为什么建议大家使用 Linux 开发?有那么爽吗?



2. 被问懵了,加密后的数据如何进行模糊查询?



3. SpringBoot+@Async注解一起用,速度提升100倍!


4. 重试框架 Spring-Retry 和 Guava-Retry,你知道该怎么选吗?






最近面试BAT,整理一份面试资料
Java面试BATJ通关手册
,覆盖了Java核心技术、JVM、Java并发、SSM、微服务、数据库、数据结构等等。



获取方式:点“
在看
”,关注公众号并回复 
Java
 领取,更多内容陆续奉上。





PS:因公众号平台更改了推送规则,如果不想错过内容,记得读完点一下
在看
,加个
星标
,这样每次新文章推送才会第一时间出现在你的订阅列表里。



“在看”支持小哈呀,谢谢啦







浏览 72
点赞
评论
收藏
分享

手机扫一扫分享

分享
举报
评论
图片
表情
推荐
点赞
评论
收藏
分享

手机扫一扫分享

分享
举报