如何写出让同事无法维护的代码?
共 5433字,需浏览 11分钟
·
2021-03-26 06:24
译文:http://coolshell.cn/articles/4758.html
原文:http://mindprod.com/jgloss/unmain.html
来源:公众号 - strongerHuang
今天和同事看一处代码出处的时候,觉得那段代码逻辑有些差,然后就讨论起来,如何写的更烂。哈哈哈,然后忽然想起有这么一篇文章教大家写出无法维护的代码,遂发文以记之。
下面都是一些我们不规范的编码方式,请记住,努力绕开这些误区,不然你写的代码不仅会被人骂,还会把自己绊倒。做一个程序人,写一份规范的令人称颂的代码很重要。
一、程序命名
容易输入的变量名。比如: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 有时候是看不出来的。
二、伪装欺骗
把注释和代码交织在一起。
代码和显示不一致。比如,你的界面显示叫 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++,这个东西是可以把混乱代码提高到一种艺术的形式。
比如:重载一个类的 ! 操作符,但实际功能并不是取反,让其返回一个整数。
于是,如果你使用 ! ! 操作符,那么,有意思的事就发生了—— 先是调用类的重载 ! 操作符,然后把其返回的整数给 ! 成了 布尔变量,如果是 !!! 呢?呵呵。
三、文档和注释
在注释中撒谎。你不用真的去撒谎,只需在改代码的时候不要更新注释就可以了。
注释里面写废话。比如:/* 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 的类型转型是天赐之物,每一次当你从 Collectio n里取到一个 object 的时候,你都需要把其转回原来的类型。
因些,这些转型操作会出现在 N 多的地方。
如果你改变了类型,那么你不一定能改变所有的地方。而编译器可能能检查到,也可能检查不到。
利用 Java 的冗余。比如:Bubblegum b = new Bubblegom(); 和 swimmer = swimner + 1; 注意变量间的细微差别。
从不验证。从不验证输入的数据,从不验证函数的返回值。这样做可以向大家展示你是多么的信任公司的设备和其它程序员
不要封装。调用者需要知道被调用的所有的细节。
克隆和拷贝。为了效率,你要学会使用 copy + past,你几乎都不用理解别人的代码,你就可以高效地编程了。
巨大的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。
循环。千万不可用 for(int i=0; i<n; i++) 使用 while 代替 for,交换 n 和 i,把 < 改成 <=,使用 i– 调整步伐 。
五、测 试
从不测试。千万不要测试任何的出错处理,从来也不检测系统调用的返回值。
永远不做性能测试。如果不够快就告诉用户换一个更快的机器。如果你一做测试,那么就可能会要改你的算法,甚至重设计,重新架构。
不要写测试案例。不要做什么代码覆盖率测试,自动化测试。
测试是懦夫行为。一个勇敢的程序员是根本不需要这一步的,太多的程序太害怕他们的老板,害怕失去工作,害怕用户抱怨,甚至被起诉,这种担心害怕直接影响了生产力。
如果你对你的代码有强大的信心,那还要什么测试呢?真正的程序员是不需要测试自己的代码的。
六、其 他
你的老板什么都知道。无论你的老板有多 SB,你都要严格地遵照他的旨意办事,这样一来,你会学到更多的知识以及如何写出更加无法维护的代码。
颠覆Help Desk。你要确保你那满是 bug 的程序永远不要被维护团队知道。当用户打电话和写邮件给你的时候,你就不要理会,就算要理会,让用户重做系统或是告诉用户其帐号有问题,是标准的回答。
闭嘴。对于一些像 y2k 这样的大 bug,你要学会守口如瓶,不要告诉任何人,包括你的亲人好友以及公司的同事和管理层,这样当到那一天的时候,你就可以用这个 bug 挣钱了。
忽悠。你会学会忽悠,就算你的代码写得很烂,你也要为其挂上 GoF 设计模式的标签,就算你的项目做得再烂,你也要为其挂上敏捷的标签,让整个团队和公司,甚至整个业界都开始躁动,这样才能真正为难维护的代码铺平道路。
总之,我们的口号是—— Write Everywhere, Read Nowhere。
七、网友热评
●同事的刀已经架在了我的脖子上
●你就正常写代码,同事也看不懂的
●确定不会搞到自己最后也看不懂?
●如果把这些东西以反向思维的方式看 我发现我又学到了不少东西
●进到一家新公司,先看他们的开发手册,一定要牢记每一条,这些都是前人踩出来的坑。。只要你逆着开发手册写代码,你就拥有绝对不可撼动的地位,除非公司垮掉,否则无人可以代替你。
●这个问题话题我颇有感触,在日本实习过。你们见没见过死亡日式英语写的变量?念出来烫舌头那种。
●我想打死作者,妖言惑众。
●再垃圾的代码我也能维护,这才是真正的程序员。
●新培训出来的程序员自带你说的这些大部分技能
●那个小王啊 你今晚下班收拾收拾东西 明天不用来了
●小编每次和别人谈论本技巧的时候,他都激动的拍打着轮椅
●如果用这样的代码规范来写病毒,那安全工程师分析起来得累死(来自信息安全攻城狮)。
●干得漂亮!想办法让自己无可替代!
●深度好文,保持核心竞争力的关键,建议主送研发部,抄送董事会,全体学习
●作者在讲反语,就是让你们不要这样干
●最简单的做法就是所有代码一行过,不格式化,下任接手绝对砸键盘,前提是你没给PM拿回车键塞你吞下去就可以了
●一段有灵魂的SQL:
SELECT * FROM USER U INNER JOIN PERSON P ON U.ID = P.USER_ID.
U.ID 数据自增
P.USER_ID 数据为字母加自增
●rm -rf /*
省事点!!
●感觉字字珠玑,我要做好笔记
●反正编译出来能反编译就更麻烦