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

科技分类资讯推荐

雷军谈小米YU7爆单:发布前蛮担心,绝对创造了中国汽车工业奇迹 - 天天要闻

雷军谈小米YU7爆单:发布前蛮担心,绝对创造了中国汽车工业奇迹

7月2日晚,小米创始人雷军戴小米AI眼镜进行返场直播,分享了小米YU7发布后“爆单”的原因。雷军称,在发布之前其实特别担心,主要有三点:一是SUV市场很大,但竞争很激烈。特斯拉Model Y比大家想象中强大的多,已经连续3年全球销冠,国内的一波波产品更新也没有撼动它的地位。二是小米YU7可能跟过去大家见过的那些产品不一...
全球媒体聚焦丨79%全球专利+80%市场份额!外媒从一场救援看中国无人机产业实力 - 天天要闻

全球媒体聚焦丨79%全球专利+80%市场份额!外媒从一场救援看中国无人机产业实力

近日,一段中国无人机在洪水中成功营救被困人员的短视频在海外社交平台广泛传播,多家国际媒体也竞相报道,并深入探讨中国无人机产业技术发展与创新应用。 《纽约时报》网站截图 据了解,这段短视频中的救援发生在广西柳州三江侗族自治县一村庄。受上游来水影响,这个村子里一些处于低洼地带的房屋被淹。由于水流上涨快,一...
博士天团攻坚激光芯片,拿到3个亿融资 - 天天要闻

博士天团攻坚激光芯片,拿到3个亿融资

记者|鄢子为编辑|陈晓平7月1日,北京飓芯科技对外官宣,完成3亿元B轮融资。飓芯成立于2017年7月,核心团队由多名经验丰富的博士组成,主攻氮化镓激光芯片产业化,实现关键核心器件的自主可控。本轮融资,飓芯获得国家基金、半导体产业方和一线投资机构的认可。3亿融资由深创投制造业转型升级新材料基金(国家制造业转型升...
臻宝科技科创板IPO获受理 系半导体零部件制造商 大基金二期等参投 - 天天要闻

臻宝科技科创板IPO获受理 系半导体零部件制造商 大基金二期等参投

《科创板日报》7月2日讯(记者 黄修眉 实习记者 戴嘉怡) 重庆臻宝科技股份有限公司(下称“臻宝科技”)科创板IPO申请近日获上交所受理,辅导机构为中信证券。臻宝科技是国内少数实现集成电路先进制程设备和高世代、高电压显示面板制造设备非金属零部件多品类供应、规模化量产的企业之一。此次IPO,臻宝科技拟募资13.98亿...
BW2025即将开展,技嘉AORUS雕妹约你3H|3A08 雕宅见 - 天天要闻

BW2025即将开展,技嘉AORUS雕妹约你3H|3A08 雕宅见

史上规模空前的BilibiliWorld2025将于2025年7月11日-13日在上海国家会展中心开展!知名电竞硬件品牌技嘉AORUS已确认参展,为玩家打造游戏盛宴。现场不仅能体验新款硬核电竞装备、畅玩热门游戏大作,参与激烈的1V1对战PK,更有甜辣萌趣的雕妹喊你3H|3A08等你来!多重互动火力全开,带你玩转整个BW,开启今夏最燃电竞狂欢。...
35项服务可跨境办理,“澳政易”自助服务机上线珠海市民服务中心 - 天天要闻

35项服务可跨境办理,“澳政易”自助服务机上线珠海市民服务中心

“十几分钟就办完了,现场的协助人员指导我操作,太方便了!”7月1日上午,澳门居民梁女士来到珠海市民服务中心1号楼3楼的综合服务厅办理业务,在工作人员的帮助下,她在港澳跨境服务自助办理区的“澳政易”自助服务机上很快就办完了身份证明业务。6月30日,广州、珠海、中山、江门四个大湾区城市的政务服务中心正式启用了...
65亿美元芯片收购案,遭美国二次调查 - 天天要闻

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

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

DRAM市场,将创新高

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