“恶心代码”帖子中的一个回复-CSDN论坛

⋅Java 24 周岁生日快乐 ⋅【生日祝福】Java 20 岁生日快乐 ⋅【生日祝福】Java 19 岁生日快乐 更多帖子 关注空间博客 火龙果被占用了 本版专家分：114296 红花 2011年3月 Java大版内专家分月排行榜第一 2010年4月 Java大版内专家分月排行榜第一 2010年3月 Java大版内专家分月排行榜第一 2007年8月 Java大版内专家分月排行榜第一 黄花 2011年4月 Java大版内专家分月排行榜第二 2010年8月 Java大版内专家分月排行榜第二 2010年5月 Java大版内专家分月排行榜第二 2008年2月 Java大版内专家分月排行榜第二 2007年7月 Java大版内专家分月排行榜第二 蓝花 2011年2月 Java大版内专家分月排行榜第三 2010年9月 Java大版内专家分月排行榜第三 2008年9月 Java大版内专家分月排行榜第三 2008年1月 Java大版内专家分月排行榜第三 2007年11月 Java大版内专家分月排行榜第三 2007年9月 Java大版内专家分月排行榜第三 结帖率 100%

曾在一个帖子中回复过最恶心的代码，摘抄如下：

public void process(File file) {

    if (file != null) {

        if (file.exists()) {

            if (!file.isDirectory()) {

                if (file.canRead()) {

                    // do something;

                }

            }

        }

    }

}

详见下面这个帖子 23 楼的回复：

你见过最恶心的代码是什么样子的？
http://topic.csdn.net/u/20111020/16/e04cf593-4789-418b-8142-88c231002402.html

本来准备睡觉的，刚才翻了一下这个帖子，得，没法睡了，得发帖子了！

打 23 楼后，回复的同学竟然都异口同声地表明这代码没有问题！

我一直告诫组员，代码中的 for、if 的嵌套不应该超过两层，超过的话得想办法重构。从那个帖子的回复来看，难道是我错了？抑或是我太敏感，还是这个代码真的没有问题？

纯技术帖，谢绝“顶”、“Mark”之类无意义的回复，谢谢配合！

1 2011-10-30 01:30:09

回复数 198 只看楼主引用举报楼主

⋅此用户还没有发表过任何帖子 关注空间博客 zhou__jh 本版专家分：59: 当初老师教的也是这么说的 for  if 循环嵌套不要超过2到3层
有代码执行效率方面和阅读代码方便的考虑吧
以后继续保持

0 2011-10-30 01:41:18

只看TA 引用举报 #1    得分 2

⋅这里比十几年前实在是冷太多 ⋅使用BCB6开发基于ActiveForm的ActiveX控件的事件问题 ⋅使用Delphi开发ActiveX控件的疑问 更多帖子 关注空间博客 薛定谔之猫 本版专家分：6260: 确实有些隐患，一些特殊情况下不执行任何代码就跳过了，在后继的代码中不好界定处于什么状态

0 2011-10-30 01:42:59

只看TA 引用举报 #2 得分 2

⋅遇到bug了 ⋅如果還有明天 ⋅刚看到一段 scala程序：Merry Xmas 更多帖子 关注空间博客 huntor 本版专家分：57683 黄花 2011年12月 Java大版内专家分月排行榜第二 2011年11月 Java大版内专家分月排行榜第二 2010年9月 Java大版内专家分月排行榜第二 蓝花 2011年10月 Java大版内专家分月排行榜第三

import com.google.common.base.Preconditions.checkNotNull;

public void process(File file){

    checkNotNull(file);

    if(file.isFile() && file.canRead(){

        ...

    }

}

1 2011-10-30 01:58:21

只看TA 引用举报 #3 得分 2

⋅面试题：Transactional同时加在类和类中的一个方法，类的其他方法没加，如何只让这个方法的事务生效？ ⋅求助：刚试用期满一个月没签合同，hr说要从说好的18000降成12000才跟我签合同，怎么办 ⋅在分模块开发时，当新增一条记录时，记录的id是在dao层去做还是在service层区做？ 更多帖子 关注空间博客 昵称是可以中文吗 本版专家分：39: 引用 2 楼 mscf 的回复:
确实有些隐患，一些特殊情况下不执行任何代码就跳过了，在后继的代码中不好界定处于什么状态

超过的话得想办法重构，怎么重构法？谢谢

1 2011-10-30 02:03:36

只看TA 引用举报 #4 得分 2

⋅书籍推荐 ⋅散分+技术博客分享 ⋅【散分】玩摄影的程序猿(媛)有吗 更多帖子 关注空间博客 龙四 本版专家分：35901: checkNotNull很多开源项目的做法是Assert.notNull(file,"参数不能为空");

很多项目中都有自己的Assert类

引用 3 楼 huntor 的回复:
Java code
import com.google.common.base.Preconditions.checkNotNull;
public void process(File file){
    checkNotNull(file);
    if(file.isFile() && file.canRead(){
        ...
    }
}

0 2011-10-30 07:04:22

只看TA 引用举报 #5    得分 2

yangyuyu7758 本版专家分：2: 是啊！！是初学者

0 2011-10-30 07:07:51

只看TA 引用举报 #6 得分 2

zaixiankaifa 本版专家分：63: 重构太多会导致混乱的，楼主不要使用这么多嵌套

0 2011-10-30 08:00:53

只看TA 引用举报 #7 得分 2

⋅bat文件突然无法运行了 ⋅为女朋友准备一份特殊的礼物 ⋅为女朋友准备一份特殊的礼物 更多帖子 关注空间博客 孟祥月 本版专家分：2578: 使用太多的嵌套是不好但是有时候需要多层嵌套解决问题更加容易

1 2011-10-30 08:04:47

只看TA 引用举报 #8 得分 2

⋅失业中，散分。。。 更多帖子 关注空间博客 Acesidonu 本版专家分：4595 红花 2012年3月 Web 开发大版内专家分月排行榜第一 黄花 2012年2月 Web 开发大版内专家分月排行榜第二: 嵌套太多了，肯定要重构。

0 2011-10-30 08:30:02

只看TA 引用举报 #9 得分 2

⋅response.setHeader "WWW-Authenticate" 进行身份验证火狐浏览器传中文乱码的问题 ⋅周五咯~~散分咯 ⋅eclipse了，卡在loading workbench就卡死了求解决 更多帖子 关注空间博客 nizhicheng 本版专家分：3669: 支持楼主~~

0 2011-10-30 09:04:59

只看TA 引用举报 #10 得分 2

⋅连接mysql数据库，寻求改进建议 ⋅谁有MFC图形编程与交互绘图基础源码 ⋅组成原理,有资料的给点资料啊 更多帖子 关注空间博客 尘缘udbwcso 本版专家分：4418: 代码中的 for、if 的嵌套三四层应该是可以接受的吧
只是可以接受,尽量不要这样写就好了

0 2011-10-30 09:09:46

只看TA 引用举报 #11 得分 2

⋅Java/Android中如何优雅的写bug？ ⋅提离职以后给安排了很多新活该咋办？ ⋅秒杀的时候怎么样才能防止卖超？ 更多帖子 关注空间博客 若鱼1919 本版专家分：14644



//一般我喜欢这么做

public void process(File file) {

	    if (file == null)return;

	    if(!file.exists())return;

	    if(file.isDirectory())return;

	    if(!file.canRead())return;

	    .....

	    doSomething

	    .....

	}

4 2011-10-30 10:47:05

只看TA 引用举报 #12 得分 2

⋅古典问题：有一对兔子，从出生后第3个月起 ⋅java 在excels中生成圆角矩形 ⋅求一正则表达式 更多帖子 关注空间博客 dxqrr 本版专家分：4170: 嵌套太多不好

0 2011-10-30 11:09:40

只看TA 引用举报 #13 得分 2

⋅正则表达式的使用 ⋅如何生成本地的API帮助文档 ⋅SWT编程-MessageDialog的使用 更多帖子 关注空间博客 蒙眼睛的小妖怪 本版专家分：503: 怎么重构啊？给个例子，莫非就是把几个if弄到一块，弄个&&吗？

引用 4 楼 scauscnu 的回复:
引用 2 楼 mscf 的回复:
确实有些隐患，一些特殊情况下不执行任何代码就跳过了，在后继的代码中不好界定处于什么状态

超过的话得想办法重构，怎么重构法？谢谢

0 2011-10-30 11:57:42

只看TA 引用举报 #15 得分 2

⋅正则表达式的使用 ⋅如何生成本地的API帮助文档 ⋅SWT编程-MessageDialog的使用 更多帖子 关注空间博客 蒙眼睛的小妖怪 本版专家分：503: com.google.common.base.Preconditions.checkNotNull
这个类原来是androidSDK里的啊，终于找到了，谢谢，又开眼界了

0 2011-10-30 12:07:31

只看TA 引用举报 #16 得分 2

⋅求一正则表达式 ⋅关于编程基础的理解 ⋅如何写程序实现DB2大字段数据转移到Oracle中，不用ETL等数据转移工具 更多帖子 关注空间博客 梦如风 本版专家分：988: 大家不要把新手当老手看，他们考虑问题的时候跟我们想的不一样；大家刚出来的时候未必就没写过恶心代码，随着经验的增长，再看以前写的代码，总觉得以前自己好白痴。

0 2011-10-30 12:39:06

只看TA 引用举报 #17 得分 2

⋅此用户还没有发表过任何帖子 关注空间博客 star_splendid 本版专家分：244: 哈哈~如果只在最里面的一层写处理代码，感觉还可以啊~
如果每层if 都有不同处理话~那就要恶心死了~ 12L的方法挺有意思的，可以借鉴啊~

0 2011-10-30 12:51:33

只看TA 引用举报 #18 得分 2

⋅遇到bug了 ⋅如果還有明天 ⋅刚看到一段 scala程序：Merry Xmas 更多帖子 关注空间博客 huntor 本版专家分：57683 黄花 2011年12月 Java大版内专家分月排行榜第二 2011年11月 Java大版内专家分月排行榜第二 2010年9月 Java大版内专家分月排行榜第二 蓝花 2011年10月 Java大版内专家分月排行榜第三: 引用 16 楼 yi_yuan 的回复:
com.google.common.base.Preconditions.checkNotNull
这个类原来是androidSDK里的啊，终于找到了，谢谢，又开眼界了

google code 上有单独的 google guava

0 2011-10-30 12:59:14

只看TA 引用举报 #19 得分 2

liango 本版专家分：1520: 一声叹息

0 2011-10-30 15:13:17

只看TA 引用举报 #20 得分 2

⋅最近回到CSDN 我发觉CSDN重新定义了404的含义 ⋅好奇的问下现在在非技术区发宣传QQ群的贴也违反你们的规定么? ⋅散分另寻找在Java开发中使用过MongoDB的童鞋 更多帖子 关注空间博客 BearKin 本版专家分：15122 黄花 2010年3月 Java大版内专家分月排行榜第二: 其实嵌套几层这个我觉得没什么硬性规定虽然我也不喜欢嵌套很多层(最近正在用while和break将原本的嵌套给展开) 可有些时候这样写是必要的比如上面的代码在不满足条件的时候要进行一些处理而满足条件还要做其他验证的情况下就要这么做

不过这段代码倒是没必要因为咋看他都可以写到一个IF里去..

0 2011-10-30 16:31:50

只看TA 引用举报 #22 得分 2

scrack 本版专家分：1447: 用断言学习了

0 2011-10-30 16:47:05

只看TA 引用举报 #23 得分 2

a78782295 本版专家分：10: 我也有这样的逻辑，随着学习的深入，以后可能会改进吧。。。虽然也知道这样写太“恶心”。。

0 2011-10-30 17:46:57

只看TA 引用举报 #24 得分 2

⋅有木有人做过类似打车类的app啊 ⋅打车app开发 ⋅打车app开发，报价 更多帖子 关注空间博客 hzc543806053 本版专家分：2074: 还是要注意代码规范，以后得要多看看好的代码

0 2011-10-30 18:15:40

只看TA 引用举报 #25 得分 2

弑忆 本版专家分：222: 嗯，嵌套少一点看上去优雅一点

0 2011-10-30 18:20:26

只看TA 引用举报 #26 得分 2

⋅Rust挺不错的，对C++不满的兄弟们可以看看了 ⋅【悲剧？】 ⋅哪位能介绍一下C++14, 我顺便就给未来7OR9天长假散粉了 更多帖子 关注空间博客 healer_kx 本版专家分：25939 黄花 2006年2月 C/C++大版内专家分月排行榜第二: 代码很严谨，表述很清晰，不是吗？几层真的是一个绝对的指标吗？

清晰和严谨才是写代码的宗旨，至于几层，什么的，那些事具体的实现方式，方式不能太死板了。

1 2011-10-30 18:31:06

只看TA 引用举报 #27 得分 2

heyongyin 本版专家分：48: 使用swich case.....

0 2011-10-30 18:32:22

只看TA 引用举报 #28 得分 2

⋅Rust挺不错的，对C++不满的兄弟们可以看看了 ⋅【悲剧？】 ⋅哪位能介绍一下C++14, 我顺便就给未来7OR9天长假散粉了 更多帖子 关注空间博客 healer_kx 本版专家分：25939 黄花 2006年2月 C/C++大版内专家分月排行榜第二: 引用 12 楼 goldenfish1919 的回复:
Java code

//一般我喜欢这么做
public void process(File file) {
        if (file == null)return;
        if(!file.exists())return;
        if(file.isDirectory())return;
        if(!file.canRead())return;
   ……

这个是我常用的一种办法，解决楼主遇到的问题，“防御式编程“。

0 2011-10-30 18:35:29

只看TA 引用举报 #29    得分 2

⋅求教存储过程调用数据库生成脚本如何实现? ⋅【内存泄露】循环多次创建ResultSet,Connection后，如何全部关闭疑问？？？ ⋅jfreechart 如何改变X轴标签位置？ 更多帖子 关注空间博客 SimonYeung 本版专家分：502



if (file != null && file.exists() && !file.isDirectory() && file.canRead())

0 2011-10-30 18:58:04

只看TA 引用举报 #31 得分 2

⋅Rust挺不错的，对C++不满的兄弟们可以看看了 ⋅【悲剧？】 ⋅哪位能介绍一下C++14, 我顺便就给未来7OR9天长假散粉了 更多帖子 关注空间博客 healer_kx 本版专家分：25939 黄花 2006年2月 C/C++大版内专家分月排行榜第二: 引用 31 楼 simonyeung 的回复:
Java code

if (file != null && file.exists() && !file.isDirectory() && file.canRead())

我很难在10秒钟内读懂

1 2011-10-30 19:17:36

只看TA 引用举报 #32 得分 2

⋅Java 24 周岁生日快乐 ⋅【生日祝福】Java 20 岁生日快乐 ⋅【生日祝福】Java 19 岁生日快乐 更多帖子 关注空间博客 火龙果被占用了 本版专家分：114296 红花 2011年3月 Java大版内专家分月排行榜第一 2010年4月 Java大版内专家分月排行榜第一 2010年3月 Java大版内专家分月排行榜第一 2007年8月 Java大版内专家分月排行榜第一 黄花 2011年4月 Java大版内专家分月排行榜第二 2010年8月 Java大版内专家分月排行榜第二 2010年5月 Java大版内专家分月排行榜第二 2008年2月 Java大版内专家分月排行榜第二 2007年7月 Java大版内专家分月排行榜第二 蓝花 2011年2月 Java大版内专家分月排行榜第三 2010年9月 Java大版内专家分月排行榜第三 2008年9月 Java大版内专家分月排行榜第三 2008年1月 Java大版内专家分月排行榜第三 2007年11月 Java大版内专家分月排行榜第三 2007年9月 Java大版内专家分月排行榜第三: 引用 29 楼 healer_kx 的回复:
引用 12 楼 goldenfish1919 的回复:

Java code

//一般我喜欢这么做
public void process(File file) {
if (file == null)return;
if(!file.exists())return;
if(file.isDirectory())return;
if(!file.canRead())return……

我也一般使用这样的方法，呵呵。

当然了，我在这里贴的只是个示例，实际上每一层中可能有很多代码，每个 if 可能还有 else 语句块，如果加上这些的话，这代码就没有那么好看了。

0 2011-10-30 19:26:43

只看TA 引用举报 #33 得分 0

⋅问大家一个SQL,两个表联查，一对多，多的取最新记录 ⋅正则实现EL表达式的替换 ⋅闷，散分！！！京东团购两周了，啥也不说了。 更多帖子 关注空间博客 wingson_shen 本版专家分：2302: 我想知道，还有哪些更好的解决之道呢。

我也常写LZ这样的烂代码。

0 2011-10-30 19:41:17

只看TA 引用举报 #34 得分 2

空心兜兜 本版专家分：1665 红花 2010年7月硬件使用大版内专家分月排行榜第一 2010年6月硬件使用大版内专家分月排行榜第一 2009年4月 Windows专区大版内专家分月排行榜第一 2009年3月 Windows专区大版内专家分月排行榜第一 2009年3月硬件使用大版内专家分月排行榜第一 2009年2月硬件使用大版内专家分月排行榜第一 2009年1月硬件使用大版内专家分月排行榜第一 黄花 2010年8月硬件使用大版内专家分月排行榜第二 蓝花 2010年7月 Windows专区大版内专家分月排行榜第三 2009年1月 Windows专区大版内专家分月排行榜第三 2008年12月 Windows专区大版内专家分月排行榜第三 2006年8月扩充话题大版内专家分月排行榜第三: 也许写的时候觉得可以理解，但后面绝对会疑惑自己为什么这样写

0 2011-10-30 20:12:43

只看TA 引用举报 #36 得分 2

⋅struts2+spring3+mybatis的JAR包整合问题 ⋅Extjs学习 ⋅各位大神们，姓王的男孩该叫什么名字才有创意呢 更多帖子 关注空间博客 24K純帥 本版专家分：16266: 我个人感觉if嵌套太多层了，12L的写法比较正常

0 2011-10-30 21:19:55

只看TA 引用举报 #37 得分 2

巴顿将军626 本版专家分：320: 帮忙顶一下。

0 2011-10-30 21:23:16

只看TA 引用举报 #38 得分 2

⋅Java+SQLite的运行问题 ⋅求帮助，错误ORA-47390该如何解决 ⋅java项目中连接Oracle数据库时，出现ClassNotFoundException 更多帖子 关注空间博客 sarahpanee 本版专家分：2

在性能一样的情况下，当然选择可读性高的表现方式了！
如果我的话，选下面这种写法：



public void process(File file) {

    if (file != null && file.exists() && !file.isDirectory() && file.canRead()){

       // do something;

    }

}

0 2011-10-30 21:23:45

只看TA 引用举报 #39 得分 2

⋅lsof 问题

⋅lsof 问题

⋅创业产品选择App还是微信公众平台？

更多帖子

关注空间博客

取号啦李刚

本版专家分：62305

GitHub 绑定GitHub第三方账户获取

银牌 2013年11月总版技术专家分月排行榜第二

铜牌 2013年10月总版技术专家分月排行榜第三

红花 2014年3月 Java大版内专家分月排行榜第一
2014年1月 Java大版内专家分月排行榜第一
2013年12月 Java大版内专家分月排行榜第一
2013年11月 Java大版内专家分月排行榜第一
2013年10月 Java大版内专家分月排行榜第一: 一般null和""放一个条件里。
其他都是
if(xxx)xxxx;
if(xxx)xxxx;
if(xxx)xxxx;
if(xxx)xxxx;
...

0 2011-10-30 21:29:07

只看TA 引用举报 #40 得分 2

⋅blog.csdn.net应该是被黑了 ⋅管理员，你好，我的博客提示关闭，能恢复么 ⋅散分 CSDN有问题，经常打开帖子就导致页面死了，很有可能是兼容性问题 更多帖子 关注空间博客 oO临时工Oo 本版专家分：18982 蓝花 2011年12月 Java大版内专家分月排行榜第三: 这个确实比较恶心，但这决不是我见过最恶心的，至少看代码就知道程序员写来干啥的，逻辑性差点，但至少可阅读。

0 2011-10-30 21:34:58

只看TA 引用举报 #41 得分 2

YHL27 本版专家分：83: 有所收获哦。。

0 2011-10-30 21:47:45

只看TA 引用举报 #42 得分 2

⋅csdn怎么老上不去？ ⋅下班路上趣事 ⋅这几天怎么登陆不上csdn了？ 更多帖子 关注空间博客 yrnaaa 本版专家分：32: 看了上面的说的方法，大概有两种，
1. if（）return；
2. if（。。。。。）所有条件&&到一起。
针对上面这个问题，用1比2更好些，至少我不太喜欢看太长的语句。

同样期待楼主给出更好的方法。。。。。。

0 2011-10-30 22:52:57

只看TA 引用举报 #43 得分 2

lfp001 本版专家分：684: 期待LZ的方法...

0 2011-10-30 23:27:22

只看TA 引用举报 #44 得分 2

⋅面试题：Transactional同时加在类和类中的一个方法，类的其他方法没加，如何只让这个方法的事务生效？ ⋅求助：刚试用期满一个月没签合同，hr说要从说好的18000降成12000才跟我签合同，怎么办 ⋅在分模块开发时，当新增一条记录时，记录的id是在dao层去做还是在service层区做？ 更多帖子 关注空间博客 昵称是可以中文吗 本版专家分：39: 引用 43 楼 yrnaaa 的回复:
看了上面的说的方法，大概有两种，
1. if（）return；
2. if（。。。。。）所有条件&&到一起。
针对上面这个问题，用1比2更好些，至少我不太喜欢看太长的语句。

同样期待楼主给出更好的方法。。。。。。

恩恩，第一比较工整

0 2011-10-30 23:32:37

只看TA 引用举报 #45 得分 2

⋅删除我的资源一下 ⋅为什么现在上传资源都显示undefined ⋅个人中心为什么没有访问我的博客的入口 更多帖子 关注空间博客 coooliang 本版专家分：4336: 引用 33 楼 bao110908 的回复:
引用 29 楼 healer_kx 的回复:

引用 12 楼 goldenfish1919 的回复:

Java code

//一般我喜欢这么做
public void process(File file) {
if (file == null)return;
if(!file.exists())return;
if(file.isDirectory())return;
……

这个帖子我也回了，我的方法是用并且。
楼主一般使用return，是不是可读性好？

0 2011-10-30 23:45:25

只看TA 引用举报 #46 得分 2

神探狄仁杰 本版专家分：1547: 一声叹息。

与我一样的小小菜鸟，须多磨练、磨练！！！！！！！！！！！！！！！！！

0 2011-10-31 00:03:10

只看TA 引用举报 #47 得分 2

⋅回来充电 ⋅【逗逗】专场 ⋅果果，小基说你快升星了吧 更多帖子 关注空间博客 lyhmy 本版专家分：1540: 能少嵌套一些固然是好，可是有时候的逻辑还是需要多重判断

代码审核吧，我有时没事了就喜欢拿自己以前写的代码来审核，虽然很多时候对自己写的不是容易就检测出问题，但是有时还是可以发现存在许多的问题，对规范啊什么的都有好处

0 2011-10-31 00:34:59

只看TA 引用举报 #48 得分 2

⋅冒个泡...... ⋅为什么把我的帖子移到非技术区？ ⋅软件架构设计包括哪些内容？ 更多帖子 关注空间博客 缪军 本版专家分：5 版主: 楼主如何去检查这些提交的代码，检查成千上万的代码是否符合那些“规范”？
换一种思路，你就不会认为这样的代码有何不妥：
1.已通过测试作为提交工单的标准；（而不是代码是否“漂亮”）；
2.程序员追求的是用最快的速度通过测试，这样单位时间他们可以去领更多的工单；
3.给没有足够经验的程序员一些建议，帮助他们少走一些弯路；

0 2011-10-31 00:38:23

只看TA 引用举报 #49 得分 2

⋅时隔三年回到CSDN怀抱，感谢那些乐于助人的大神们，鞠躬 ⋅求大神解惑，在线等 ⋅求大神帮忙 更多帖子 关注空间博客 和永恒有關 本版专家分：4: 引用 32 楼 healer_kx 的回复:
引用 31 楼 simonyeung 的回复:

Java code

if (file != null &amp;&amp; file.exists() &amp;&amp; !file.isDirectory() &amp;&amp; file.canRead())

我很难在10秒钟内读懂

这也不至于10秒读不懂吧

0 2011-10-31 08:41:03

只看TA 引用举报 #50 得分 2

“恶心代码”帖子中的一个回复 [问题点数：300分，结帖人bao110908]