代码洁癖症的表现

有下列情形之一的，你患上了代码洁癖症。症状程度可轻可重，轻者帮助写出优雅整洁的代码，重者走火入魔，万劫不复。

1. 多余的空行、分号，没有使用的变量，见一个删一个。
2. tab或者空格没有对齐的必须纠正过来，除了缩进用，不允许看到代码内连续两个空格。
3. 看到一个类某个方法没有注释，不由自主地加上，不管有没有意义。
4. 错误的拼写，无论是在命名还是注释必须纠正过来；不一致的大小写，必须要纠正过来；标点符号的遗漏，必须补上。
5. 看到if(a==0)这样的代码必须改成if(0==a)这样的形式。
6. 所有IDE对代码的告警必须消除，无论采取的方式是否有实际意义。
7. 看到赤裸的数字，必须定义成常量，即便数字表意很直观，还是只能接受常量数字。
8. 见不得非静态的公有变量，必须建立get/set方法。
9. 不断地按代码格式整理的快捷键，在Eclipse就是不断地CTRL+Shift+F、CTRL+Shift+O，甚至不住地CTRL+S。
10. 一旦看到超过连续3个的if-else判断分支，就要优化；类似的方法调用代码，如果连续出现，就要优化；超过若干行的方法，必须重构。
11. 最本质的表现，喜欢长时间阅读自己的代码，心中一边啧啧赞赏不已，一边自我陶醉。

文章系本人原创，转载请保持完整性并注明出自《四火的唠叨》

评论

89 楼 kidneyball 2013-02-26  

evanzzy 写道

youarestupid 写道

MrLee23 写道

evanzzy 写道

ifox 写道

看到if(a==0)这样的代码必须改成if(0==a)这样的形式。
这个有神马不同么?

这是从C语言过来的程序员的习惯，就是为了防止写成赋值语句a=0。这是实际出过事儿的，这样写有道理

java真没有这么写的，N多人都说为了避免a=0，难道现在这么多牛逼的人只用记事本编写java代码吗？据我所知，外国大牛都用IDE编写java代码，看来中国大师N多啊。还有IDE如果写成if(a=0)直接编译报错，根本过不去，就别说运行了。所以，这个真不是洁癖，应该算是java的陋习。如果还一直拿C说事，那就当我废话。

多此一举，0==a的写法很难看，不推荐用。

在很久很久以前，C语言是用来控制火箭、飞船、导弹、核wu qi、核潜艇、卫星等战略武器的基本语言，一丁点的差错，都会带来灾难性的后果，因为一个指针没控制好的小问题，让几亿美元打水漂的事情是事实出现过的。所以要以最严谨的态度、最严谨的写法，完全不考虑美观和成本去书写程序逻辑。这种等号前面放常量的做法，只不过是小儿科而已。想多了解些的同志，可以看一下《Effective C++》，《Effective Java》就不要看了，水平和前者差远了。

量子物理学家才是最好的厨师，大家都别看什么菜谱了，水平差太远了，都来研究基本粒子吧。

88 楼 evanzzy 2013-02-26  

runfriends 写道

我一直想知道过多的if-else怎么优化。有很多时候只能使用if-else做判断

过多if-else的首要问题是：到底有多少？是否真的需要优化？

if-else太多，主要是设计问题。

选择结构作为程序语言的三大基本结构，从理论上讲就是无法去除的，也没有必要去除。

我倒是有个C和Java通用的解决办法供你参考：Java里面还有Map，C语言STL模板库里面也有Map，就把你要执行的类放在Map里面作为value，你需要哪个，用key取出来就是了。枚举和switch..case并不会让代码更好看，不是好的解决办法。

如果只有那个十几个if-else，还是可以接受的，把注释写清楚就可以了。

87 楼 evanzzy 2013-02-26  

youarestupid 写道

MrLee23 写道

evanzzy 写道

ifox 写道

看到if(a==0)这样的代码必须改成if(0==a)这样的形式。
这个有神马不同么?

这是从C语言过来的程序员的习惯，就是为了防止写成赋值语句a=0。这是实际出过事儿的，这样写有道理

java真没有这么写的，N多人都说为了避免a=0，难道现在这么多牛逼的人只用记事本编写java代码吗？据我所知，外国大牛都用IDE编写java代码，看来中国大师N多啊。还有IDE如果写成if(a=0)直接编译报错，根本过不去，就别说运行了。所以，这个真不是洁癖，应该算是java的陋习。如果还一直拿C说事，那就当我废话。

多此一举，0==a的写法很难看，不推荐用。

在很久很久以前，C语言是用来控制火箭、飞船、导弹、核wu qi、核潜艇、卫星等战略武器的基本语言，一丁点的差错，都会带来灾难性的后果，因为一个指针没控制好的小问题，让几亿美元打水漂的事情是事实出现过的。所以要以最严谨的态度、最严谨的写法，完全不考虑美观和成本去书写程序逻辑。这种等号前面放常量的做法，只不过是小儿科而已。想多了解些的同志，可以看一下《Effective C++》，《Effective Java》就不要看了，水平和前者差远了。

86 楼 youarestupid 2013-02-26  

七月十五 写道

ifox 写道

看到if(a==0)这样的代码必须改成if(0==a)这样的形式。
这个有神马不同么?

这个必须不同啊，当==敲成=的时候，一个会报错，一个不会报错。

我用netbeans，在nb中，如果if(a==0)写成if(a=0)，IDE自己就会提示错误；
然后，a == 0这种写法符合常人的阅读顺序，看一下现在很多国外开源项目的代码里面，也很多是a == 0的格式；
如果一种代码写法，阅读起来很别扭，那么即使这种写法可能更“严谨”一些，但是我也觉得不应该提倡写“别扭的代码”，这种场景下。

85 楼 alvin198761 2013-02-26  

楼主估计是脑残，有意义么？写完这个，对你有什么提高？

84 楼 七月十五 2013-02-26  

ifox 写道

看到if(a==0)这样的代码必须改成if(0==a)这样的形式。
这个有神马不同么?

这个必须不同啊，当==敲成=的时候，一个会报错，一个不会报错。

83 楼 thoughtbear 2013-02-26  

楼主言过了。
整洁的代码总归是好事，能做好，为什么要做差呢？
我认为，对项目有益的清洁是值得去做的，不执着即可。

82 楼 七月十五 2013-02-26  

有强迫症的人容易患上

81 楼 duoduodeai 2013-02-26  

  你们这些人，真正的技术不去探讨，一些擦边问题却答得津津有味，仿佛置身于大师的幻想中，写自己的代码，管他那么多。

80 楼 huangqibing0626 2013-02-26  

、分号，没有使用的变量，见一个删一个。
tab或者空格没有对齐的必须纠正过来，除了缩进用，不允许看到代码内连续两个空格。
看到一个类某个方法没有注释，不由自主地加上，不管有没有意义。
错误的拼写，无论是在命名还是注释必须纠正过来；不一致的大小写，必须要纠正过来；标点符号的遗漏，必须补上。
看到if(a==0)这样的代码必须改成if(0==a)这样的形式。
所有IDE对代码的告警必须消除，无论采取的方式是否有实际意义。
看到赤裸的数字，必须定义成常量，即便数字表意很直观，还是只能接受常量数字。
见不得非静态的公有变量，必须建立get/set方法。
不断地按代码格式整理的快捷键，在Eclipse就是不断地CTRL+Shift+F、CTRL+Shift+O，甚至不住地CTRL+S。
一旦看到超过连续3个的if-else判断分支，就要优化；类似的方法调用代码，如果连续出现，就要优化；超过若干行的方法，必须重构。
最本质的表现，喜欢长时间阅读自己的代码，心中一边啧啧赞赏不已，一边自我陶醉。

duanxianling 写道

liukai 写道

liukai 写道

    function (parameter) {  
       boolean result = false;

        if( condition == true) {  
      
            //省略若干行代码  
           result  = true ;  
      
        }  
      
        //省略若干行代码  
        return result

标记器不好用啊

function (parameter) {
  return condition == true?true:false;
}
或者干脆：
function (parameter) {
  return condition;
}
干嘛还要定义那么多变量？

function (parameter){return condition == true;}这样如何？

79 楼 elihe2011 2013-02-26  

没有什么强迫不强迫的，写代码的，最基本的格式还是要遵循的，最好多参照一下国外那些老爷爷级别程序员的编码风格，简洁明了，可读性高。if-else结构，if已经return了，还要else干啥。另外，if(常量==变量)，这种结构我发觉国人最喜欢，甚至面试都会问，不知道为啥！反正看一些开源代码，很少出现这种风格！

78 楼 duanxianling 2013-02-26  

liukai 写道

liukai 写道

    function (parameter) {  
       boolean result = false;

        if( condition == true) {  
      
            //省略若干行代码  
           result  = true ;  
      
        }  
      
        //省略若干行代码  
        return result

标记器不好用啊

function (parameter) {
  return condition == true?true:false;
}
或者干脆：
function (parameter) {
  return condition;
}
干嘛还要定义那么多变量？

77 楼 sharew 2013-02-26  

80%都中了····

76 楼 wlxlz 2013-02-26  

第11点中枪了

75 楼 khan 2013-02-26  

楼主还很年轻，里面很多内容都是片面的，害人害己啊！

74 楼 122829827 2013-02-26  

中枪不少啊

73 楼 franciswang 2013-02-26  

细节决定成败，代码的规范和整洁是个好的习惯。上述的很多条是标准，如果不严格要求自己怎么做出好的产品出来。如果这些叫做洁癖的话，不太苟同楼主的意见。反问一句，什么才叫不洁癖，或你认为的正常。

72 楼 gundumw100 2013-02-26  

if语句会使代码变得凌乱，for语句能使代码变得整洁。

71 楼 crud0906 2013-02-26  

中枪N多。。。

70 楼 zps_77 2013-02-26  

很多時候是為了 代碼簡潔，讓人一看就明了等等~  怎麽都說中了。。