哲思茶馆  - 讨论区

标题:好的代码里只要一个返回语句

2011年03月23日 星期三 07:59

别再这样写了:

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里的方法只可以返回一个值,相应的,一个方法应该有且只有一个返回语句。

本文是从 A return to Good Code 这篇文章翻译而来。

来源: 外刊IT评论 http://www.aqee.net/ 

2011年03月23日 星期三 09:06

学习了 我还真的是一直都写多个return  汗颜啊

2011年03月23日 星期三 09:16

这个不能一概而论。

2011年03月23日 星期三 09:26

不会吧?你说的不合理!

假如你在函数的第一个判断语句就需要返回了,而后面的程序还有几百行。你是不是要用goto跳到最后一行return flag? 那你的程序将到处是goto。

在函数中使用多个 return TRUE; return FALSE; 的例子到处可见。

2011年03月23日 星期三 11:20

我觉得3楼说的靠谱……不能一概而论吧……

2011年03月23日 星期三 15:22

提前return 也是好的习惯。
如果条件不满足,直接返回,何必弄那么多 if-elseif-else.
关键问题是当return的时候,是否进行了收尾工作,
比如类状态是否一致,该清理的是否清理,该还原的是否还原。

2011年03月23日 星期三 19:08

第二个算法比第一个多了1个push,三个mov,而且都是针对内存的。

过去,cpu访问内存需要6个时钟周期获得总线,现在呢?

处理器都当编译器是SB,编译器都当程序猿是SB,程序猿呢?

Java就是脱了裤子放P.   <-14年前的结论.


 

 

 

如下红色区域有误,请重新填写。

    你的回复:

    请 登录 后回复。还没有在Zeuux哲思注册吗?现在 注册 !

    Zeuux © 2024

    京ICP备05028076号