一、前言
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。
通過下面的源碼截圖就能知道原因:


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)
看源碼一目了然,該方法對左邊的對象做了非空判斷

3、用其他開源的工具類庫或者自己實現
如:
org.apache.commons.lang3.StringUtils
cn.hutool.core.util.StrUtil;
原文鏈接:juejin.cn/post/7031445206152577061
原文作者:l拉不拉米