培训的?公司刚入职的中级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拉不拉米

科技分类资讯推荐

发力低空经济 襄垣县竞逐产业发展“新赛道” - 天天要闻

发力低空经济 襄垣县竞逐产业发展“新赛道”

“嗡嗡嗡……”“森林防火,人人有责。您已进入东山林区……”近日,在襄垣县城市设计馆中央控制室内,随着县低空经济项目技术人员轻点鼠标,一架无人机从古韩镇森林防火指挥部新建的无人智能方舱腾空而起,沿着既定航线开启东山林区巡航任务。这一场景,标志
Win11 24H2 安全隐患:AppLocker 漏洞、性能投诉、数据丢失风险 - 天天要闻

Win11 24H2 安全隐患:AppLocker 漏洞、性能投诉、数据丢失风险

IT之家 5 月 6 日消息,微软上周宣布广泛发布 Windows 11 24H2,不过用户反馈在升级过程中频现重大 Bug,系统性能普遍下降,甚至有数据丢失的风险。IT之家曾于 5 月 4 日报道,微软针对符合升级条件,当前运行 23H2、22H2 和 21H2 版本的家庭版、专业版设备,将自动推送 24H2 更新。不过用户反馈在更新过程中存在诸多问题,.
大张伟成“联想 AI 挚友”;传大疆 Osmo 360 全景相机现身;全球最大Deepfake成人网站永久关闭 | 极客早知道 - 天天要闻

大张伟成“联想 AI 挚友”;传大疆 Osmo 360 全景相机现身;全球最大Deepfake成人网站永久关闭 | 极客早知道

OpenAI 重组计划被迫改变,将继续由非营利实体控制当地时间周一(5 月 5 日),人工智能(AI)研究公司 OpenAI 宣布,它将继续由其非营利性母公司控制,同时将推进营利性部门结构调整的计划,以便在 AI 竞赛中筹集更多资金、保持领先地位。去年 12 月,OpenAI 公布了一项重组计划,拟转型为一家公益企业(Public Benefit C...
学习商标合规使用要点!今天这场直播带你了解 - 天天要闻

学习商标合规使用要点!今天这场直播带你了解

今天下午2点,“宝山汇”APP将进行“云普法”直播。本课程立足企业实际需求,从商标使用的定义切入,结合《中华人民共和国商标法》相关条款和实际案例,详细阐述了商标使用的具体表现形式,并着重讲解了“撤销三年不使用”程序和“非商标使用”抗辩环节这两类特殊情形,旨在为企业提供可借鉴的商标合规使用要点。直播时间2...
突发,华为Mate80再次延期! - 天天要闻

突发,华为Mate80再次延期!

前段时间,毒哥给大家聊了有关华为Mate80系列的事情。对于年度压轴大戏,华为Mate80系列的发布时间,是外界最关注的。目前,外界对于华为Mate80系列发布日期,普遍都认为会在2025年Q4,也就是12月份。但是,毒哥得到的最新消息。
雷军,职务调整! - 天天要闻

雷军,职务调整!

5月5日,雷军职务调整冲上微博热搜。天眼查显示,近日,小米之家商业有限公司发生工商变更,雷军由执行董事改任董事,同时经营范围新增智能家庭消费设备销售、美发饰品销售。
雷军2025年遭老罪了!卸任董事又遇车祸,小米这回还能挺住不? - 天天要闻

雷军2025年遭老罪了!卸任董事又遇车祸,小米这回还能挺住不?

2025年5月,小米之家工商信息悄摸儿变了——雷军的职务从“执行董事”改成“董事”。就这俩字的变动,跟往舆论锅里扔了颗炸弹似的,炸出一堆问号:雷总这是要放权了?小米是不是出啥大事儿了?咱今儿就掰开揉碎了,聊聊雷军今年有多难,顺便看看小米这船