免费注册 查看新帖 |

Chinaunix

  平台 论坛 博客 文库
最近访问板块 发新帖
查看: 4589 | 回复: 3
打印 上一主题 下一主题

好的代码里只要一个return语句(转一口水贴) [复制链接]

论坛徽章:
0
跳转到指定楼层
1 [收藏(0)] [报告]
发表于 2011-03-24 09:57 |显示全部楼层 |倒序浏览
本帖最后由 2009从-1开始 于 2011-03-24 09:58 编辑

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

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

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

内容如下:

别再这样写了:

  1. public boolean foo() {
  2. if (true) {
  3.    return true;  
  4. }  else {  
  5. return false;  
  6. }
  7. }
复制代码
每次当我深入某个开源项目,看到大概是某个专家写的、并被有经验的专业人士审查过的这样的代码,我都会惊讶不已,竟然没有人去阻止这个开发者在这个方法里胡乱的放置返回语句。

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

  1. public boolean foo() {  
  2. boolean flag = true;  
  3. if (true) {
  4.    flag=true;
  5. }  
  6. else {
  7.    flag=false;
  8. }
  9. return flag;
  10. }
复制代码
这是Java基本常识。实际上,这不仅是Java基本常识,这是小学水平的Java知识。如果你的方法返回一个值,你应该在方法的开始处把它声明做 一个变量。然后再去做一些赋予这个变量正确意义的操作。然后,在你的最后一行,把这个变量返回给调用程序。这样做不仅仅是为写出好的代码,这是一种有教养 的表现。

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

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

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

论坛徽章:
0
2 [报告]
发表于 2011-03-24 09:59 |显示全部楼层
我不认同这个观点

论坛徽章:
0
3 [报告]
发表于 2011-03-24 10:03 |显示全部楼层
回复 3# jnjn999


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

论坛徽章:
0
4 [报告]
发表于 2011-03-24 10:12 |显示全部楼层
回复 8# hellioncu


    我不认同主要是觉得一些业务的逻辑,用太多ifelse很乱啊
您需要登录后才可以回帖 登录 | 注册

本版积分规则 发表回复

  

北京盛拓优讯信息技术有限公司. 版权所有 京ICP备16024965号-6 北京市公安局海淀分局网监中心备案编号:11010802020122 niuxiaotong@pcpop.com 17352615567
未成年举报专区
中国互联网协会会员  联系我们:huangweiwei@itpub.net
感谢所有关心和支持过ChinaUnix的朋友们 转载本站内容请注明原作者名及出处

清除 Cookies - ChinaUnix - Archiver - WAP - TOP