從list.isEmpty與list.size()==0引發的爭論
公司里有4個員工,分別為CTO技術負責人老C,架構師小賈,高級開發工程師小高,開發工程師小開.
有一天,針對一段代碼,小賈,小高,小開起了爭執:
List<Method> methods = interfaze.getMethods();
if (changeMethods.size() == 0){
...
}
小高: ?一看就是新人的代碼，好low.判斷集合是否為空集合，用isEmpty.
小賈: ?有個工具類:
public static boolean isEmpty(Collection coll) {
return (coll == null || coll.isEmpty());
}
小高:?一般來說判斷一個集合是否為empty,要先判斷集合是否為空,而后是否為empty,工具類只調用一次,書寫簡單.
小賈: ?嗯,同時風險也降低了,不會手抖時候把==0寫成==1.
小開一臉茫然: ?這代碼不是我寫的啊.
老C這時也過來了: ?嗯,我們先看看這個代碼在哪里被調用吧,我們看到,interfaze.getMethods()肯定不會返回null 嗯,我們再看看集合類的isEmpty():
public boolean isEmpty() {
return size == 0;
}
小開:?咦,他也是size == 0的判斷呀.
小高: ?優秀的代碼就是短小精悍,通俗易懂,直接==0也太low了.
小賈: ?告誡你們一下,在使用方法傳遞的參數時,一定要判斷是否為null,因為你不知道這個方法會不會會被其他人改寫,是不是會有null.
小開: ?這個代碼是核心代碼,怎么會被別人隨便改呢,就算改,也要有一定的規則啊,原來返回個空對象,現在非要返回null,含義也不一樣,說不通啊.
老C: ?其實從形式上看,兩者并沒有什么不同,代碼是給人看的,能看懂,能跑,那基本要求就達到了.
?????????從效率上看,直接使用size==0,少一個判斷,那么累加起來,對于執行效率的提升也是有一定幫助的,但是,如果后期有人變更了方法參數,使其為null,那么代碼就防護不了了.如果要這樣用,那么就一定要強制遵循規范,返回對象可以是空對象,但絕不能是null.
????????? 所以從代碼防護角度考慮,我們可以用工具類的isEmpty(),但是呢,知道他的具體實現后,是不是就一定要用他呢.也不一定,如果你有絕對的掌控力,那就沒問題.不管怎么用.,沒有low不low的,合適的就是好的.
??這個,也反應了一個代碼掌控力,知道怎么用,在哪里用,更知道為什么這樣用!
boss:?看你們今天活力四射啊,今天聚餐,走起!哦,對了,站起來,要慢一點,別急.
