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

最新推荐文章于 2024-10-09 11:13:12 发布

Java精选

最新推荐文章于 2024-10-09 11:13:12 发布

阅读量79

点赞数

文章标签： java 编程语言大数据 spring 面试

原文链接：https://mp.weixin.qq.com/s?__biz=MzAwMTE3MDY4MQ==&mid=2652464934&idx=1&sn=b6c88e9126affeba6033aff1f492c4f5&chksm=81300ab0b64783a67ebd53f7013f4e163937c2549f9c516354b5149bd32b5e7e13d6c0932846&scene=126&&sessionid=0

版权

一、前言

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精选，回复“加群”，即可入群！

Java精选面试题（微信小程序）：3000+道面试题，包含Java基础、并发、JVM、线程、MQ系列、Redis、Spring系列、Elasticsearch、Docker、K8s、Flink、Spark、架构设计等，在线随时刷题！
------ 特别推荐 ------
特别推荐：专注分享最前沿的技术与资讯，为弯道超车做好准备及各种开源项目与高效率软件的公众号，「大咖笔记」，专注挖掘好东西，非常值得大家关注。点击下方公众号卡片关注。

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