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