如何写出无法维护的代码
酷壳里有很多我觉得很不错的文章,但是访问量最大的却是那篇《6个变态的Hello World》,和它能在本站右边栏“全站热门”中出现的还有“如何加密源代码”,以及编程真难啊等这样的文章。可见本站的读者们的偏好,我也相信你们都是“身怀绝技”的程序员。所以,今天给大家推荐这篇文章,相信一定能触动大家的兴奋点。
这篇文章的原文在这里(http://mindprod.com/jgloss/unmain.html),我看完后我想说——
- 什么叫“创造力”,创造力就是——就算是要干一件烂事都能干得那么漂亮那么有创意的能力。
- 什么叫“抓狂”,抓狂就是——以一种沉着老练的不屈不挠的一本正经的精神一点一点把你推向崩溃的边缘。
我把文章节选了一些,也并没有完全翻译,简译一下,也加入了一些自己的调侃。对于有下面这些编程习惯的朋友,请大家对号入座。另外,维护程序的朋友们,你们死定了!!
If builders built buildings the way programmers write programs, then the first woodpecker that came along would destroy civilization. (如果建筑师盖房子就像程序员写程序一样,那么,第一只到来的啄木鸟就能毁掉我们的文明)
~ Gerald Weinberg (born: 1933-10-27 age: 77) Weinberg’s Second Law
目录
程序命名
- 容易输入的名字。比如:Fred,asdf
- 单字母的变量名。比如:a,b,c, x,y,z(陈皓注:如果不够用,可以考虑a1,a2,a3,a4,….)
- 有创意地拼写错误。比如:SetPintleOpening, SetPintalClosing。这样可以让人很难搜索代码。
- 抽象。比如:ProcessData, DoIt, GetData… 抽象到就跟什么都没说一样。
- 缩写。比如:WTF,RTFSC …… (陈皓注:使用拼音缩写也同样给力,比如: BT,TMD,TJJTDS)
- 随机大写字母。比如:gEtnuMbER..
- 重用命名。在内嵌的语句块中使用相同的变量名有奇效。
- 使用重音字母。比如:int ínt(注:第二个 ínt不是int)
- 使用下划线。比如:_, __, ___。
- 使用不同的语言。比如混用英语,德语,或是中文拼音。
- 使用字符命名。比如:slash, asterix, comma…
- 使用无关的单词。比如:god, superman, iloveu….
- 混淆l和1。字母l和数字1有时候是看不出来的。
伪装欺诈
- 把注释和代码交织在一起。
for(j=0; j<array_len; j+ =8) { total += array[j+0 ]; total += array[j+1 ]; total += array[j+2 ]; /* Main body of total += array[j+3]; * loop is unrolled total += array[j+4]; * for greater speed. total += array[j+5]; */ total += array[j+6 ]; total += array[j+7 ]; }
- 隐藏宏定义。如:#define a=b a=0-b,当人们看到a=b时,谁也想不到那是一个宏。
- 换行。如下所示,下面的示例使用搜索xy_z变得困难。
#define local_var xy\ _z // local_var OK
- 代码和显示不一致。比如,你的界面显示叫postal code,但是代码里确叫 zipcode.
- 隐藏全局变量。把使用全局变量以函数参数的方式传递给函数,这样可以让人觉得那个变量不是全局变量。
- 使用同意词。如:
#define xxx global_var // in file std.h #define xy_z xxx // in file ..\other\substd.h #define local_var xy_z // in file ..\codestd\inst.h
- 使用相似的变量名。如:单词相似,swimmer 和 swimner,字母相似:ilI1| 或 oO08。parselnt 和 parseInt, D0Calc 和 DOCalc。还有这一组:xy_Z, xy__z, _xy_z, _xyz, XY_Z, xY_z, Xy_z。
- 重载函数。使用相同的函数名,但是其功能和具体实现完全没有关系。
- 操作符重载。重载操作符可以让你的代码变得诡异,感谢CCTV,感谢C++。这个东西是可以把混乱代码提高到一种艺术的形式。比如:重载一个类的 ! 操作符,但实际功能并不是取反,让其返回一个整数。于是,如果你使用 ! ! 操作符,那么,有意思的事就发生了—— 先是调用类的重载 ! 操作符,然后把其返回的整数给 ! 成了 布尔变量,如果是 !!! 呢?呵呵。
- #define。看过本站那些混乱代码的文章,你都会知道宏定义和预编译对于写出不可读的代码的重大意义。不过,一个具有想像力的东西是——在头文件中使用预编译来查看这个头文件被include了几次,而被include不同的次数时,其中的函数定义完全不一样。
#ifndef DONE #ifdef TWICE // put stuff here to declare 3rd time around void g(char* str); #define DONE #else // TWICE #ifdef ONCE // put stuff here to declare 2nd time around< void g(void* str); #define TWICE #else // ONCE // put stuff here to declare 1st time around void g(std::string str); #define ONCE #endif // ONCE #endif // TWICE #endif // DONE
文档和注释
- 在注释中撒谎。你不用真的去撒谎,只需在改代码的时候不要更新注释就可以了。
- 注释明显的东西。比如:/* add 1 to i */。(参看本站的“五种应该避免的注释”)
- 只注释是什么,而不是为什么。
- 不要注释秘密。如果你开发一个航班系统,请你一定要保证每有一个新的航班被加入,就得要修改25个以上的位置的程序。千万别把这个事写在文档中。
- 注重细节。当你设计一个很复杂的算法的时候,你一定要把所有的详细细设计都写下来,没有100页不能罢休,段落要有5级以上,段落编号要有500个以上,例如:1.2.4.6.3.13 – Display all impacts for activity where selected mitigations can apply (short pseudocode omitted). 这样,当你写代码的时候,你就可以让你的代码和文档一致,如:Act1_2_4_6_3_13()
- 千万不要注释度衡单位。比如时间用的是秒还是毫秒,尺寸用的是像素还是英寸,大小是MB还是KB。等等。另外,在你的代码里,你可以混用不同的度衡单位,但也不要注释。
- Gotchas。陷阱,千万不要注释代码中的陷阱。
- 在注释和文档中发泄不满。(参看本站的“五种应该避免的注释”)
程序设计
- Java Casts。Java的类型转型是天赐之物。每一次当你从Collection里取到一个object的时候,你都需要把其转回原来的类型。因些,这些转型操作会出现在N多的地方。如果你改变了类型,那么你不一定能改变所有的地方。而编译器可能能检查到,也可能检查不到。
- 利用Java的冗余。比如:Bubblegum b = new Bubblegom(); 和 swimmer = swimner + 1; 注意变量间的细微差别。
- 从不验证。从不验证输入的数据,从不验证函数的返回值。这样做可以向大家展示你是多么的信任公司的设备和其它程序员。
- 不要封装。调用者需要知道被调用的所有的细节。
- 克隆和拷贝。为了效率,你要学会使用copy + paste。你几乎都不用理解别人的代码,你就可以高效地编程了。(陈皓注:Copy + Paste出来的代码bug多得不能再多)
- 巨大的listener。写一个listener,然后让你的所有的button类都使用这个listener,这样你可以在这个listener中整出一大堆if…else…语句,相当的刺激。
- 使用三维数组。如果你觉得三维还不足够,你可以试试四维。
- 混用。同时使用类的get/set方法和直接访问那个public变量。这样做的好处是可以极大的挫败维护人员。
- 包装,包装,包装。把你所有的API都包装上6到8遍,包装深度多达4层以上。然后包装出相似的功能。
- 没有秘密。把所有的成员都声明成public的。这样,你以后就很难限制其被人使用,而且这样可以和别的代码造成更多的耦合度,可以让你的代码存活得更久。
- 排列和阻碍。把drawRectangle(height, width) 改成 drawRectangle(width, height),等release了几个版本后,再把其改回去。这样维护程序的程序员们将不能很快地明白哪一个是对的。
- 把变量改在名字上。例如,把setAlignment(int alignment)改成,setLeftAlignment, setRightAlignment, setCenterAlignment。
- Packratting。保留你所有的没有使用的和陈旧的变量,方法和代码。
- That’s Final。Final你所有的子结点的类,这样,当你做完这个项目后,没有人可以通过继承来扩展你的类。java.lang.String不也是这样吗?
- 避免使用接口。在java中,BS接口,在C++中BS使用虚函数。
- 避免使用layout。这样就使得我们只能使用绝对坐标。如果你的老大强制你使用layout,你可以考虑使用GridBagLayout,然后把grid坐标hard code.
- 环境变量。如果你的代码需要使用环境变量。(getenv() – C++ / System.getProperty() – Java ),那么,你应该把你的类的成员的初始化使用环境变量,而不是构造函数。
- 使用Magic number。参看《Linux一个插曲》。
- 使用全局变量。1)把全局变量的初始化放在不同的函数中,就算这个函数和这个变量没有任何关系,这样能够让我们的维护人员就像做侦探工作一样。2)使用全局变量可以让你的函数的参数变得少一些。
- 配置文件。配置文件主要用于一些参数的初始化。在编程中,我们可以让配置文件中的参数名和实际程序中的名字不一样。
- 膨胀你的类。让你的类尽可能地拥有各种臃肿和晦涩的方法。比如,你的类只实现一种可能性,但是你要提供所有可能性的方法。不要定义其它的类,把所有的功能都放在一个类中。
- 使用子类。面向对象是写出无法维护代码的天赐之物。如果你有一个类有十个成为(变量和方法)你可以考虑写10个层次的继承,然后把这十个属性分别放在这十个层次中。如果可能的话,把这十个类分别放在十个不同的文件中。
混乱你的代码
- 使用XML。XML的强大是无人能及的。使用XML你可以把本来只要10行的代码变成100行。而且,还要逼着别人也有XML。(参看,信XML得永生,信XML得自信)
- 混乱C代码。在《如何加密源代码》中已经说过一些方法了,这里再补充一些。
- 使用不同的进制。比如:10 和010不是一样的。再比如:array = new int[]{ 111, 120, 013, 121,};
- 尽量使用void*。然后把其转成各种类型
- 使用隐式的转型。C++的构造函数可以让你神不知鬼不觉得完成转型。
- 分解条件表达式。如:把 a==100分解成,a>99 && a<101
- 学会利用分号。如:if ( a );else;{ int d; d = c;}
- 间接转型。如:把double转string,写成new Double(d).toString() 而不是 Double.toString(d)
- 大量使用嵌套。一个NB的程序员可以在一行代码上使用超过10层的小括号(),或是在一个函数里使用超过20层的语句嵌套{},把嵌套的if else 转成 [? :] 也是一件很NB的事。
- 使用C的变种数组。myArray[i] 可以变成*(myArray + i) 也可以变成 *(i + myArray) 其等价于 i[myArray]。再看一个函数调用的示例,函数声明:int myfunc(int q, int p) { return p%q; } 函数调用myfunc(6291, 8)[Array];
- 长代码行。一行的代码越长越好。这样别人阅读时就需要来来回回的
- 不要较早的return。不要使用goto,不要使用break,这样,你就需要至少5层以上的if-else来处理错误。
- 不要使用{}。不要在if else使用{},尤其是在你重量地使用if-else嵌套时,你甚至可以在其中乱缩进代码,这样一来,就算是最有经验的程序员也会踩上陷阱。
- 使用宏定义。宏定义绝对是混乱C/C++代码的最佳利器。参看 老手是这样教新手编程的。
- 琐碎的封装。比较封装一个bool类,类里面什么都做,就是一个bool.
- 循环。千万不可用for(int i=0; i<n; i++)使用while代替for,交换n和i,把<改成<=,使用 i–调整步伐 。
测试
- 从不测试。千万不要测试任何的出错处理,从来也不检测系统调用的返回值。
- 永远不做性能测试。如果不够快就告诉用户换一个更快的机器。如果你一做测试,那么就可能会要改你的算法,甚至重设计,重新架构。
- 不要写测试案例。不要做什么代码覆盖率测试,自动化测试。
- 测试是懦夫行为。一个勇敢的程序员是根本不需要这一步的。太多的程序太害怕他们的老板,害怕失去工作,害怕用户抱怨,甚至被起诉。这种担心害怕直接影响了生产力。如果你对你的代码有强大的信心,那还要什么测试呢?真正的程序员是不需要测试自己的代码的。
其它
- 你的老板什么都知道。无论你的老板有多SB,你都要严格地遵照他的旨意办事,这样一来,你会学到更多的知识如何写出无法维护的代码来的。
- 颠覆Help Desk。你要确保你那满是bug的程序永远不要被维护团队知道。当用户打电话和写邮件给你的时候,你就不要理会,就算要理会,让用户重做系统或是告诉用户其帐号有问题,是标准的回答。
- 闭嘴。对于一些像y2k这样的大bug,你要学会守口如瓶,不要告诉任何人,包括你的亲人好友以及公司的同事和管理层,这样当到那一天的时候,你就可以用这个bug挣钱了。
- 忽悠。你会学会忽悠,就算你的代码写得很烂,你也要为其挂上GoF设计模式的标签,就算你的项目做得再烂,你也要为其挂上敏捷的标签,只有学会像中国Thoughtworks的咨询师那样去忽悠,你才能学会更炫更酷的方法,让整个团队和公司,甚至整个业界都开始躁动,这样才能真正为难维护的代码铺平道路。
这个文档中还有很多很多,实在是太TMD强大了,大家自己去看看吧。有精力有能力的朋友不妨把其翻译成中文。
总之,我们的口号是——
Write Everywhere, Read Nowhere
(全文完)
(转载本站文章请注明作者和出处 酷 壳 – CoolShell ,请勿用于任何商业用途)
《如何写出无法维护的代码》的相关评论
TJJTDS
崩溃中。。。
Write Everywhere, Read Nowhere。。。 = =
代码里面怎么到处是 啊?
都是箴言, 记下来了
Write Everywhere, Read Nowhere.
皓君,您博客是我提高水平的一大来源,
但是我想问个问题,您这些文章都是怎么得到的捏。很多都是小网站上的。
您如何获取到这些高质量的原创文章的。digg?推特?求赐教~/抱拳
没什么意思 这些都是每个程序员与生俱来的能力
另外, 把变量写的“丑”或者隐晦、间接跟加密貌似没什么关系吧,只是难以阅读,并不是所以的加密把
中枪了
实在是太强大了!不得不出来顶一下!最后一句Write Everywhere, Read Nowhere更新精髓!
拉出去,TJJTDS
很好,这样可以保住你的饭碗
TJJTDS 这个看文章没看懂, 看到11楼的我就明白了。
文章页列出了不少自己在写程序的时候的毛病…
毕业了,老师让我吧所有毕业设计的代码交接清楚,结果他发现我程序没有多少注释~ 要求添加上
没看懂。。。不维护怎么后期使用
TJJTDS这个变量亮了!
看到最后一句的时候乐喷了……
学习了……其实反过来看就是如何写出优质代码
太坏了╮(╯▽╰)╭
有同事喜欢在自己修改代码的地方加上注释,注释内容一般就只有自己的名字。这是个好习惯吗?反正我看着不太爽,毕竟SVN能查到所有记录。
你可以参看—— 五种应该避免的代码注释 http://coolshell.cn/articles/2746.html
博主最后还不忘挖苦一下Thoughtworks- –
@砖家
我觉得挺好的,一段没有注明的修改代码可能会在很久之后被挖出来,此时查SVN很麻烦。
只注释名字嘛,是不是浪费了点,既然修改了就有原因啊。
@zephyro
git blame
@zephyro
blame很麻烦么。
Wolegequ Gelivable。。。
把〈br /〉都去掉能让人看着舒服一些……
我就是维护前任留下来的项目的,那代码实在强大。在书店看到一本,《代码简洁之道》虽然是用java描述的,而我是写C语言的,不过还是很想,买去问的时候居然没新的了,而那最后一本已经千疮百孔。所以只能先下电子版的来看了。
哈哈 有意思
还有用很多enum, 很多#define来不断重复地定义一些常量, 增加函数的抽象层, 逻辑和状态机, 让 别人追踪不了.
很好很强大。
不过,犯过其中大多数错误!
“巨大的listener。”
我所在的项目switch有大概500行?但每次还规定要从最后添加case。结果只好滚鼠标滚到手酸——拖拽滑动块的话会因为轻轻一动就错过整个函数!
我艹,我们头还说:这么做有问题吗?你那么重构的话不是也要用if-else或者switch-case吗?
要听老板的话。这样代码才会变得更难维护。
我特意去GCC编译了一下
“隐藏宏定义。如:#define a=b a=0-b,当人们看到a=b时,谁也想不到那是一个宏。”
不能通过, 你写的是C语言吧, 你不知道名字不能带符号么? 最多是下划线吧?
你是不是有点想当然了?
也许可以用某些编译选项跳过这个检查, 再或者, 你可以试试用全角等号
遇到很多, 还有一些这里没提到的:
曾经接手一个项目, 数据库有300张表, 几乎所有字段全是 Text, 所有数字/日期/ID/… 一切字段全部用 NText, 而且命名没有一个能猜得懂.
又曾经接手一个大公司的项目, 有600张表, 检查结果只有20张表有用,
如果是因為老闆機車要作怪,不知這方法可不可行,
自己留好改的原始碼,然後將原始碼replace string 掉,
當然要可以再replace string 回來。
然後再交出去。
別人會改到瘋掉,除非靠你replace 回來的原始碼,不然是無法維護的。
看来博主跟totwoks是打算干到底了
看看这个网站(如何写出晕菜的C代码):http://www.ioccc.org/
受益匪浅:使用XML你可以把本来只要10行的代码变成100行。而且,还要逼着别人也有XML。
万能的xml,压力好大啊
看到过用a,b,c,a1,b1。。。。来命名变量的。直接看都不要看了
弓虽!
很自豪的说,我现在公司的旧代码几乎符合全部条件!!
哦也,万岁!
asdf是我的最爱
当我不想混下去时,我会严格按照文章中的“标准”来编码!哈哈~~~
这个是很好的“反向编码方式”的文章:)
不太明白,这是代码混淆吗,没有必要吧,程序本身也不是写给自己一个人看的。。
为什么不把代码写的很好,大家看着都舒服呢,搞不懂。。
如果所有程序员都这样,还会有几个人做维护,估计维护就和大熊猫差不多了,