培训的?公司刚入职的中级Java开发,短短4行代码居然凑齐了3个bug

2022年10月08日21:14:48 科技 1661

一、前言

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

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

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

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

二、事故重现

1、伪代码

说明:伪代码并非真实线上代码,只是为了更方便,更形象的重现事故现场而编写的;真实的业务场景往往更加复杂,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);

2、分析与解决

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

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

我们逐行分析:

3、第一行分析

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

4、解决办法

1、使用防御性编程,提前返回(需根据具体业务场景而定)

// 如果channelNo是方法逻辑执行的必须元素,推荐用此方法
Channel channel = channelDao.getOne;
if (channel == ) {
return;
}

2、使用三目运算,返回空字符串("")

// 返回兜底的空字符串
String channelNo = channelDao.getOne == ? "" : channelDao.getOne.getChannelNo;

3、使用Optional函数,返回空字符串("")

String channelNo = Optional.ofable(channelDao.getOne).orElse("");

5、第三行分析(1)

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

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

培训的?公司刚入职的中级Java开发,短短4行代码居然凑齐了3个bug - 天天要闻培训的?公司刚入职的中级Java开发,短短4行代码居然凑齐了3个bug - 天天要闻

6、解决办法

1、使用防御性编程,提前返回(推荐)

// 推荐使用集合工具类判空
if (CollectionUtils.isEmpty(thirdDataList)) {
return;
}

2、使用if条件语句包裹(不推荐)

if (CollectionUtils.isNotEmpty(thirdDataList)) {
// 执行后面的逻辑
}

7、第三行分析(2)

channelNo如果返回为,那么执行channelNo.equals(o.getChannelNo)会报NPE。

我们知道,按Java的规范String的equals方法的调用,要求左边是确定值,就是为了避免调用方为的情况。然而这里调用方和equals的入参都是变量,这种情况该怎么办呢?

1、再加一句判断:

channelNo != && channelNo.equals(o.getChannelNo)

2、其实可以用java.uti包下的Objects类的equals方法

Objects.equals(channelNo, o.getChannelNo)

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

培训的?公司刚入职的中级Java开发,短短4行代码居然凑齐了3个bug - 天天要闻

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

如:

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

原文链接:juejin.cn/post/7031445206152577061

原文作者:l拉不拉米

科技分类资讯推荐

65亿美元芯片收购案,遭美国二次调查 - 天天要闻

65亿美元芯片收购案,遭美国二次调查

本文由半导体产业纵横(ID:ICVIEWS)综合 美国FTC对软银收购Ampere展开深度调查。 据知情人士透露,美国联邦贸易委员会就软银拟收购 Arm 服务器处理器厂商Ampe....
DRAM市场,将创新高 - 天天要闻

DRAM市场,将创新高

本文由半导体产业纵横(ID:ICVIEWS)综合 传统通用型DRAM和服务器高价值DRAM量价齐升双重驱动,2025年DRAM市场有望创新高。 根据CFM最新报告显示,2025年....
国产晶圆代工,市场巨变! - 天天要闻

国产晶圆代工,市场巨变!

未来十年,将是晶圆代工业的关键转折期。 这一判断,在近期一组数据中得到了清晰印证。根据 Yole Group 的最新报告,中国大陆有望在 2030 年超越中国台湾,跃居全球最大半导体晶圆代....
像卖车一样卖机器人?机器人体验店迎来爆发潮 - 天天要闻

像卖车一样卖机器人?机器人体验店迎来爆发潮

变现是目前人形机器人最大的问题。为了提高销量,更快形成社会氛围,拉动人才集中,业内诞生出多个全新思路。其中,集中展示4S店以及比赛,无疑是近期人形机器人行业关注的焦点。本文盘点了近日出现的4S店新模式▍具微科技(机器人4S店)6月30日,杭州迎来全国首家具身智能机器人4S店的开业。这个由具微科技打造的具身智能...
79元,小米刚上架的这服务,真的太爽啦 - 天天要闻

79元,小米刚上架的这服务,真的太爽啦

虽然今天比较受关注的,是小米1999元/月租金的“小米青年公寓”。但机哥还是想提醒一波。手持小米老机型的机友们,要是电池健康度已经掉了很多、又或是觉得手机续航不太够用。可别错过小米今天开启的“七月服务周”活动。活动时间是7月1日10点-7月
“全球Z世代创未来”:“上海·未来城市”全球创意作品征集 - 天天要闻

“全球Z世代创未来”:“上海·未来城市”全球创意作品征集

上海的未来是什么样子?让青年人来说,才算数。无论你来自哪里,带着怎样的故事,都能在这座城市找到属于自己的归属感。只因上海,从来不只是一座城,更是无数梦想的交汇点。用年轻的方式,去看见这座城市的无限可能吧。一则视频,一张照片,一幅插画,甚至是一行代码……你的创意,将成为链接世界的桥梁;你的表达,将定义...
行进中国丨重庆汽车产业向“新”而行 - 天天要闻

行进中国丨重庆汽车产业向“新”而行

这座超级工厂内,1600多台智能终端、3000多台机器人协同运作,实现了焊接过程自动化率100%、喷涂自动化率100%、全自动智能伺服压机线自动化率100%,满产后下线一台车最快仅需30秒。
微信朋友圈有变动!网友:最讨厌的功能终于取消了 - 天天要闻

微信朋友圈有变动!网友:最讨厌的功能终于取消了

1日有网友发消息称微信最新版本可以关闭共同好友点赞提醒了开启这项功能后,共同好友对你互动过的朋友圈进行点赞或评论时,系统不再推送红点提醒,但仍可在消息列表中查看,也就是共友互动时就不再通知了。