好的代碼里只要一個return語句(轉(zhuǎn)一口水貼)

2009從-1開始

好的代碼里只要一個return語句

http://sd.csdn.net/a/20110323/294448.html

導(dǎo)讀：原文根據(jù)作者Jasone Tee在theserverside.com上發(fā)表的一篇《A return to Good Code》 翻譯而來。譯文來自外刊IT評論《只要一個返回語句》。

內(nèi)容如下：

別再這樣寫了：

1. 
   
2. public boolean foo() {
   
3. if (true) {
   
4.    return true;  
   
5. }  else {  
   
6. return false;  
   
7. }
   
8. }
   

復(fù)制代碼

每次當(dāng)我深入某個開源項(xiàng)目，看到大概是某個專家寫的、并被有經(jīng)驗(yàn)的專業(yè)人士審查過的這樣的代碼，我都會驚訝不已，竟然沒有人去阻止這個開發(fā)者在這個方法里胡亂的放置返回語句。

請告訴我，把代碼寫成下面的樣子很難嗎？

1. 
   
2. public boolean foo() {  
   
3. boolean flag = true;  
   
4. if (true) {
   
5.    flag=true;
   
6. }  
   
7. else {
   
8.    flag=false;
   
9. }
   
10. return flag;
    
11. }
    

復(fù)制代碼

這是Java基本常識。實(shí)際上，這不僅是Java基本常識，這是小學(xué)水平的Java知識。如果你的方法返回一個值，你應(yīng)該在方法的開始處把它聲明做 一個變量。然后再去做一些賦予這個變量正確意義的操作。然后，在你的最后一行，把這個變量返回給調(diào)用程序。這樣做不僅僅是為寫出好的代碼，這是一種有教養(yǎng) 的表現(xiàn)。

你是否曾試圖修改過一些在方法里到處都是返回語句的程序代碼？無從下手。事實(shí)上，去維護(hù)這樣的代碼，你第一要做的是重新組織它的結(jié)構(gòu)，讓它里面不再有一大 堆的返回語句。這樣才能把事情做好。沒有任何一個方法是不可以寫成只在末尾處有一個的、單一的、易于找到的返回語句的形式的。

的確，爛程序員總有一萬個理由來說明他們?yōu)槭裁淳帉懗鲞@樣糟糕的程序代碼。“我只是為了避免在返回時一堆的多余的條件判斷語句。”那好，首先，我告訴你， 計(jì)算機(jī)中執(zhí)行一些條件判斷語句時是該死的快，你用短路一個方法來節(jié)省CPU的一兩個指令操作不是顯的太荒誕了嗎。此外，如果這些所謂多余的條件判斷語句最 終沒有派上用場的話，這是否是一個有用的信號來說明你的“多余”的代碼可能需要重寫，也許可以把它們重構(gòu)成另外一個方法，讓它們顯的不多余？

關(guān)鍵要說的是，沒有任何理由可以為寫糟糕的代碼或當(dāng)懶惰的程序員做托辭，特別是當(dāng)寫出好的代碼并不是那么困難的情況下。不要在寫出里面有成百上千個返回語句的方法了。Java里的方法只可以返回一個值，相應(yīng)的，一個方法應(yīng)該有且只有一個返回語句。

2009從-1開始

我不認(rèn)同這個觀點(diǎn)

jnjn999

這樣的話，還不如goto呢

2009從-1開始

回復(fù) 3# jnjn999


你是支持題目觀點(diǎn)吧？
goto還是不同些吧。。

zhaohongjian000

不認(rèn)同 + 1

按這個觀點(diǎn)，break、continue也可以去掉了，全換成flag。

kingyueyang

不認(rèn)同++

flag不易于閱讀代碼，在OO語言中。
多出口函數(shù)沒有必要全盤否決。

hellioncu

我還是比較支持的

zhangsuozhu

我喜歡的方法如下兩種：

1. 
   
2. function ()
   
3. {
   
4.     if(  xxx==false)
   
5.               return -1;
   
6.     if ( yyyy == false)
   
7.               return -1;
   
8.    
   
9.      ........
   
10. 
    
11.      return 0;
    
12. 
    
13. }
    
14. 
    
15. function ()
    
16. {
    
17.     if (xxxx==false)
    
18.         goto false_process;
    
19.     if (yyyy==false)
    
20.         goto false_process;
    
21.     .....
    
22.     return 0;
    
23. false_process:
    
24.     .......
    
25.     return -1;
    
26. }
    

復(fù)制代碼

2009從-1開始

回復(fù) 8# hellioncu


    我不認(rèn)同主要是覺得一些業(yè)務(wù)的邏輯，用太多ifelse很亂啊

nizvoo

想辦法把函數(shù)寫短點(diǎn)。