|
|
本文是從 A return to Good Code 這篇文章翻譯而來。
別再這樣寫了：
public boolean foo(){
if (true){
return true;
} else {
return false;
}
}
每次當我深入某個開源項目，看到大概是某個專家寫的、并被有經驗的專業人士審查過的這樣的代碼，我都會驚訝不已，竟然沒有人去阻止這個開發者在這個方法里胡亂的放置返回語句。
請告訴我，把代碼寫成下面的樣子很難嗎？
public boolean foo(){
boolean flag = true;
if (true){
flag=true;
}
else {
flag=false;
}
return flag;
}
這是Java基本常識。實際上，這不僅是Java基本常識，這是小學水平的Java知識。如果你的方法返回一個值，你應該在方法的開始處把它聲明做一個變量。然后再去做一些賦予這個變量正確意義的操作。然后，在你的最后一行，把這個變量返回給調用程序。這樣做不僅僅是為寫出好的代碼，這是一種有教養的表現。
你是否曾試圖修改過一些在方法里到處都是返回語句的程序代碼？無從下手。事實上，去維護這樣的代碼，你第一要做的是重新組織它的結構，讓它里面不再有一大堆的返回語句。這樣才能把事情做好。沒有任何一個方法是不可以寫成只在末尾處有一個的、單一的、易于找到的返回語句的形式的。
的確，爛程序員總有一萬個理由來說明他們為什么編寫出這樣糟糕的程序代碼。“我只是為了避免在返回時一堆的多余的條件判斷語句。”那好，首先，我告訴你，計算機中執行一些條件判斷語句時是該死的快，你用短路一個方法來節省CPU的一兩個指令操作不是顯的太荒誕了嗎。此外，如果這些所謂多余的條件判斷語句最終沒有派上用場的話，這是否是一個有用的信號來說明你的“多余”的代碼可能需要重寫，也許可以把它們重構成另外一個方法，讓它們顯的不多余？
關鍵要說的是，沒有任何理由可以為寫糟糕的代碼或當懶惰的程序員做托辭，特別是當寫出好的代碼并不是那么困難的情況下。不要在寫出里面有成百上千個返回語句的方法了。Java里的方法只可以返回一個值，相應的，一個方法應該有且只有一個返回語句。
鄭重聲明：本文版權歸原作者所有，轉載文章僅為傳播更多信息之目的，如作者信息標記有誤，請第一時間聯系我們修改或刪除，多謝。
