代码洁癖症的表现

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

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

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

评论

29 楼 yuechen323 2013-02-24  

全中，呵呵
从普通人到卓越就是这些一个个的细节积累而成的，我不觉得有什么不好，起码是对自己工作的尊重，而且是不会耽误太多的时间的，可想而知做任何事情都是很用心。我觉得lz肯定自身就患有代码洁癖症，否则绝对不会总结的这么多，很专业！
本人同样凡有标准停车综合症，必须停正，且距离前后左右障碍物等距，这就是态度

28 楼 yuan 2013-02-24  

除了 7，其它都扯淡。

27 楼 Nightlee 2013-02-24  

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

26 楼 runfriends 2013-02-24  

zw1502071 写道

runfriends 写道

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

可以用switch

很多语言只能使用switch判断定值，不能进行复杂逻辑判断

25 楼 zw1502071 2013-02-24  

runfriends 写道

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

可以用switch

24 楼 lzl836 2013-02-24  

中枪。。。

23 楼 weng 2013-02-24  

我倒不觉得是洁癖，这是好习惯

22 楼 liukai 2013-02-24  

全中...

21 楼 bat0906 2013-02-24  

楼主 做对日的吧

20 楼 zhukewen_java 2013-02-24  

ifox 写道

evanzzy 写道

ifox 写道

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

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

哦哦，原来如此。谢谢指点啦

Java不用这么写，不伦不类。

19 楼 runfriends 2013-02-24  

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

18 楼 shangql 2013-02-24  

全中^_^

17 楼 ifox 2013-02-23  

evanzzy 写道

ifox 写道

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

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

哦哦，原来如此。谢谢指点啦

16 楼 evanzzy 2013-02-23  

ifox 写道

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

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

15 楼 witcheryne 2013-02-23  

中了不少抢...

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

function (parameter) {

    if( condition == true) {

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

    } else {

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

}

我就特么的想改成:

function (parameter) {

    if( condition == true) {

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

    }

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

}

14 楼 runfriends 2013-02-23  

除了第五条全中

13 楼 gundumw100 2013-02-23  

其实java里面还是推荐if(a==0)的写法，而不是if(0==a)

12 楼 OuYangGod 2013-02-23  

Except the last one...

11 楼 chenjinbo1983 2013-02-23  

有这习惯。

10 楼 zhou405680603 2013-02-23  

su1216 写道

前几条都是公司规定，不然review无法通过。
个人觉得这也是一个良好的习惯。

不过后几条就不怎么样了，5、6条也不是洁癖的表现阿

+1,每次commit前都要检查几遍啊