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

科技分類資訊推薦

年中購物「雙促」聯動:5月13日晚8點,京東心動購物季和京東618接踵而來 - 天天要聞

年中購物「雙促」聯動:5月13日晚8點,京東心動購物季和京東618接踵而來

【TechWeb】5月7日消息,京東舉辦「2025年京東618招商啟動大會」,會上宣布年中購物「雙促」節奏:5月13日晚8點至5月28日24點,京東將重磅推出心動購物季活動,消費者可以在京東618啟幕前率先體驗購物驚喜。在這之後,京東618將於5月31日晚8點現貨開賣。今年京東618設置了「開門紅、專場期、高潮期、返場期」四大階段,期間...
華為鴻蒙PC即將登場,摺疊產業鏈再掀波瀾 - 天天要聞

華為鴻蒙PC即將登場,摺疊產業鏈再掀波瀾

在科技領域,華為一直是創新的代名詞。近期,華為又有大動作吸引了全球目光 —— 即將推出鴻蒙PC,這一消息猶如一顆重磅炸彈,在科技圈引發廣泛熱議。眾多科技博主如 @數碼閑聊站 和 @超維界 紛紛預測,此次華為鴻蒙PC 產品線將呈現雙驚喜,不僅有傳統設計風格的PC,更令人期待的是一款摺疊PC產品。博主@定焦數碼 也爆料稱...
從圓桌派到2025大會,尹燁的演講始終帶來希望 - 天天要聞

從圓桌派到2025大會,尹燁的演講始終帶來希望

「這知識量和語速對於普通人來講相當於開了腦機介面了」「聽出了雷庫茲韋爾預言的味道」 「科技在發展,也要以人為本,才得始終」 「講的真好,中國需要更多的尹哥」 「最適合年輕人好好自我思考自己該如何把握這個時代的脈搏,跟上這個時代的腳步」 「為什麼偉大不能被計劃,這是因為科學提倡的是無中生有,計劃就是有中生...
「『區塊鏈+調解服務』探索糾紛化解新模式」入選本市區塊鏈創新應用典型案例|法律科技專刊 - 天天要聞

「『區塊鏈+調解服務』探索糾紛化解新模式」入選本市區塊鏈創新應用典型案例|法律科技專刊

4月30日,市委網信辦組織召開上海市區塊鏈發展管理統籌協調機制聯絡員會議,總結交流區塊鏈創新發展工作進展,通報本市區塊鏈創新應用典型案例評選情況並頒發證書。市司法局和徐匯區司法局聯合申報的「『區塊鏈+調解服務』探索糾紛化解新模式」被評為2024年上海市區塊鏈創新應用15個典型案例之一。評選背景為貫徹落實國家區...
聯想的AI野心:用「超級智能體」卡位萬億市場?​ - 天天要聞

聯想的AI野心:用「超級智能體」卡位萬億市場?​

最近兩年,在AIGC(生成式人工智慧)的推動下,各類AI硬體層出不窮,但除了AI PC和AI手機,其他AI硬體幾乎都還未被市場認可。作為全球最大的PC廠商之一,以及旗下還擁有摩托羅拉智能手機業務,聯想顯然也渴望在這輪AI硬體浪潮中,打下一片屬於自己的天地。5月7日,聯想集團以「讓AI成為創新生產力」為主題,在Tech World 2...
星海圖雙臂移動平台 R1 Lite 發布,搭載酷睿 i9-12900HK - 天天要聞

星海圖雙臂移動平台 R1 Lite 發布,搭載酷睿 i9-12900HK

IT之家 5 月 7 日消息,星海圖今日發布了全新升級的雙臂移動平台 R1 Lite。R1 Lite 可選上裝 / 側裝雙布局,支持 60cm 臂長 6 DOF A1X 雙臂協同作業,還配有 3 DOF 軀幹、6 DOF 矢量底盤。R1 Lite 搭載了英特爾酷睿 i9-12900HK 處理器、32GB 內存 + 1TB SSD 存儲,IT之家附官方一圖知如
理想汽車回應李想6.39億元年薪:說法不準確 - 天天要聞

理想汽車回應李想6.39億元年薪:說法不準確

紅星資本局5月7日消息,近日,理想汽車(02015.HK/LI.US)創始人、董事長兼CEO李想2024年領取6.39億元「天價年薪」衝上熱搜引發熱議。5月7日,理想汽車方面向紅星資本局表示:「『李想年薪6.39億』這一說法並不準確。