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

科技分類資訊推薦

全球媒體聚焦丨79%全球專利+80%市場份額!外媒從一場救援看中國無人機產業實力 - 天天要聞

全球媒體聚焦丨79%全球專利+80%市場份額!外媒從一場救援看中國無人機產業實力

近日,一段中國無人機在洪水中成功營救被困人員的短視頻在海外社交平台廣泛傳播,多家國際媒體也競相報道,並深入探討中國無人機產業技術發展與創新應用。 《紐約時報》網站截圖 據了解,這段短視頻中的救援發生在廣西柳州三江侗族自治縣一村莊。受上游來水影響,這個村子裏一些處於低洼地帶的房屋被淹。由於水流上漲快,一...
博士天團攻堅激光芯片,拿到3個億融資 - 天天要聞

博士天團攻堅激光芯片,拿到3個億融資

記者|鄢子為編輯|陳曉平7月1日,北京颶芯科技對外官宣,完成3億元B輪融資。颶芯成立於2017年7月,核心團隊由多名經驗豐富的博士組成,主攻氮化鎵激光芯片產業化,實現關鍵核心器件的自主可控。本輪融資,颶芯獲得國家基金、半導體產業方和一線投資機構的認可。3億融資由深創投製造業轉型升級新材料基金(國家製造業轉型升...
臻寶科技科創板IPO獲受理 系半導體零部件製造商 大基金二期等參投 - 天天要聞

臻寶科技科創板IPO獲受理 系半導體零部件製造商 大基金二期等參投

《科創板日報》7月2日訊(記者 黃修眉 實習記者 戴嘉怡) 重慶臻寶科技股份有限公司(下稱「臻寶科技」)科創板IPO申請近日獲上交所受理,輔導機構為中信證券。臻寶科技是國內少數實現集成電路先進制程設備和高世代、高電壓顯示面板製造設備非金屬零部件多品類供應、規模化量產的企業之一。此次IPO,臻寶科技擬募資13.98億...
BW2025即將開展,技嘉AORUS雕妹約你3H|3A08 雕宅見 - 天天要聞

BW2025即將開展,技嘉AORUS雕妹約你3H|3A08 雕宅見

史上規模空前的BilibiliWorld2025將於2025年7月11日-13日在上海國家會展中心開展!知名電競硬件品牌技嘉AORUS已確認參展,為玩家打造遊戲盛宴。現場不僅能體驗新款硬核電競裝備、暢玩熱門遊戲大作,參與激烈的1V1對戰PK,更有甜辣萌趣的雕妹喊你3H|3A08等你來!多重互動火力全開,帶你玩轉整個BW,開啟今夏最燃電競狂歡。...
35項服務可跨境辦理,「澳政易」自助服務機上線珠海市民服務中心 - 天天要聞

35項服務可跨境辦理,「澳政易」自助服務機上線珠海市民服務中心

「十幾分鐘就辦完了,現場的協助人員指導我操作,太方便了!」7月1日上午,澳門居民梁女士來到珠海市民服務中心1號樓3樓的綜合服務廳辦理業務,在工作人員的幫助下,她在港澳跨境服務自助辦理區的「澳政易」自助服務機上很快就辦完了身份證明業務。6月30日,廣州、珠海、中山、江門四個大灣區城市的政務服務中心正式啟用了...
65億美元芯片收購案,遭美國二次調查 - 天天要聞

65億美元芯片收購案,遭美國二次調查

本文由半導體產業縱橫(ID:ICVIEWS)綜合 美國FTC對軟銀收購Ampere展開深度調查。 據知情人士透露,美國聯邦貿易委員會就軟銀擬收購 Arm 服務器處理器廠商Ampe....
DRAM市場,將創新高 - 天天要聞

DRAM市場,將創新高

本文由半導體產業縱橫(ID:ICVIEWS)綜合 傳統通用型DRAM和服務器高價值DRAM量價齊升雙重驅動,2025年DRAM市場有望創新高。 根據CFM最新報告顯示,2025年....
國產晶圓代工,市場巨變! - 天天要聞

國產晶圓代工,市場巨變!

未來十年,將是晶圓代工業的關鍵轉折期。 這一判斷,在近期一組數據中得到了清晰印證。根據 Yole Group 的最新報告,中國大陸有望在 2030 年超越中國台灣,躍居全球最大半導體晶圓代....