小秦哥 发表于 2024-5-27 16:22:52

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

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

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

伪代码

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


有经验、技术踏实的同砚看到这里应该或多或少能发现问题了。实在啊,这四段代码是作者经心设计的,可谓是卧龙凤雏
免责声明:如果侵犯了您的权益,请联系站长,我们会及时删除侵权内容,谢谢合作!更多信息从访问主页:qidao123.com:ToB企服之家,中国第一个企服评测及商务社交产业平台。
页: [1]
查看完整版本: 公司刚入职了一名 Java 中级开发,短短 4 行代码居然凑齐了 3 个 bug!我哭