阅读全文

⭐⭐⭐ Spring Boot 项目实战	⭐⭐⭐ Spring Cloud 项目实战
《Dubbo 实现原理与源码解析 —— 精品合集》	《Netty 实现原理与源码解析 —— 精品合集》
《Spring 实现原理与源码解析 —— 精品合集》	《MyBatis 实现原理与源码解析 —— 精品合集》
《Spring MVC 实现原理与源码解析 —— 精品合集》	《数据库实体设计合集》
《Spring Boot 实现原理与源码解析 —— 精品合集》	《Java 面试题 + Java 学习指南》

摘要: 原创出处 juejin.cn/post/7031445206152577061 「l拉不拉米」欢迎转载，保留摘要，谢谢！

• 一、前言
• 二、事故重现
  • 第一行分析
  • 第三行分析（1）
  • 第三行分析（2）
• 最后

---

🙂🙂🙂关注**微信公众号：【芋道源码】**有福利：

1. RocketMQ / MyCAT / Sharding-JDBC 所有源码分析文章列表
2. RocketMQ / MyCAT / Sharding-JDBC 中文注释源码 GitHub 地址
3. 您对于源码的疑问每条留言都将得到认真回复。甚至不知道如何读源码也可以请教噢。
4. 新的源码解析文章实时收到通知。每周更新一篇左右。
5. 认真的源码交流微信群。

---

一、前言

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

分析与解决

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

短短四行代码居然凑齐了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())

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

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

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

最后

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

SonarLint

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

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