好的代码里只要一个return语句(转一口水贴)

2009从-1开始

好的代码里只要一个return语句

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

导读：原文根据作者Jasone Tee在theserverside.com上发表的一篇《A return to Good Code》 翻译而来。译文来自外刊IT评论《只要一个返回语句》。

内容如下：

别再这样写了：

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

复制代码

每次当我深入某个开源项目，看到大概是某个专家写的、并被有经验的专业人士审查过的这样的代码，我都会惊讶不已，竟然没有人去阻止这个开发者在这个方法里胡乱的放置返回语句。

请告诉我，把代码写成下面的样子很难吗？

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. }
    

复制代码

这是Java基本常识。实际上，这不仅是Java基本常识，这是小学水平的Java知识。如果你的方法返回一个值，你应该在方法的开始处把它声明做 一个变量。然后再去做一些赋予这个变量正确意义的操作。然后，在你的最后一行，把这个变量返回给调用程序。这样做不仅仅是为写出好的代码，这是一种有教养 的表现。

你是否曾试图修改过一些在方法里到处都是返回语句的程序代码？无从下手。事实上，去维护这样的代码，你第一要做的是重新组织它的结构，让它里面不再有一大 堆的返回语句。这样才能把事情做好。没有任何一个方法是不可以写成只在末尾处有一个的、单一的、易于找到的返回语句的形式的。

的确，烂程序员总有一万个理由来说明他们为什么编写出这样糟糕的程序代码。“我只是为了避免在返回时一堆的多余的条件判断语句。”那好，首先，我告诉你， 计算机中执行一些条件判断语句时是该死的快，你用短路一个方法来节省CPU的一两个指令操作不是显的太荒诞了吗。此外，如果这些所谓多余的条件判断语句最 终没有派上用场的话，这是否是一个有用的信号来说明你的“多余”的代码可能需要重写，也许可以把它们重构成另外一个方法，让它们显的不多余？

关键要说的是，没有任何理由可以为写糟糕的代码或当懒惰的程序员做托辞，特别是当写出好的代码并不是那么困难的情况下。不要在写出里面有成百上千个返回语句的方法了。Java里的方法只可以返回一个值，相应的，一个方法应该有且只有一个返回语句。

2009从-1开始

我不认同这个观点

jnjn999

这样的话，还不如goto呢

2009从-1开始

回复 3# jnjn999


你是支持题目观点吧？
goto还是不同些吧。。

zhaohongjian000

不认同 + 1

按这个观点，break、continue也可以去掉了，全换成flag。

kingyueyang

不认同++

flag不易于阅读代码，在OO语言中。
多出口函数没有必要全盘否决。

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. }
    

复制代码

2009从-1开始

回复 8# hellioncu


    我不认同主要是觉得一些业务的逻辑，用太多ifelse很乱啊

nizvoo

想辦法把函數寫短點。