【轉】【Stackoverflow好問題】去掉煩人的「!=null"(判空語句)

【Stackoverflow好問題】去掉煩人的「!=null"(判空語句)

問題

爲了不空指針調用，咱們常常會看到這樣的語句

1.  
   ... if (someobject != null) {
2.  
   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給我作甚"。

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

(1)assert語句，你能夠把錯誤緣由放到assert的參數中，這樣不只能保護你的程序不往下走，並且還能把錯誤緣由返回給調用方，豈不是一箭雙鵰。（原文介紹了assert的使用，這裏省略）

(2)也能夠直接拋出空指針異常。上面說了，此時null是個不合理的參數，有問題就是有問題，就應該大大方方往外拋。

 

第1種狀況會更復雜一些。

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

這裏給一些實踐建議：

一、假如方法的返回類型是collections，當返回結果是空時，你能夠返回一個空的collections（empty list),而不要返回null，這樣調用側就能大膽地處理這個返回，例如調用側拿到返回後，能夠直接print list.size()，又無需擔憂空指針問題。（什麼？想調用這個方法時，不記得以前實現該方法有沒按照這個原則？因此說，代碼習慣很重要！若是你養成習慣，都是這樣寫代碼（返回空collections而不返回null)，你調用本身寫的方法時，就能大膽地忽略判空）

二、返回類型不是collections，又怎麼辦呢？

那就返回一個空對象（而非null對象），下面舉個「栗子」，假設有以下代碼

1.  
   public interface Action {
2.  
   void doSomething();}
3.  
    
4.  
   public interface Parser {
5.  
   Action findAction(String userInput);}

其中，Parse有一個接口FindAction，這個接口會依據用戶的輸入，找到並執行對應的動做。假如用戶輸入不對，可能就找不到對應的動做（Action），所以findAction就會返回null，接下來action調用doSomething方法時,就會出現空指針。

解決這個問題的一個方式，就是使用Null Object pattern（空對象模式）

 

咱們來改造一下

類定義以下，這樣定義findAction方法後，確保不管用戶輸入什麼，都不會返回null對象

1.  
   public class MyParser implements Parser {
2.  
   private static Action DO_NOTHING = new Action() {
3.  
   public void doSomething() { /* do nothing */ }
4.  
   };
5.  
    
6.  
   public Action findAction(String userInput) {
7.  
   // ...
8.  
   if ( /* we can't find any actions */ ) {
9.  
   return DO_NOTHING;
10.  
    }
11.  
    }}

 

對比下面兩份調用實例

一、冗餘:每獲取一個對象，就判一次空

1.  
   Parser parser = ParserFactory.getParser();
2.  
   if (parser == null) {
3.  
   // now what?
4.  
   // this would be an example of where null isn't (or shouldn't be) a valid response
5.  
   }
6.  
   Action action = parser.findAction(someInput);
7.  
   if (action == null) {
8.  
   // do nothing} else {
9.  
   action.doSomething();}

二、精簡

ParserFactory.getParser().findAction(someInput).doSomething();

由於不管什麼狀況，都不會返回空對象，所以經過findAction拿到action後，能夠放心地調用action的方法。

 

 

 

其餘回答精選：

一、若是要用equal方法，請用object<不可能爲空>.equal(object<可能爲空>))

例如：

使用

"bar".equals(foo) 

而不是

foo.equals("bar") 

二、Java8或者guava lib中，提供了Optional類，這是一個元素容器，經過它來封裝對象，能夠減小判空。不過代碼量仍是很多。不爽。

三、若是你想返回null，請挺下來想想，這個地方是否更應該拋出一個異常

 

stackoverflow連接：

http://stackoverflow.com/questions/271526/avoiding-null-statements-in-java?page=2&tab=votes#tab-top

 

專欄介紹：

很是喜歡stackoverflow，總能在上面找到疑難雜症的解決辦法。偶然發現該網站有一個熱度榜單。因而精選了熱度較高的一些問題，閱讀每一個問題的全部答案，而後按照本身的理解梳理出來。但願能把上面的討論，用更精簡，更適當的方式展示給你們。

如需轉載，請註明原文地址

http://blog.csdn.net/lizeyang