當前位置: 妍妍網 > 碼農

拜托!別再濫用 ! = null 判空了!!

2024-04-02碼農

點選關註公眾號,Java幹貨 及時送達 👇

問題

為了避免空指標呼叫,我們經常會看到這樣的語句

...
if (someobject != null) {
someobject.doCalc();
}
...

最終,計畫中會存在大量判空程式碼,多麽醜陋繁冗!如何避免這種情況?我們是否濫用了判空呢?

精華回答

這是初、中級程式猿經常會遇到的問題。他們總喜歡在方法中返回null,因此,在呼叫這些方法時,也不得不去判空。另外,也許受此習慣影響,他們總潛意識地認為,所有的返回都是不可信任的,為了保護自己程式,就加了大量的判空。

吐槽完畢,回到這個題目本身:

進行判空前,請區分以下兩種情況:

  • null 是一個有效有意義的返回值(Where null is a valid response in terms of the contract; and)

  • null是無效有誤的(Where it isn't a valid response.)

  • 你可能還不明白這兩句話的意思,不急,繼續往下看,接下來將詳細討論這兩種情況

    先說第2種情況

    null就是一個不合理的參數,就應該明確地中斷程式,往外拋錯誤。這種情況常見於api方法。例如你開發了一個介面,id是一個必選的參數,如果呼叫方沒傳這個參數給你,當然不行。你要感知到這個情況,告訴呼叫方「嘿,哥們,你傳個null給我做甚"。

    相對於判空語句,更好的檢查方式有兩個

  • assert語句,你可以把錯誤原因放到assert的參數中,這樣不僅能保護你的程式不往下走,而且還能把錯誤原因返回給呼叫方,豈不是一舉兩得。(原文介紹了assert的使用,這裏省略)

  • 也可以直接丟擲空指標異常。上面說了,此時null是個不合理的參數,有問題就是有問題,就應該大大方方往外拋。

  • 第1種情況會更復雜一些。

    這種情況下,null是個」看上去「合理的值,例如,我查詢資料庫,某個查詢條件下,就是沒有對應值,此時null算是表達了「空」的概念。

    這裏給一些實踐建議:

    1、假如方法的返回型別是collections,當返回結果是空時,你可以返回一個空的 collections(empty list) ,而不要返回null,這樣呼叫側就能大膽地處理這個返回,例如呼叫側拿到返回後,可以直接 print list.size() ,又無需擔心空指標問題。(什麽?想呼叫這個方法時,不記得之前實作該方法有沒按照這個原則?

    所以說,程式碼習慣很重要!如果你養成習慣,都是這樣寫程式碼(返回空collections而不返回null),你呼叫自己寫的方法時,就能大膽地忽略判空)

    2、返回型別不是collections,又怎麽辦呢?

    那就返回一個空物件(而非null物件),下面舉個「栗子」,假設有如下程式碼

    publicinterfaceAction{
    voiddoSomething();
    }
    publicinterfaceParser{
    Action findAction(String userInput);
    }

    其中,Parse有一個介面FindAction,這個介面會依據使用者的輸入,找到並執行對應的動作。假如使用者輸入不對,可能就找不到對應的動作(Action),因此findAction就會返回null,接下來action呼叫doSomething方法時,就會出現空指標。解決這個問題的一個方式,就是使用Null Object pattern(空物件模式)

    我們來改造一下

    類別定義如下,這樣定義findAction方法後,確保無論使用者輸入什麽,都不會返回null物件

    public classMyParserimplementsParser{
    privatestatic Action DO_NOTHING = new Action() {
    publicvoiddoSomething()/* 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,請挺下來想一想,這個地方是否更應該丟擲一個異常

    來源|http://blog.csdn.net/lizeyang


    END


    看完本文有收獲?請轉發分享給更多人

    關註「Java編程鴨」,提升Java技能

    關註Java編程鴨微信公眾號,後台回復:碼農大禮包可以獲取最新整理的技術資料一份。涵蓋Java 框架學習、架構師學習等!

    文章有幫助的話,在看,轉發吧。

    謝謝支持喲 (*^__^*)