名扬数据:如何写出无法维护的代码
什么叫“发明力”发明力就是就算是要干一件烂事都能干得那么漂亮那么有创意的能力。
什么叫“抓狂”抓狂就是以一种沉着老练的不屈不挠的一本正经的精神一点一点把你推向解体的边缘。
把文章节选了一些,也并没有完全翻译,简译一下,也加入了一些自己的调侃。对于有下面这些编程习惯的朋友,请大家对号入座。另外,维护顺序的朋友们死定了
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 Weinbergs Second Law
顺序命名
容易输入的名字。比方:Fredasdf
单字母的变量名。比方:a,b,c, x,y,z陈皓注:如果不够用,可以考虑a1,a2,a3,a4,.
有创意地拼写错误。比方:SetPintleOpen SetPintalClos这样可以让人很难搜索代码。
笼统。比方:ProcessData, DoIt, GetData 笼统到就跟什么都没说一样。
缩写。比方:WTFRTFSC 陈皓注:使用拼音缩写也同样给力,比方: BTTMDTJJTDS
随机大写字母。比方:gEtnuMbER..
重用命名。内嵌的语句块中使用相同的变量名有奇效。
使用重音字母。比方:int ínt注:第二个 ínt不是int
使用下划线。比方:_, __, ___
使用不同的语言。比如混用英语,德语,或是中文拼音。
使用字符命名。比方:slash, asterix, comma
使用无关的单词。比方:god, superman, iloveu.
混淆l和1字母l和数字1有时候是看不出来的
伪装欺诈
把注释和代码交织在一起。
forj=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 cod但是代码里确叫 zipcode.
隐藏全局变量。把使用全局变量以函数参数的方式传送给函数,这样可以让人觉得那个变量不是全局变量。
使用同意词。如:
使用相似的变量名。如:单词相似,swimmer 和 swimner字母相似:ilI1| 或 oO08parselnt 和 parseInt D0Calc 和 DOCalc还有这一组:xy_Z, xy__z, _xy_z, _xyz, XY_Z, xY_z, Xy_z
重载函数。使用相同的函数名,但是其功能和具体实现完全没有关系。
操作符重载。重载操作符可以让你代码变得诡异,感谢CCTV感谢C++这个东西是可以把混乱代码提高到 一种艺术的形式。比方:重载一个类的 ! 操作符,但实际功能并不是取反,让其返回一个整数。于是如果你使用 ! ! 操作符,那么,有意思的事就发生了 先是调用类的重载 ! 操作符,然后把其返回的整数给 ! 成了 布尔变量,如果是 !!! 呢?呵呵。
#defin看过本站那些混乱代码的文章,都会知道宏定义和预编译对于写出不可读的代码的重大意义。不过,一个具有想像力的东西是头文件中使用预编译来检查这个头文件被includ几次,而被includ不同的次数时,其中的函数定义完全不一样。
文档和注释
注释中撒谎。不用真的去撒谎,只需在改代码的时候不要更新注释就可以了
注释明显的东西。比方:/* 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 omit. 这样,当你写代码的时候,就可以让你代码和文档一致,如:Act1_2_4_6_3_13
千万不要注释度衡单位。比如时间用的秒还是毫秒,尺寸用的像素还是英寸,大小是MB还是KB等等。另外,代码里,可以混用不同的度衡单位,但也不要注释。
Gotcha陷阱,千万不要注释代码中的陷阱。
注释和文档中发泄不满。
顺序设计
Java CastJava类型转型是天赐之物。每一次当你从Collect里取到一个object时候,都需要把其转回原来的类型。因些,这些转型操作会出现在N多的地方。如果你改变了类型,那么你不一定能改变所有的地方。而编译器可能能检查到也可能检查不到
利用Java冗余。比方:Bubblegum b = new Bubblegom; 和 swimmer = swimner + 1; 注意变量间的细微差异。
从不验证。从不验证输入的数据,从不验证函数的返回值。这样做可以向大家展示你多么的信任公司的设备和其它顺序员。
不要封装。调用者需要知道被调用的所有的细节。
克隆和拷贝。为了效率,要学会使用copy + past几乎都不用理解他人的代码,就可以高效地编程了陈皓注:Copy + Past进去的代码bug多得不能再多)
巨大的listen写一个listen然后让你所有的button类都使用这个listen这样你可以在这个listen中整出一大堆ifels语句,相当的安慰。
使用三维数组。如果你觉得三维还不足够,可以试试四维。
混用。同时使用类的get/set方法和直接访问那个public变量。这样做的好处是可以极大的挫败维护人员。
包装,包装,包装。把你所有的API都包装上68遍,包装深度多达4层以上。然后包装出相似的功能。
没有秘密。把所有的成员都声明成public这样,以后就很难限制其被人使用,而且这样可以和别的代码造成更多的耦合度,可以让你代码存活得更久。
排列和阻碍。把drawRectanglheight, width 改成 drawRectanglwidth, height等releas几个版本后,再把其改回去。这样维护顺序的顺序员们将不能很快地明白哪一个是对的
把变量改在名字上。例如,把setA lignint align改成,setLeftA lignment, setRightA lignment, setCenterA lign
Packrat保存你所有的没有使用的和陈旧的变量,方法和代码。
Thats FinalFinal所有的子结点的类,这样,当你做完这个项目后,没有人可以通过继承来扩展你类。java.lang.Str不也是这样吗?
防止使用接口。java中,BS接口,C++中BS使用虚函数。
防止使用layout这样就使得我只能使用绝对坐标。如果你老大强制你使用layout可以考虑使用GridBagLayout然后把grid坐标hard code.
环境变量。如果你代码需要使用环境变量。getenv – C++ / System.getProperti – Java 那么,应该把你类的成员的初始化使用环境变量,而不是构造函数。
使用Magic numb参看《Linux一个插曲》
使用全局变量。1把全局变量的初始化放在不同的函数中,就算这个函数和这个变量没有任何关系,这样能够让我维护人员就像做侦探工作一样。2使用全局变量可以让你函数的参数变得少一些。
配置文件。配置文件主要用于一些参数的初始化。编程中,可以让配置文件中的参数名和实际顺序中的名字不一样。
膨胀你类。让你类尽可能地拥有各种臃肿和晦涩的方法。比方,类只实现一种可能性,但是要提供所有可能性的方法。不要定义其它类,把所有的功能都放在一个类中。
使用子类。面向对象是写出无法维护代码的天赐之物。如果你有一个类有十个成为(变量和方法)可以考虑写10个层次的继承,然后把这十个属性分别放在这十个层次中。如果可能的话,把这十个类分别放在十个不同的文件中。
混乱你代码
使用XMLXML强大是无人能及的使用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;}
间接转型。如:把doubl转string写成new Doubld.toStr 而不是 Double.toStrd
大量使用嵌套。一个NB顺序员可以在一行代码上使用超过10层的小括号(或是一个函数里使用超过20层的语句嵌套{}把嵌套的if else 转成 [? :] 也是一件很NB事。
使用C变种数组。myA rray[i] 可以变成*myA rray + i 也可以变成 *i + myA rrai 其等价于 i[myA rray]再看一个函数调用的示例,函数声明:int myfuncint q, int p { return p%q; } 函数调用myfunc6291, 8[A rray];
长代码行。一行的代码越长越好。这样他人阅读时就需要来来回回的
不要较早的return不要使用goto不要使用break这样,就需要至少5层以上的if-els来处理错误。
不要使用{}不要在if els使用{}尤其是重量地使用if-els嵌套时,甚至可以在其中乱缩进代码,这样一来,就算是最有经验的顺序员也会踩上陷阱。
使用宏定义。宏定义绝对是混乱C/C++代码的最佳利器。参看 老手是这样教新手编程的
琐碎的封装。比较封装一个bool类,类里面什么都做,就是一个bool.
循环。千万不可用forint i=0; i<n; i++使用while代替for交换n和i把<改成<=使用 i–调整步伐
测试
从不测试。千万不要测试任何的出错处置,历来也不检测系统调用的返回值。
永远不做性能测试。如果不够快就告诉用户换一个更快的机器。如果你一做测试,那么就可能会要改你算法,甚至重设计,重新架构。
不要写测试案例。不要做什么代码覆盖率测试,自动化测试。
测试是懦夫行为。一个勇敢的顺序员是根本不需要这一步的太多的顺序太害怕他老板,害怕失去工作,害怕用户埋怨,甚至被起诉。这种担心害怕直接影响了生产力。如果你对你代码有强大的信心,那还要什么测试呢?真正的顺序员是不需要测试自己的代码的
其它
老板什么都知道。无论你老板有多SB都要严格地遵照他旨意办事,这样一来,会学到更多的知识如何写出无法维护的代码来的
颠覆Help Desk要确保你那满是bug顺序永远不要被维护团队知道。当用户打电话和写邮件给你时候,就不要理会,就算要理会,让用户重做系统或是告诉用户其帐号有问题,规范的回答。
闭嘴。对于一些像y2k这样的大bug要学会守口如瓶,不要告诉任何人,包括你亲人好友以及公司的同事和管理层,这样当到那一天的时候,就可以用这个bug挣钱了
忽悠。会学会忽悠,就算你代码写得很烂,也要为其挂上GoF设计模式的标签,就算你项目做得再烂,也要为其挂上敏捷的标签,只有学会像中国Thoughtwork咨询师那样去忽悠,才干学会更炫更酷的方法,让整个团队和公司,甚至整个业界都开始躁动,这样才干真正为难维护的代码铺平道路。
这个文档中还有很多很多,实在太TMD强大了大家自己去看看吧。有精力有能力的朋友无妨把其翻译成中文。