本报讯 柬埔寨王宫事务部9月24日发布通告称，最受柬埔寨民众尊敬、爱戴的诺罗敦·莫尼列·西哈努克王太后陛下已年过花甲，目前正在北京的医院接受悉心治疗。...
2025-09-25 8
公司刚入职了一名中级 Java 开发，短短 4 行代码居然凑齐了 3 个 bug！我哭了~~
火币pro官网下载
2022年10月21日 11:02 111
Connor
↓推荐关注↓
作者：l拉不拉米
作者：l拉不拉米
今天带来一个真实案例，让大家更深刻的理解空指针异常。
公司刚入职了一名中级Java开发，经过一个星期的适应学习，各方面表现还不错，于是分配了一个小的迭代给新人做。
需求很简单，把从第三方拉取的数据匹配到自身公司后台设置的渠道后，聚合到一个列表中，批量入库。
然而就在匹配的逻辑中，上线后报了个NPE，这是作为一名中级开发不应犯的简单错误，新人被我狠狠的训了，记生产事故一次。
二、事故重现1、伪代码
说明：伪代码并非真实线上代码，只是为了更方便，更形象的重现事故现场而编写的；真实的业务场景往往更加复杂，NPE的漏洞隐藏在更深处，不易code view出来，也不易测试出来；生产环境NPE是较常见的异常，希望大家不要纠结为什么测试没测出来，关键还是通过这样一个案例了解NPE的原因和解决方案。
说明：伪代码并非真实线上代码，只是为了更方便，更形象的重现事故现场而编写的；真实的业务场景往往更加复杂，NPE的漏洞隐藏在更深处，不易code view出来，也不易测试出来；生产环境NPE是较常见的异常，希望大家不要纠结为什么测试没测出来，关键还是通过这样一个案例了解NPE的原因和解决方案。
String channelNo = channelDao.getOne.getChannelNo;
// 第三方拉取的数据
List<ThirdData> thirdDataList = );
// 匹配过滤
thirdDataList.stream.filter(o ->channelNo.equals(o.getChannelNo)).collect(Collectors.toList);
// 批量入库
thirdDataDao.saveAll(thirdDataList);
2、分析与解决
有经验、技术扎实的同学看到这里应该或多或少能发现问题了。其实啊，这四段代码是作者精心设计的，可谓是 卧龙凤雏。
展开全文
短短四行代码居然凑齐了3个NPE，我枯了~~
我们逐行分析：
3、第一行分析
channelDao.getOne如果返回为null，那么调用getChannelNo会报NPE。
4、 解决办法
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( "");
5、第三行分析（1）
thirdDataList如果为null，那么调用stream会报NPE。
通过下面的源码截图就能知道原因：
6、解决办法
1、使用防御性编程，提前返回（推荐）
// 推荐使用集合工具类判空
if(CollectionUtils.isEmpty(thirdDataList)) {
return;
2、使用if条件语句包裹（不推荐）
if(CollectionUtils.isNotEmpty(thirdDataList)) {
// 执行后面的逻辑
7、第三行分析（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。
- EOF -
点击标题可跳转
1、 新来个技术总监，给公司项目引入了全新的业务架构，堪称最佳实践！
2、 Android 异步加载布局，优化页面启动速度的几种方案
3、 Controller 层代码这么写，简洁又优雅！
看完本文有收获？请分享给更多人
推荐关注「Java开发精选」，提升Java开发技术
点赞和在看就是最大的支持❤️
相关文章
- 详细阅读
-
终于知道疼了？加拿大外长阿南德将访华，想劝中方收回成命，寻求改善两国关系！详细阅读
自从特鲁多政府在美国的怂恿下，对中国产电动汽车、钢铁和铝产品加征高额关税以来，加拿大与中国的关系就如同滑下了拼命向下的轨道。中国这一举动无疑是给加拿大...
2025-09-25 7
-
中方连抛3820亿美债，美国担忧的事来了，关键时刻巴菲特清空中企股票，信号不简单详细阅读
据报道，近年来的中美博弈已不仅仅体现在贸易战、科技封锁和军事对抗等领域，金融和资本市场的较量同样成为了全球关注的焦点。中国连续大规模减持美国国债，特别...
2025-09-25 7
-
成都自行车比赛发生集体摔车，成都市体育局：伤情最重者将观察是否安装假体详细阅读
近日，四川成都一自行车车迷赛中发生选手集体摔车事故，一名年轻男选手疑似膝盖伤情严重，被传“膝盖骨没了”，引发众多网友热议。日前，成都市体育局回应新黄河...
2025-09-25 5
-
以色列军方宣布：一名以军士兵在加沙城遭哈马斯狙击手袭击身亡详细阅读
【环球网快讯】据《以色列时报》25日最新消息，以色列军方宣布，一名以色列国防军士兵在加沙城遭巴勒斯坦伊斯兰抵抗运动（哈马斯）狙击手袭击身亡。 报道称，...
2025-09-25 7
-
事发上海五星级酒店！夫妻半夜被迫躲阳台，房间刺鼻气体致一人双眼角膜受伤详细阅读
花费不菲入住五星级酒店 半夜却无奈躲在阳台 究竟发生了什么？ 近日 30岁的孙先生反映 夫妻俩入住上海星河湾酒店后 自己眼角膜受伤，视物不清 妻子肺部...
2025-09-25 5
-
绍兴地铁“列车撞上保洁员致多人死伤”？浙江已提级调查！地铁客服：会在非运营时间派保洁员清理站台屏蔽门详细阅读
每经编辑｜何小桃 据新黄河9月24日报道，记者近日获悉，浙江省绍兴市轨道交通2号线9月13日晚曾发生一起地铁列车撞上保洁员致多人伤亡的安全事故。 多名...
2025-09-25 5
-
上海“臭味学生餐”供应商被立案侦查当日再次中标详细阅读
资料图：上海绿捷 本文综合上观新闻、正观新闻、采招网、21世纪经济报道等 昨天（9月23日）深夜，针对公众关注的“臭味学生餐”事件，上海官方发布通报称...
2025-09-25 6
发表评论