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

科技分類資訊推薦

發力低空經濟 襄垣縣競逐產業發展“新賽道” - 天天要聞

發力低空經濟 襄垣縣競逐產業發展“新賽道”

“嗡嗡嗡……”“森林防火,人人有責。您已進入東山林區……”近日,在襄垣縣城市設計館中央控制室內,隨着縣低空經濟項目技術人員輕點鼠標,一架無人機從古韓鎮森林防火指揮部新建的無人智能方艙騰空而起,沿着既定航線開啟東山林區巡航任務。這一場景,標誌
Win11 24H2 安全隱患:AppLocker 漏洞、性能投訴、數據丟失風險 - 天天要聞

Win11 24H2 安全隱患:AppLocker 漏洞、性能投訴、數據丟失風險

IT之家 5 月 6 日消息,微軟上周宣布廣泛發布 Windows 11 24H2,不過用戶反饋在升級過程中頻現重大 Bug,系統性能普遍下降,甚至有數據丟失的風險。IT之家曾於 5 月 4 日報道,微軟針對符合升級條件,當前運行 23H2、22H2 和 21H2 版本的家庭版、專業版設備,將自動推送 24H2 更新。不過用戶反饋在更新過程中存在諸多問題,.
大張偉成「聯想 AI 摯友」;傳大疆 Osmo 360 全景相機現身;全球最大Deepfake成人網站永久關閉 | 極客早知道 - 天天要聞

大張偉成「聯想 AI 摯友」;傳大疆 Osmo 360 全景相機現身;全球最大Deepfake成人網站永久關閉 | 極客早知道

OpenAI 重組計劃被迫改變,將繼續由非營利實體控制當地時間周一(5 月 5 日),人工智能(AI)研究公司 OpenAI 宣布,它將繼續由其非營利性母公司控制,同時將推進營利性部門結構調整的計劃,以便在 AI 競賽中籌集更多資金、保持領先地位。去年 12 月,OpenAI 公布了一項重組計劃,擬轉型為一家公益企業(Public Benefit C...
學習商標合規使用要點!今天這場直播帶你了解 - 天天要聞

學習商標合規使用要點!今天這場直播帶你了解

今天下午2點,“寶山匯”APP將進行“雲普法”直播。本課程立足企業實際需求,從商標使用的定義切入,結合《中華人民共和國商標法》相關條款和實際案例,詳細闡述了商標使用的具體表現形式,並着重講解了“撤銷三年不使用”程序和“非商標使用”抗辯環節這兩類特殊情形,旨在為企業提供可借鑒的商標合規使用要點。直播時間2...
突發,華為Mate80再次延期! - 天天要聞

突發,華為Mate80再次延期!

前段時間,毒哥給大家聊了有關華為Mate80系列的事情。對於年度壓軸大戲,華為Mate80系列的發布時間,是外界最關注的。目前,外界對於華為Mate80系列發布日期,普遍都認為會在2025年Q4,也就是12月份。但是,毒哥得到的最新消息。
雷軍,職務調整! - 天天要聞

雷軍,職務調整!

5月5日,雷軍職務調整衝上微博熱搜。天眼查顯示,近日,小米之家商業有限公司發生工商變更,雷軍由執行董事改任董事,同時經營範圍新增智能家庭消費設備銷售、美髮飾品銷售。
雷軍2025年遭老罪了!卸任董事又遇車禍,小米這回還能挺住不? - 天天要聞

雷軍2025年遭老罪了!卸任董事又遇車禍,小米這回還能挺住不?

2025年5月,小米之家工商信息悄摸兒變了——雷軍的職務從“執行董事”改成“董事”。就這倆字的變動,跟往輿論鍋里扔了顆炸彈似的,炸出一堆問號:雷總這是要放權了?小米是不是出啥大事兒了?咱今兒就掰開揉碎了,聊聊雷軍今年有多難,順便看看小米這船