公司刚入职了一名中级 Java 开发，短短 4 行代码居然凑齐了 3 个 bug！我哭了~~

火币pro官网下载 2022年10月21日 11:02 127 Connor

↓推荐关注↓

作者：l拉不拉米

今天带来一个真实案例，让大家更深刻的理解空指针异常。

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

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

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

二、事故重现1、伪代码

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

String channelNo = channelDao.getOne.getChannelNo;

// 第三方拉取的数据

List<ThirdData> thirdDataList = );

// 匹配过滤

thirdDataList.stream.filter(o ->channelNo.equals(o.getChannelNo)).collect(Collectors.toList);

// 批量入库

thirdDataDao.saveAll(thirdDataList);

2、分析与解决

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

展开全文

短短四行代码居然凑齐了3个NPE，我枯了~~

我们逐行分析：

3、第一行分析

channelDao.getOne如果返回为null，那么调用getChannelNo会报NPE。

4、解决办法

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( "");

5、第三行分析（1）

thirdDataList如果为null，那么调用stream会报NPE。

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

6、解决办法

1、使用防御性编程，提前返回（推荐）

// 推荐使用集合工具类判空

if(CollectionUtils.isEmpty(thirdDataList)) {

return;

2、使用if条件语句包裹（不推荐）

if(CollectionUtils.isNotEmpty(thirdDataList)) {

// 执行后面的逻辑

7、第三行分析（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)

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

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

如：

org.apache.commons.lang3.StringUtils

cn.hutool.core.util.StrUtil;

最后

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

这款插件就是：SonarLint

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

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

- EOF -

点击标题可跳转

1、新来个技术总监，给公司项目引入了全新的业务架构，堪称最佳实践！

2、 Android 异步加载布局，优化页面启动速度的几种方案

3、 Controller 层代码这么写，简洁又优雅！

看完本文有收获？请分享给更多人

推荐关注「Java开发精选」，提升Java开发技术

点赞和在看就是最大的支持❤️

C# 多线程ThreadPool用法举例

盟可睐Moncler成都远洋太古里旗舰店隆重揭幕

发表评论

公司刚入职了一名中级 Java 开发，短短 4 行代码居然凑齐了 3 个 bug！我哭了~~

C# 多线程ThreadPool用法举例

盟可睐Moncler成都远洋太古里旗舰店隆重揭幕

最新留言

标签列表