说一说编程恶习

最近在做一个汽车销售系统的改善工作， 这个系统已经运行两年了， 两年来， 客户不断的提出新需求， 系统也在不断的改来改去。 这次轮到我来改它了。  想想N年前初学编程的时候， 书上， 网上， 杂志上不断的在说， 要养成良好的编程习惯。 然后还给出了N长的一大篇文章来介绍一些编程规范。 我这个人是很懒的， 大概的看了一下就过去了。 没有特意的记什么。 好在我这个人也不是特别的懒， 对自己的工作也是很上心。 编程的时候尽可能做到更好。 性能功能能考虑到的都要做到最好。  慢慢的也养成了一些编程的习惯， 时间长了， 下意识的就去遵守一些模式，模范之类的东西了。
         有了这些习惯， 再看这次修改的系统， 真的是生可忍熟不可忍了。  这次我也不说什么编程规范了， 我就说说这些编程恶习 。 

         一，  程序没有注释
       注释!!注释!!! 如果只是打印了一个HELLO WORLD， 您不注释那也就算了， 如果是只有一两百行的小功能类您不注释， 那我也忍了， 可是3000多行一个类的业务逻辑代码， 您老人家还不注释!!!   你TM让我怎么去改代码，  一点业务逻辑的说明都没有， 我改代码的时候，得一边用DEBUG调试， 一边替他加注释。 然后才能进行自己的工作。 几千行的一个类，  一行注释都没有， 你TM就不觉得颜色单调了点吗?

       二，  不遵守基本的编程约定
       变量名大小写混乱， 明明是变量， 非要完全大写， 要不就大写开头。 要不就是方法名全是大写， 最牛B的一个方法是用中文做方法名， 你丫这时候想起打中文来了， 累不累呀。
还有人用拼音做变量名方法名，就算您英文不好，稍微查一下金山词霸行不行，现在百度和GOOGLE都有翻译功能，稍微查一下英文，也当是学英语了行不行？ 您实在太忙的话，不查也就算了，拼音就拼音吧，好赖也算是中国话的。 可是您就别用拼音简写了，英文简写还认不出来呢， 您还用拼音的开头字母当变量名， 那我TM上哪儿猜去呀！

三， 不明就里的代码
系统中经常会出现这样的代码，尤其是在controller里居多：
// some code
If(flag .equals(“submit”)){
model.getInfo();
}else{
model.getInfo();
}
我没写错，if和else调用的方法完全一样，大家也放心，我仔细的看过调用的代码，调用的方法里，也没有根据其它情况来改变他的运行路线。我就不明白为什么要做这个if判断了。担心会有什么特殊的业务逻辑， 所以也不赶随便去改他。 猜了半天，感觉最理想的答案是写代码的人，担心以后会有新的逻辑分支， 所以在这里用if预留了一个位置， 以后改的时候方便。
数日之后有幸遇见了当初写这代码的老兄，问过之后立刻晕倒，原来是这个代码是参照别的模块的样子写的，别的模块在这里都有N个程序分支，通过if来判断后决定调用哪个model里的方法。但他这个模块很简单，没有什么分支，就是调用那一个方法，但他写代码时，看别人的模块在这里都进行if判断了，所以觉得自己也应该判断一下，于是就出现了上面这样的代码。

四， 面向过程式的编程方法
遇到过好几次2000多行的方法，所有业务逻辑，一气呵成，就用了一个方法搞定。如果是简单的逻辑也就算了， 可是几千行的代码全放在一个方法里，一个类里有无数的重复代码。 这回到好，重构那本书没白看， 现在有了实践的机会了。
难道您自己调试的时候就不觉得麻烦吗？ 我在这里不想讨论什么面向过程还是面向对象，也别和我说什么方法多了也不一定就是面向对象的思想。 平时对自己写的代码多上点心， 大家都是在这行干了几年的人了，把代码写的漂亮点有什么不好。

五， 代码缩进混乱
我们公司有规定，改代码的时候，不许修改原有代码的格式。 不管他多乱，也不许改。 我不明白这是为什么，也许是检查代码的人，要用文件比较工具吧。 
但这下苦坏我了， 代码的格式那叫一个乱。 有顶着行头写的， 有向后空了N格的，大概是写代码的人， 为了方便自己找到正在调试的那段代码，所以把代码的缩进变得和其它代码与众不同吧。 那您调试完了到是重新排一下版呀， 这真的不累～～， 现在的IDE工具都有自动排版代码的功能， 一个快捷键就搞定了，稍微勤快一点行吗？？
最BT的一段代码是缩进居然出了屏幕！！！ 你吃饱了撑的呀， 没事缩那么远干吗， 我根据后台输出找了半天也没找到那段代码在哪儿， 原来是因为缩进的太远了，不在屏幕范围之内， 向右拉了半天滚动条才找到。 你丫是不是写着代码睡着了？ 脸正好砸在TAB键上。

六， 多余的后台输出
好几个循环嵌套在一起～～～ 也行， 就算是因为业务逻辑需要，没别的办法也将就了。 好几个循环嵌套在一起查数据库， 咱们最好还是开动一下脑筋， 看看有什么更好的办法。如果还是没别的办法， 那也凑合了。 可这种情况您就别在后台输出SQL语句啦， 每次一执行程序，成百上千个SQL语句在后台输出， 查数据库才用了一两秒，结果输出这些SQL就用了半分钟。 您自己就没觉出程序慢在哪里吗？ 您调试程序的时候输出一下也就算了， 提交到正式运行的环境时，就麻烦您，劳您大驾～～ 把那些输出注释掉吧，实在不行留几个重要的输出就行了。 让这种代码影响系统性能～～ 也太冤了吧。

七， 打肿脸充胖子
我也不知道这条算不算恶习，也许不算，在有些人眼里还是好事。但也要看具体情况，经常有些人写代码不喜欢用IDE，只用EDITPLUS这类工具。按常理说，初学者都应该尽量用这些编辑器写代码，对加深学习印象有好处。也有人说高手不屑于用那些IDE，我少见多怪， 这种绝顶高手我没见过。
但咱平时工作的时候，要的是效率，您不是那种高手就乖乖的用IDE吧。经常见到有些人，为查一个方法的调用，搜来搜去的。真正的高手是工作效率最高的人，不是用最简单工具的人。

今天就写这么多， 大家还遇到过什么样的编程恶习，欢迎补充。
大家不要总是抱怨什么工资太少，工作量太大。工作效率这玩意儿是要经验来做基础，这没错，经验少也没事。咱平时写程序的时候多上点心，多对自己的代码思考一下，多动动脑子。自然就能总结出最好的工作经验了，工作效率自然就提高了。
也别总是说什么STRUTS不好，HIBERNATE太慢，不屑去用它。人家的程序能在全世界流行，自然有他的过人之处。多读读他的代码，学习一下他到底好在哪里，如何才能把这些优点应用到自己的代码上。这才是最重要的。
写出上面这些代码的人，如果你的工资真的很少，那我只能惋惜的说一句：你的工资是可怜了点，但看您写的这代码，连这点工资都不应该给你！！

273 个解决方案

#1

学习一下

#2

哈哈 说的好
学习学习

#3

说的好！鼓掌！

#4

呵呵 说的不错

#5

楼 主 辛苦  了

#6

没代码？？？！！！没注？？？！！！我绝受不了！！！

#7

我個人認爲其實日本人寫代碼的規範性我們有必要學習一下。取其精華嘛！

#8

>> 最BT的一段代码是缩进居然出了屏幕！！！


太强了!

#9

学习了,看过之后我也范过同样的错误啊!以后也要改啊!唉...

你说的那些问题好多人都是那样做的.中国人写程序就是这样啊!不过也是改的时候了.

#10

最BT的一段代码是缩进居然出了屏幕！！！ 你吃饱了撑的呀， 没事缩那么远干吗， 我根据后台输出找了半天也没找到那段代码在哪儿， 原来是因为缩进的太远了，不在屏幕范围之内， 向右拉了半天滚动条才找到。 你丫是不是写着代码睡着了？ 脸正好砸在TAB键上。



强人啊！！！！！

#11

呵呵
还好,
我只犯了第一条,
以后绝对会改的!!!

#12

很有收获！
谢谢拉，楼主！

#13

最BT的一段代码是缩进居然出了屏幕！！！ 你吃饱了撑的呀， 没事缩那么远干吗， 我根据后台输出找了半天也没找到那段代码在哪儿， 原来是因为缩进的太远了，不在屏幕范围之内， 向右拉了半天滚动条才找到。 你丫是不是写着代码睡着了？ 脸正好砸在TAB键上。
━━━━━━━━━━━━━━━━━━━━━
这句话太可笑了，居然还有这样的人哇
前几天看一个初学者的代码，是JSP文件里的<table><tr><td>的，大家应该知道的
这位朋友用得太好了，真佩服，几十层的<tr><td><table>一大堆，不过这位严格遵循规范
每一层都有缩进，靠近里边的层，需要移动滚动条才能看到

━━━━━━━━━━━━━━━━━━━━━
If(flag .equals(“submit”)){
model.getInfo();
}else{
model.getInfo();
}
━━━━━━━━━━
if 语句没有用就不说了，应该把字符串放在前边(避免异常)，这是编程的良好习惯哈
( "submit".equals(flag) )

#14

多看 c++ gotchas 一书。

#15

确实要注意了

#16

我刚刚又遇到了这样一个代码, 整个模块没有JSP, 所有的JSP,HTML, 连JS代码都是在servlet里Out.pringln(String)的...........天啊, 神啊救救我吧............... 我们工程布署不规范, 哪怕是改个普通的工具类都要重启才行.重启的可是weblogic呀,我只是想改一个JS代码呀 ..........累死人不偿命啊

#17

良好的变成习惯确实重要，楼主总结的不错

#18

仰慕楼主啊。。。。

#19

哈哈．

#20

不过话说回来， 实际工作中常常避免不了写一些难看的代码
客户要求变来变去， 再好的设计最终也会因为赶工而变成一堆乱七八糟的代码

#21

最怕的是客户不断让你加一些“小功能”，或者“稍微”改一下。

原来的逻辑很可能就被彻底破坏打乱

#22

写出上面这些代码的人，如果你的工资真的很少，那我只能惋惜的说一句：你的工资是可怜了点，但看您写的这代码，连这点工资都不应该给你！！
----------
说得好
顶lz
不过我还没有见过这样的代码

我们做项目都有严格的规定
甚至方法的大括号在右边还是在下边都有规定

#23

深有体会，我都修改别人代码好久了，前人都是所谓的高手，留下的代码没有什么注释，代码也编写得不规范，和散文比较相似，我都要cao TNND 了。不过现在基本全部重新写完了，前人的功能我都实现了，不过代码基本全给丫废了，自己写的，整齐规范而且注释全。

#24

mark

#25

高山景仰--向LZ同志学习

#26

学习

#27

好。
向lz致敬。

#28

最反感用中文拼音简称做变量名的
这种人完全是粗人，没文化

#29

种种问题是程序员的惹得祸？
谁不想把程序写好，但是，你的老大天天让你快，什么什么问题n天搞定，从来不关怎么做，你也许加个通宵班才能做完。
还有关于缩进，现在ide都可以自动format，你可以format成自己喜欢的格式，难道这还成问题么，又不是用写字板编程。

#30

幽默并富有哲理，顶了

#31

good

#32

有则改之，无则加勉
谢谢楼主

#33

一次改别人代码，一个方法里2500+行的代码，利害啊，高手啊！
注释有！







大段大段被注释掉的代码

#34

mark

#35

我维护的这个项目是四年前的,
现在还在卖,里面的代码可想而知.

LZ说的那些问题,我这里基本上有,
我是新来的,有什么问题也不大好问,
有些人压根没有把我放在心上,
也不知道高傲什么.

我们有个TableBean几K行,没有注释的.
把数据查出来,一个打印方法就把
所有的东西全输出来了,
public void printReport(JspWriter writer)
一口气全输出来,
这个表格还有排序的功能,厉害

#36

to:zzhzzh204553(真的好想你) 
呵呵你不是说我吧,怎么跟我写的东西一样啊,呵呵,不光成列排序,还有很多可扩展的功能!
to:楼主
在很多方面用editplus比用ide效率要高,当然只是我个人认为啊!

#37

难以想象，

#38

哦

#39

收藏。

居然有上千行的类，晕了。

我的超过百行的几乎就没有了。

#40

补充：我用的是 asp.net 。

没有注意看分类。

#41

哈哈!!! 我真的感谢..今天能看到这篇文章!!! 太好了.. ... 
   我大三了..  楼主说的那些行为 除了IDE外 别的自己感觉良好!!! 
   我在学习的时候经常看的是Sun 的 源码!!! 你可别说 真管用 !! 自然而然 的在写程序的时候就想 加注释 多态 异常 继承 封装 那些最最基础的概念 也都是在看源码+实践慢慢体会的!!! 

  谢谢楼主写的这篇文章!!  !!!

#42

良好的变成习惯确实重要，楼主总结的不错

#43

最怕的是客户不断让你加一些“小功能”，或者“稍微”改一下。

原来的逻辑很可能就被彻底破坏打乱


我TMD身有同感！！！！！！！！！！！！！！！！！！！！！！！！！

#44

好

#45

楼主太幽默了.哈哈.

#46

良好的书写习惯有利于初学者。工作效率也用这里来的，还有就是团队的配合也需要。

#47

有些人提到用EDITPLUS的效率其实也很高, 这点我也承认. 但我想说明一下, 我所见到的用EDITPLUS那个人, 根本没有发挥出editplus应有的效率和功能. 在他的手里editplus只是一个多了颜色区分的记事本. 编译程序还是用javac , 也不会用ANT, 发布程序还是手工拷贝. 号称四年经验的程序员, 刚到公司时用了一天时间重装好电脑后, 写了一个打印hello world的程序测试环境. 结果不写static void main方法, 直接就要用java去运行. 运行不成功还说系统配置有问题, 需要再重装.  每次帮他调试程序, 想查一个方法的调用, 一个文件一个文件的搜, 看得我这个急...........

时间全耽误在这上了, 这样的工作效率, 加班都是活该的

#48

一个项目启动之前的准备实在是不能就这么：省了也罢，国内开发行业的诸多不规范，原因有很多，有一部分是客户不懂软件工程，急于早点看到系统面庞，还有一部分原因是开发承接公司的管理水平不足，特别是小公司，根本谈不上软件工程管理，只求速度不求质量。
  编程人员的习惯、经验、水平不尽相同，关键是需要管理上去统一去要求才能避免这些情况的出现。代码写成这样实在不是Codinger的单方责任。想办法提高一下管理水平才是根本的。

#49

估计是新手写的吧.
文笔还不纯熟呀,风格还没形成.
哈哈~~~
 
楼主辛苦了,是不是觉得还不如自己写一遍好呢.呵呵

#50

mark