4 行代码写 3 个NPE异常,服了!

Java后端技术

共 2720字,需浏览 6分钟

 ·

2022-09-15 18:04

往期热门文章:

1、扔掉工具类!MyBatis 一个简单配置搞定加密、解密,不能太方便了~!
2、小公司里用SpringBoot做MySQL分库分表,踩了一些坑!
3、没有几十年功力,写不出这一行“看似无用”的代码!!
4、为什么 Spring和IDEA 都不推荐使用 @Autowired 注解
5、从阿里跳槽来的工程师,写个Controller都这么优雅!

作者:l拉不拉米 

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

一、前言

公司刚入职了一名中级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

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

image.png
image.png

解决办法

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

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

image.png

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

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

最后

在这里l拉不拉米要推荐一款IDEA的插件:

SonarLint

image.png

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

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

往期热门文章:

1、国产开发工具的天花板,用来撸项目真香!
2、这个 MySQL bug 99% 的人会踩坑!
3、公司产品太多了,怎么实现一次登录产品互通?
4、Redis 官方可视化工具,高颜值,功能太强大!
5、用了BigDecimal就不会资损?了解下BigDecimal这五个坑
6、一个依赖搞定 Spring Boot 反爬虫,防止接口盗刷!
7、千万不要把 Request 传递到异步线程里面!有坑!
8、不卷了!入职字节一周就果断跑了。
9、SpringBoot+ShardingSphereJDBC实现读写分离!
10、不好意思, Maven 该换了!

浏览 22
点赞
评论
收藏
分享

手机扫一扫分享

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

手机扫一扫分享

分享
举报