4 行代碼寫 3 個NPE異常，服了！

作者：l拉不拉米

來源：https://juejin.cn/post/7031445206152577061

一、前言

公司剛入職了一名

中級Java開發

，經過一個星期的适應學習，各方面表現還不錯，于是配置設定了一個小的疊代給新人做。

需求很簡單，把從第三方拉取的資料比對到自身公司背景設定的管道後，聚合到一個清單中，批量入庫。

然而就在比對的邏輯中，上線後報了個NPE，這是作為一名中級開發不應犯的簡單錯誤，新人被我

狠狠的訓了

，記生産事故一次。

二、事故重制

僞代碼

說明 ：僞代碼并非真實線上代碼，隻是為了更友善，更形象的重制事故現場而編寫的；真實的業務場景往往更加複雜，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);

複制

分析與解決

有經驗、技術紮實的同學看到這裡應該或多或少能發現問題了。其實啊，這四段代碼是作者精心設計的，可謂是卧龍鳳雛😀😀。

短短

四行代碼

居然湊齊了3個NPE，我枯了/(ㄒoㄒ)/~~

我們逐行分析：

第一行分析

channelDao.getOne()如果傳回為null，那麼調用getChannelNo()會報

NPE

。

解決辦法

1、使用防禦性程式設計，提前傳回（需根據具體業務場景而定）

// 如果channelNo是方法邏輯執行的必須元素，推薦用此方法
Channel channel = channelDao.getOne();
if (channel == null) {
    return;
}

複制

2、使用三目運算，傳回空字元串（""）

// 傳回兜底的空字元串
String channelNo = channelDao.getOne() == null ? "" : channelDao.getOne().getChannelNo();
複制代碼

複制

3、使用Optional函數，傳回空字元串（""）

String channelNo = Optional.ofNullable(channelDao.getOne()).orElse("");

複制

第三行分析（1）

thirdDataList如果為null，那麼調用stream()會報

NPE

。

通過下面的源碼截圖就能知道原因：

image.png

解決辦法

1、使用防禦性程式設計，提前傳回（推薦）

// 推薦使用集合工具類判空
if (CollectionUtils.isEmpty(thirdDataList)) {
    return;
}

複制

2、使用if條件語句包裹（不推薦）

if (CollectionUtils.isNotEmpty(thirdDataList)) {
    // 執行後面的邏輯
}

複制

第三行分析（2）

channelNo如果傳回為null，那麼執行channelNo.equals(o.getChannelNo())會報

NPE

。

我們知道，按Java的規範String的equals()方法的調用，要求左邊是确定值，就是為了避免調用方為null的情況。然而這裡調用方和equals的入參都是變量，這種情況該怎麼辦呢？

1、再加一句判斷：

channelNo != null && channelNo.equals(o.getChannelNo())
複制代碼

複制

2、其實可以用 `java.uti` 包下的 `Objects` 類的equals方法

Objects.equals(channelNo, o.getChannelNo())

複制

看源碼一目了然，該方法對左邊的對象做了非空判斷

image.png

3、用其他開源的工具類庫或者自己實作

如：
org.apache.commons.lang3.StringUtils
cn.hutool.core.util.StrUtil;

複制

最後

在這裡

l拉不拉米

要推薦一款IDEA的插件：

SonarLint

image.png

能動态的幫您檢查代碼漏洞，像NPE這種代碼風險都會給于相應的提示。

SonarLint還有一個大名鼎鼎的服務端叫

SonarQube

。

------

我們建立了一個高品質的技術交流群，與優秀的人在一起，自己也會優秀起來，趕緊點選加群，享受一起成長的快樂。另外，如果你最近想跳槽的話，年前我花了2周時間收集了一波大廠面經，節後準備跳槽的可以點選這裡領取！

4 行代碼寫 3 個NPE異常，服了！

一、前言

二、事故重制

僞代碼

分析與解決

第一行分析

解決辦法

1、使用防禦性程式設計，提前傳回（需根據具體業務場景而定）

2、使用三目運算，傳回空字元串（""）

3、使用Optional函數，傳回空字元串（""）

第三行分析（1）

解決辦法

1、使用防禦性程式設計，提前傳回（推薦）

2、使用if條件語句包裹（不推薦）

第三行分析（2）

1、再加一句判斷：

2、其實可以用 java.uti 包下的 Objects 類的equals方法

3、用其他開源的工具類庫或者自己實作

最後

推薦閱讀

2、其實可以用 `java.uti` 包下的 `Objects` 類的equals方法