如何完美写出让同事无法维护的代码？

（反面教材，谨慎使用 ）

程序命名

• 容易输入的变量名。 比如：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 有时候是看不出来的。

02

伪装欺诈

• 把注释和代码交织在一起。

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 ];}

• 代码和显示不一致。 比如，你的界面显示叫 postal code，但是代码里确叫  zipcode.

• 隐藏全局变量。 把使用全局变量以函数参数的方式传递给函数，这样可以让人觉得那个变量不是全局变量。

• 使用相似的变量名。 如：单词相似，swimmer 和 swimner，字母相似：ilI1| 或 oO08。parselnt 和 parseInt， D0Calc 和 DOCalc。还有这一组：xy_Z, xy__z, _xy_z, _xyz, XY_Z, xY_z, Xy_z。

• 重载函数。 使用相同的函数名，但是其功能和具体实现完全没有关系。

• 操作符重载。 重载操作符可以让你的代码变得诡异，感谢 CCTV，感谢 C++。这个东西是可以把混乱代码提高到一种艺术的形式。

  比如：重载一个类的 ! 操作符，但实际功能并不是取反，让其返回一个整数。于是，如果你使用 ! ! 操作符，那么，有意思的事就发生了—— 先是调用类的重载 ! 操作符，然后把其返回的整数给 ! 成了 布尔变量，如果是 !!! 呢？呵呵。

03

文档和注释

• 在注释中撒谎。 你不用真的去撒谎，只需在改代码的时候不要更新注释就可以了。

• 注释里面写废话。 比如：/* 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。 陷阱，千万不要注释代码中的陷阱。

• 在注释和文档中发泄不满。

04

程序设计

• Java Casts。 Java 类型转型是天赐之物。每一次当你从 Collection 里取到一个 object 的时候，你都需要把其转回原来的类型。因此，这些转型操作会出现在 N 多的地方。如果你改变了类型，那么你不一定能改变所有的地方。而编译器可能能检查到，也可能检查不到。

• 利用 Java 的冗余。 比如：Bubblegum b = new Bubblegom(); 和 swimmer = swimner + 1; 注意变量间的细微差别。

• 从不验证。 从不验证输入的数据，从不验证函数的返回值。这样做可以向大家展示你是多么的信任公司的设备和其它程序员

• 不要封装。 调用者需要知道被调用的所有的细节。

• 克隆和拷贝。 为了效率，你要学会使用 copy + paste。你几乎都不用理解别人的代码，你就可以高效地编程了。

• 巨大的 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。

• 保留你所有的没有使用的和陈旧的变量，方法和代码。

• Final 你所有的子结点的类。 这样，当你做完这个项目后，没有人可以通过继承来扩展你的类。java.lang.String 不也是这样吗？

• 避免使用 layout。 这样就使得我们只能使用绝对坐标。如果你的老大强制你使用 layout，你可以考虑使用 GridBagLayout，然后把 grid 坐标 hard code.

• 环境变量。 如果你的代码需要使用环境变量。那么，你应该把你的类的成员的初始化使用环境变量，而不是构造函数。

• 使用全局变量。

  1）把全局变量的初始化放在不同的函数中，就算这个函数和这个变量没有任何关系，这样能够让我们的维护人员就像做侦探工作一样。

  2）使用全局变量可以让你的函数的参数变得少一些。

• 配置文件。 配置文件主要用于一些参数的初始化。在编程中，我们可以让配置文件中的参数名和实际程序中的名字不一样。

• 膨胀你的类。 让你的类尽可能地拥有各种臃肿和晦涩的方法。比如，你的类只实现一种可能性，但是你要提供所有可能性的方法。不要定义其它的类，把所有的功能都放在一个类中。

• 使用子类。 面向对象是写出无法维护代码的天赐之物。如果你有一个类有十个成为（变量和方法）你可以考虑写 10 个层次的继承，然后把这十个属性分别放在这十个层次中。如果可能的话，把这十个类分别放在十个不同的文件中。

• 混乱你的代码。 使用 XML。XML 的强大是无人能及的。使用 XML 你可以把本来只要 10 行的代码变成 100 行。而且，还要逼着别人也有 XML。（参看，信 XML 得永生，信 XML 得自信）

• 分解条件表达式。 如：把 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 的事。

• 长代码行。 一行的代码越长越好。这样别人阅读时就需要来来回回的

• 不要过早的 return。 不要使用 break，这样，你就需要至少 5 层以上的 if-else  来处理错误。

• 不要使用{}。 不要在 if else 使用{}，尤其是在你重量地使用 if-else 嵌套时，你甚至可以在其中乱缩进代码，这样一来，就算是最有经验的程序员也会踩上陷阱。

• 琐碎的封装。 比较封装一个 bool 类，类里面什么都做，就是一个 bool。

• 循环。