培訓的?公司剛入職的中級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拉不拉米

科技分類資訊推薦

中國首個獨立研發在美獲批的全球首創新葯,來自無錫高新區! - 天天要聞

中國首個獨立研發在美獲批的全球首創新葯,來自無錫高新區!

7月3日無錫高新區企業迪哲醫藥宣布舒沃哲的新葯上市申請正式獲得美國食品藥品監督管理局批准成為全球首個且唯一在美國獲批的表皮生長因子受體(EGFR)20號外顯子插入突變(exon20ins)非小細胞肺癌(NSCLC)國創新葯同時也是中國首個獨立研發在美獲批的全球首創新葯舒沃哲是一款口服、不可逆、針對多種EGFR突變亞型的高...
百度前副總裁璩靜開醫美診所,人均消費2218元,曾因 「霸道女總裁」 言論道歉辭職 - 天天要聞

百度前副總裁璩靜開醫美診所,人均消費2218元,曾因 「霸道女總裁」 言論道歉辭職

紅星資本局7月2日消息,百度前副總裁璩靜在華為總部坂田基地附近開了一家醫美診所。據公開資料,璩靜名下新增一家存續企業——深圳大為診所。該診所成立於2024年12月23日,璩靜持股比例為100%,認繳出資額為100萬元,經營範圍為診所服務等。企查查顯示,根據璩靜關聯的任職歷史,她曾任深圳市星光健康管理有限公司(以下簡...
推薦數商雲B2B撮合交易平台,搭建企業間合作橋樑 - 天天要聞

推薦數商雲B2B撮合交易平台,搭建企業間合作橋樑

來源:數商雲引言:企業數字化轉型與B2B撮合交易平台的崛起在全球經濟一體化與數字經濟深度融合的背景下,企業間的合作模式正經歷深刻變革。傳統供應鏈中信息不對稱、交易成本高、協同效率低等問題,已成為制約企業發展的核心痛點。
當無人機 「飛」 入生活,數字低空如何重構我們的出行與未來? - 天天要聞

當無人機 「飛」 入生活,數字低空如何重構我們的出行與未來?

交匯點訊 近日,蘇州國際博覽中心的展廳里人頭攢動,2025數字低空大會在此隆重舉行。作為第三屆低空(蘇州)產業創新生態大會的重要組成部分,這場由未來移動通信論壇與鵬城實驗室聯合主辦的盛會,承載著特殊的行業意義——繼2024年在深圳首次提出「數字低空」概念並引發廣泛共鳴後,它既是對過去一年行業探索的總結,更是...
雷軍披露小米 YU7 銷售數據;特斯拉 Q2 交付汽車 38.4 萬輛;榮耀 Magic V5 發布|Do早報 - 天天要聞

雷軍披露小米 YU7 銷售數據;特斯拉 Q2 交付汽車 38.4 萬輛;榮耀 Magic V5 發布|Do早報

Hello,大家早上好,又是元氣滿滿的一天,先來瀏覽新鮮的早報吧~【微信多端正式更新】7 月 2 日,微信 iOS / 安卓 8.0.61 正式版攜手登場;而在 7 月 1 日,微信在官網也上架了 PC 的新版本,Windows / Mac 4.0.6 迎來全新發布。【多地開放往屆高考成績網上查詢功能】河北、浙江、陝西、山東等地都開放了往屆高考成績的查...
長城為何不做增程?詳解Hi4背後的技術和戰略邏輯 - 天天要聞

長城為何不做增程?詳解Hi4背後的技術和戰略邏輯

在中國新能源汽車產業邁入智能化與電動化深水區的當下,動力技術路線的選擇,不僅關乎產品體驗,也在重塑車企的技術底盤與品牌認知。與越來越多自主品牌押注增程技術不同,長城汽車選擇走另一條更具挑戰性的路線——全場景智能四驅電混技術體系Hi4。這個選擇背後,是對效率、安全與技術普適性的權衡與押注。Hi4,一種技術「...
解碼「民生答卷」丨如何「喚醒」沉睡的養老資源 - 天天要聞

解碼「民生答卷」丨如何「喚醒」沉睡的養老資源

湖北日報全媒記者 方琳 李光正 張華「幸福食堂、健身房、閱覽室、娛樂室、理療室一應俱全,可沒多少人來。」6月19日,在鄂東南某鎮,一家開業一個多月的養老服務綜合體經營者王先生望著冷清的門廳,難掩焦慮。截然不同的是,在武漢眾多社區養老中心,唱歌、跳舞等活動日程排得滿滿當當,老人們排隊「搶位」,養老床位更是「...
YU7爆火是雷軍營銷太強?張全蛋貼臉開大車企大佬,遮羞布都撕了 - 天天要聞

YU7爆火是雷軍營銷太強?張全蛋貼臉開大車企大佬,遮羞布都撕了

6月26日,小米YU7正式公布售價,很多人都預料到小米YU7會非常火,但是完全沒有料到小米YU7能那麼火,3分鐘大定破20萬輛,18小時鎖單24.8萬輛,可以說創造了國內汽車市場前所未有的新紀錄。這也讓之前那些攻擊小米汽車的車企大佬和業外人士的表述顯得蒼白無力,小米YU7的價格不便宜,現在整個汽車行業的需求量也呈現疲軟態勢...