拜託!不要用 “ - - null " 做判空了
轉自:lizeyang
鏈接:https://blog.csdn.net/lizeyang/article/details/40040817
爲了避免空指針調用,我們經常會看到這樣的語句
...if (someobject != null) {
someobject.doCalc();}...
最終,項目中會存在大量判空代碼,多麼醜陋繁冗!如何避免這種情況?我們是否濫用了判空呢?
精華回答:
這是初、中級程序猿經常會遇到的問題。他們總喜歡在方法中返回 null,因此,在調用這些方法時,也不得不去判空。另外,也許受此習慣影響,他們總潛意識地認爲,所有的返回都是不可信任的,爲了保護自己程序,就加了大量的判空。
吐槽完畢,回到這個題目本身:
進行判空前,請區分以下兩種情況:
1、null 是一個有效有意義的返回值 (Where null is a valid response in terms of the contract; and)
2、null 是無效有誤的 (Where it isn't a valid response.)
你可能還不明白這兩句話的意思,不急,繼續往下看,接下來將詳細討論這兩種情況
先說第 2 種情況
null 就是一個不合理的參數,就應該明確地中斷程序,往外拋錯誤。這種情況常見於 api 方法。例如你開發了一個接口,id 是一個必選的參數,如果調用方沒傳這個參數給你,當然不行。你要感知到這個情況,告訴調用方 “嘿,哥們,你傳個 null 給我做甚 "。
相對於判空語句,更好的檢查方式有兩個
(1)assert 語句,你可以把錯誤原因放到 assert 的參數中,這樣不僅能保護你的程序不往下走,而且還能把錯誤原因返回給調用方,豈不是一舉兩得。(原文介紹了 assert 的使用,這裏省略)
(2) 也可以直接拋出空指針異常。上面說了,此時 null 是個不合理的參數,有問題就是有問題,就應該大大方方往外拋。
第 1 種情況會更復雜一些。
這種情況下,null 是個”看上去 “合理的值,例如,我查詢數據庫,某個查詢條件下,就是沒有對應值,此時 null 算是表達了“空” 的概念。
這裏給一些實踐建議:
1、假如方法的返回類型是 collections,當返回結果是空時,你可以返回一個空的 collections(empty list), 而不要返回 null,這樣調用側就能大膽地處理這個返回,例如調用側拿到返回後,可以直接 print list.size(),又無需擔心空指針問題。(什麼?想調用這個方法時,不記得之前實現該方法有沒按照這個原則?所以說,代碼習慣很重要!如果你養成習慣,都是這樣寫代碼(返回空 collections 而不返回 null),你調用自己寫的方法時,就能大膽地忽略判空)
2、返回類型不是 collections,又怎麼辦呢?
那就返回一個空對象(而非 null 對象),下面舉個 “栗子”,假設有如下代碼
public interface Action {
void doSomething();}
public interface Parser {
Action findAction(String userInput);}
其中,Parse 有一個接口 FindAction,這個接口會依據用戶的輸入,找到並執行對應的動作。假如用戶輸入不對,可能就找不到對應的動作(Action),因此 findAction 就會返回 null,接下來 action 調用 doSomething 方法時, 就會出現空指針。
解決這個問題的一個方式,就是使用 Null Object pattern(空對象模式)
我們來改造一下
類定義如下,這樣定義 findAction 方法後,確保無論用戶輸入什麼,都不會返回 null 對象
public class MyParser implements Parser {
private static Action DO_NOTHING = new Action() {
public void doSomething() { /* do nothing */ }
};
public Action findAction(String userInput) {
// ...
if ( /* we can't find any actions */ ) {
return DO_NOTHING;
}
}}
對比下面兩份調用實例
1、冗餘: 每獲取一個對象,就判一次空
Parser parser = ParserFactory.getParser();
if (parser == null) {
// now what?
// this would be an example of where null isn't (or shouldn't be) a valid response
}
Action action = parser.findAction(someInput);
if (action == null) {
// do nothing} else {
action.doSomething();}
2、精簡
ParserFactory.getParser().findAction(someInput).doSomething();
因爲無論什麼情況,都不會返回空對象,因此通過 findAction 拿到 action 後,可以放心地調用 action 的方法。
其他回答精選:
1、如果要用 equal 方法,請用 object <不可能爲空>.equal(object < 可能爲空 >))
例如:
使用
"bar".equals(foo)
而不是
foo.equals("bar")
2、Java8 或者 guava lib 中,提供了 Optional 類,這是一個元素容器,通過它來封裝對象,可以減少判空。不過代碼量還是不少。不爽。
3、如果你想返回 null,請挺下來想一想,這個地方是否更應該拋出一個異常
本文由 Readfog 進行 AMP 轉碼,版權歸原作者所有。
來源:https://mp.weixin.qq.com/s/iqvMDVOZNmiuomy5pCRxuw