公司刚入职了一名 Java 中级开发，短短 4 行代码居然凑齐了 3 个 bug！我哭 ...

作者：l拉不拉米
链接：https://juejin.cn/post/7031445206152577061
一、前言

公司刚入职了一名中级Java开发，经过一个星期的适应学习，各方面表现还不错，于是分配了一个小的迭代给新人做。
需求很简单，把从第三方拉取的数据匹配到自身公司后台设置的渠道后，聚合到一个列表中，批量入库。
然而就在匹配的逻辑中，上线后报了个NPE，这是作为一名中级开发不应犯的简单错误，新人被我狠狠的训了，记生产事故一次。
二、事故重现

伪代码

分析：伪代码并非真实线上代码，只是为了更方便，更形象的重现事故现场而编写的；真实的业务场景每每更加复杂，NPE的漏洞隐藏在更深处，不易code view出来，也不易测试出来；生产环境NPE是较常见的异常，希望大家不要纠结为什么测试没测出来，关键还是通过这样一个案例了解NPE的缘故起因和办理方案。

1. // 后台设置的渠道
2. String channelNo = channelDao.getOne().getChannelNo();
3. // 第三方拉取的数据
4. List<ThirdData> thirdDataList = httpClientUtils.getThirdDatas(DateUtils.today());
5. // 匹配过滤
6. thirdDataList.stream().filter(o ->channelNo.equals(o.getChannelNo())).collect(Collectors.toList());
7. // 批量入库
8. thirdDataDao.saveAll(thirdDataList);

复制代码

分析与办理


有经验、技术踏实的同砚看到这里应该或多或少能发现问题了。实在啊，这四段代码是作者经心设计的，可谓是卧龙凤雏
免责声明：如果侵犯了您的权益，请联系站长，我们会及时删除侵权内容，谢谢合作！更多信息从访问主页：qidao123.com:ToB企服之家，中国第一个企服评测及商务社交产业平台。