我不明白

Question

我正在審查一些Java代碼與這條線

if (list.size() == 1 || list.size() <= 4) {...} 

我評論，我看不出這是剛剛做

if (list.size() <= 4) {...} 

他說有什麼不同的區別它很重要，需要成爲第一個。我不明白。也許，如果它是像

if (list.size() == 1 || list.size() <= someVeryCostlyFunction()) {...} 

和規模預計爲1的大部分時間，你可能同時使用，如果someVeryCostlyFunction（）總是返回一些正數> = 1。否則我看不到區別。我們應該檢查效率。

Answer 1

代碼smells與兩個條件：

if (list.size() == 1 || list.size() <= 4) 

也許筆者有幾點：

如果在4以下列表中的元素一定數目。

即使列表中有零個元素，哪個最有可能是錯誤的，它還可以滿足條件。

此情況的另一個問題是使用magic number 4？
關於它的重要性以及爲什麼它不是5？

int MAX_HANDLED = 4; 
if (list.size() > 0 && list.size() <= MAX_HANDLED) 
: 
: 
int ALL_TIRES = 4; 
if (car.getTires() < ALL_TIRES) { 
    car.stop(); 
} 

至於性能，我看不出有任何顯著原因，現有的條件應該是更快了，然後你提出一個（甚至： 應當自來自可能出現在代碼中的其他4的記載和尊貴第二個會更快，當list.size > 1）。看到類似關注的這個question。