代码洁癖症的表现

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

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. 最本质的表现，喜欢长时间阅读自己的代码，心中一边啧啧赞赏不已，一边自我陶醉。

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

评论

49 楼 wangshu3000 2013-02-25  

这是公司的要求，除了最后一条。。 

48 楼 rox 2013-02-25  

搞IT的，多半都会有些强迫症。

47 楼 lg_asus 2013-02-25  

Nightlee 写道

witcheryne 写道

中了不少抢...

多层 if 嵌套也会受不了.
想都不想就用 if-else 也受不了.

function (parameter) {

    if( condition == true) {

        //省略若干行代码
        return a;

    } else {

        //省略若干行代码
        return b;
    }

}

我就特么的想改成:

function (parameter) {

    if( condition == true) {

        //省略若干行代码
        return a;

    }

    //省略若干行代码
    return b;

}

这个写法其实有弊端，代码中间的return相当于变相的goto，破坏了代码的顺序，还是喜欢if-else里面赋值最后再return

这种写法叫做guard clause

46 楼 lazynote 2013-02-25  

if/else有时候去掉还是挺难的。我知道唯一的方式就是改成接口编程。if变成switch实质还是没有变化。

45 楼 xchd 2013-02-25  

这是洁癖吗？我基本中枪。是不是要改正？

44 楼 lg_asus 2013-02-25  

除了最后一条，其它都是good habbits

43 楼 youarestupid 2013-02-25  

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的写法很难看，不推荐用。

42 楼 youarestupid 2013-02-25  

RayChase 写道

kidneyball 写道

runfriends 写道

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

如果每个分支做的事情很简单且分支不大可能增加可以改为switch。如果每个分支做的事情比较多，或者将来有可能增加分支可以改为责任链模式。

评论里说的对于过多if else优化的方式都不对，至少是很片面的，我去写一篇怎么重构过多if else的文章。

应该是策略模式。

41 楼 haohao-xuexi02 2013-02-25  

我觉得吧，最后一条总结非常好。。。很多码奴的自恋症表现。

40 楼 RayChase 2013-02-25  

bat0906 写道

楼主 做对日的吧

不是 但是我很想知道你这个判断的根据是什么？

39 楼 RayChase 2013-02-25  

kidneyball 写道

runfriends 写道

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

如果每个分支做的事情很简单且分支不大可能增加可以改为switch。如果每个分支做的事情比较多，或者将来有可能增加分支可以改为责任链模式。

评论里说的对于过多if else优化的方式都不对，至少是很片面的，我去写一篇怎么重构过多if else的文章。

38 楼 wwwcomy 2013-02-25  

中了1,2,4,6,8,9

好像已经挺严重了。。

37 楼 llh110220 2013-02-25  

如果一条都没有的人，90%的概率不是好程序员

36 楼 archy123 2013-02-25  

最本质的表现，喜欢长时间阅读自己的代码，心中一边啧啧赞赏不已，一边自我陶醉。

35 楼 taoge2121 2013-02-25  

基本全中……

34 楼 kidneyball 2013-02-25  

runfriends 写道

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

如果每个分支做的事情很简单且分支不大可能增加可以改为switch。如果每个分支做的事情比较多，或者将来有可能增加分支可以改为责任链模式。

33 楼 white_crucifix 2013-02-24  

我次奥，最后一点大赞！

32 楼 fnet 2013-02-24  

我确实有代码洁癖，
我感觉这很不好，是一种神经病。

31 楼 MrLee23 2013-02-24  

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说事，那就当我废话。

30 楼 leon709 2013-02-24  

还是是比较重的，原来这叫洁癖的哦