查看原文
其他

20k招的程序员,4行代码写3个NPE异常,服了!

点击关注 👉 Java精选 2022-10-20
一、前言
Hello 大家好,今天带来一个真实案例,让大家更深刻的理解空指针异常。
公司刚入职了一名中级Java开发,经过一个星期的适应学习,各方面表现还不错,于是分配了一个小的迭代给新人做。
需求很简单,把从第三方拉取的数据匹配到自身公司后台设置的渠道后,聚合到一个列表中,批量入库。
然而就在匹配的逻辑中,上线后报了个NPE,这是作为一名中级开发不应犯的简单错误,新人被我狠狠的训了,记生产事故一次。
二、事故重现
伪代码
说明:伪代码并非真实线上代码,只是为了更方便,更形象的重现事故现场而编写的;真实的业务场景往往更加复杂,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);
分析与解决
有经验、技术扎实的同学看到这里应该或多或少能发现问题了。其实啊,这四段代码是作者精心设计的,可谓是卧龙凤雏😀😀。面试宝典:https://www.yoodb.com
短短四行代码居然凑齐了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。
通过下面的源码截图就能知道原因:
解决办法
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(channelNo, o.getChannelNo())
看源码一目了然,该方法对左边的对象做了非空判断,另外,公众 号Java精选,回复java面试,获取面试资料,支持在线刷题。
3、用其他开源的工具类库或者自己实现,如:
org.apache.commons.lang3.StringUtilscn.hutool.core.util.StrUtil;
最后
在这里要推荐一款IDEA的插件:SonarLint
能动态的帮您检查代码漏洞,像NPE这种代码风险都会给于相应的提示。
SonarLint还有一个大名鼎鼎的服务端叫SonarQube。

作者:l拉不拉米

https://juejin.cn/post/7031445206152577061

公众号“Java精选”所发表内容注明来源的,版权归原出处所有(无法查证版权的或者未注明出处的均来自网络,系转载,转载的目的在于传递更多信息,版权属于原作者。如有侵权,请联系,笔者会第一时间删除处理!

最近有很多人问,有没有读者交流群!加入方式很简单,公众号Java精选,回复“加群”,即可入群!

(微信小程序):3000+道面试题,包含Java基础、并发、JVM、线程、MQ系列、Redis、Spring系列、Elasticsearch、Docker、K8s、Flink、Spark、架构设计等,在线随时刷题!

------ 特别推荐 ------

特别推荐:专注分享最前沿的技术与资讯,为弯道超车做好准备及各种开源项目与高效率软件的公众号,「大咖笔记」,专注挖掘好东西,非常值得大家关注。点击下方公众号卡片关注

点击“阅读原文”,了解更多精彩内容!文章有帮助的话,点在看,转发吧!

您可能也对以下帖子感兴趣

文章有问题?点此查看未经处理的缓存