程序员在加班时间写的代码 bug 骤增

作为一个最底层的程序员 ,我先记录一些只有底层程序员才会知道的事情。如果多年后,我违背自己进入这个行业的初心,走上管理岗位,也能回想起一些禁忌,避免一些错误。

其中最重要的就是这条: 不要相信一个程序员在加班时间写出来的代码 。

(软件工程的学说表明,连正常时间好好写的代码,也不要太相信。不过这不是本文的重点,略过不提。)

(不懂代码的人,看到本文中的Java代码可以略过,不影响理解。)

创造力的时限

写代码,与写文章、绘画、思考复杂问题,并没有本质上的区别,都是创造性的活动。

每个人的创造力,都会随着身体状态而波动。广为人知的是,一个人年老体衰后,相比年富力强时,创造力会急剧下降。其实,人每天的状态起伏,也同样会剧烈影响这一点。

如果是拧螺丝,那么在精疲力尽、拧不动以前,身体状态对结果不会产生太大影响。因为拧螺丝的指标非常简单——拧紧,要做的事也非常机械化——拧,直到它紧,换下一个。

但如果是写代码,有些事,是不能在状态不好的时候完成的。

比如,在Java里,遍历一个外部的 List ,做一些处理。如果状态不佳、做事前想的东西少了点,那么很可能直接这么做:

public void handleAList(List<Integer> aList) {
for (int i = 0; i < aList.size(); ++i) {
// Do sth with List#get(int)
}
}

这样做是从C/C++带来的一种很直观的做法。有什么问题吗?

假如外面传入的 aList 是一个 ArrayList ,那么 List.get(int) 的时间复杂度是O(1),算上外面那重循环则是O(n);而假如 aList 是一个 LinkedList ,那么 List.get(int) 的时间复杂度是O(n),算上外面那重循环则是O(n2)!

(为不懂算法时间复杂度评估的人解释下:在这个场景下,O(n)代表最优、最快,而O(n2)代表不可接受地慢。)

如果时间充分,那么可以去查看 handleAList() 的调用位置,看看它传递的是哪种List ;而如果思考得够充分,考虑到这两种情况都有可能,那么代码就会做兼容处理,改成这样:

public void handleAList(List<Integer> aList) {
for (int i : aList) {
// Do sth with i
}
}

这使用了for-each语法,实际上是用Iterator来做遍历,无论对哪种List都是总共是O(n)的开销。

注意,这通常不被看做一个bug,普通的黑盒与白盒测试都是无法发现的。只是你的App会比较卡,或者后台会比较慢。当需要解决这种性能问题时,可能需要非常经验丰富的程序员,在海量代码里找数周时间——而这一切,在开发之初,只要那个程序员状态好一点,就可以避免。

一个人,每天的创造力是有时限的。在时限外,他不再是一个优秀的创造者,而是一个笨蛋。

(为了便于理解,这个例子非常简单,以至于不够贴切。对Java来说,优先使用for-each或Iterator来遍历,已经是一个共识,是技术素养的一部分。)

失误率的飙升

程序员在写代码的过程中,每天做得最多的应该就是等价变换。

if (isSthTrue()) {
// Take some actions.
}

变换成

if (!isSthTrue()) return;
// Take some actions.

这只是最简单的一种逻辑反转,实际上还有更多、更复杂的形式。通过这类变化,对代码做出调整后,程序员可以把代码变得更好,或者做到以前不能做的事。

而在加班时间、大脑不那么清醒的情况下,很可能会写成这样:

if (isSthTrue()) return;
// Take some actions.

区别仅仅只是少了一个符号,而意义则完全走样。

这个例子比较简单,出错后也很容易在调试过程中发现、纠正。但是,请不要怀疑,的确会有程序员为了这么个简单的问题,调试整整一个晚上!

( 、 ! 、 i ,(字体未配置好时)本就难以区分,眼睛疲劳昏花时,在数百个字符里扫来扫去,难以分辨 (!i 中是否少了个符号,也并不奇怪。而如果换成第二天早晨,很可能只需要瞥一眼。

大多数管理者,往往会对熬夜的程序员给出一些肯定,并且允许第二天可以休息一天(有些甚至只给一早上)。但如果他们知道内情,会发现自己其实亏了一天。如果程序员正常下班,第二天花一小时解决这个问题,剩下的七个小时可以继续开发。

还有很多比这复杂得多的变换,或其它类型的代码改动,即使在大脑清醒的情况下也需要花费一些时间,认真思考、小心调试。而如果来了一个问题,你说“必须要今天下班前搞定”,那么程序员会很烦躁,并且越来越烦躁。

烦躁的后果

一件需要冷静思考、谋定后动的事,如果逼迫人们在烦躁的情况下去做,那么往往会得到意想不到的糟糕结果。

我有一位前同事,技术实力且不论,心性也不太稳(实际上,像我这种少年老成、未老先衰、找不到妹子都不急的青年,还真不多)。他是一个可以解决问题的人,但是在烦躁的情况下,也经常做出令我瞠目结舌的事。

比如,有一天,项目组要求某个bug必须解决。他搞到晚上9点还没搞定,找我帮忙。我当时水平也很差,不然也不会那时还在加班,没能帮他解决,只是因此而知道这件事。他后来在10点半时采用了一个规避方案,然后下班了事。

具体一点是这样的:在一个class中,有多个地方调用同一个Method。其它地方没有问题,唯独某个位置的结果不正确。他改成这样:

private boolean isSthTrue(int sth) {
// Implementation A
}
private boolean isSth1True() {
// Implementation B
}
private boolean isSth2True() {
// Implementation C
}

本来 isSthTrue() 是可以做通用判断的,他没有在规定时间内找到根本原因(Root Cause),实际上当时他也根本没有往发现根本原因的方向去查找代码,而是一晚上都在做一些无效的调试。最后没办法调试出好的结果,于是给出问题的地方一个特殊处理——新增了 isSth1True() 和 isSth2True() 去那个出错的地方顶替。结果,那个bug的确是解决了,但是后来带出来了另外一个bug。

不过他也达到了目的,当天下班了。

而后来,我在代码里发现了另外一组更早就有的接口。

private boolean isTrueSth1() {
// Implemented like B
}
private boolean isTrueSth2() {
// Implemented like C
}

我问了一下这两个Method的作者(另一位同事),他根本没有看到有 isSthTrue()。

这件事的最终结果是,解决了一个bug,后来又引起了多个bug,连我也跟着一起焦头烂额。

借着这个例子,回头再说一下 创造力的时限 。

这位同事,之所以不去找Root Cause,是因为项目组的催逼和自身的烦躁,他平时是可以解决问题的。但是为什么一个简单问题会这么难解决,为什么代码里之前就有一套他要的Method,他却新写一个?

外部代码环境就不说了,这个class共有2000行。2000行可能并不是特别直观的数目,既不能说多,也不能说少,取决于这个class干什么事。

后来,另一个比较老道的同事,重构(refactor)了这个class,只用了不到500行——这就说明了一个问题,这个class之前就太过冗余。

约半年后,我水平也提高了些,总体的项目时间也松散了些,我花了六周重写(rewrite)了这个不大的代码库。这个class最终只用了100行,部分功能都独立封装到了其它class中。

如果之前,在这个代码库写就之初,就能有一个充分的时间做一个好的架构设计,不需要rewrite就可以只有100行;而如果时间不太充分,却能给 应有的时间好好写,也起码能有refactor后的水平,也就是500行。无论是100行,还是500行,后面出的一大堆问题,都不会出现,或者更容易解 决。

这个代码库是怎么来的?

当初某领导,交给了一个比较厉害的同事,只给一周时间。这位同事加班加点,一周当成两周用,从别的代码里剥离、拼凑出来了一个编译能通过的东西——这就是交给我们维护的代码库。

来自项目最底层的复仇

前面说的,无论是写出隐蔽的bug,还是解决一个带出俩,其实都是这类事情的阳光面。你没看错,这是阳光的一面。

还有我不想多说的阴暗面。

前面说的事情,没有一类是故意的。无论出事的原因是程序员的技术素养不足、加班情况下大失水准、还是原先的代码就非常容易诱导失误,都是程序员在认真努力的情况下,不可自控地犯错。

还有一类是故意的。

比如,去年(2015)携程那小哥儿,就是怒删数据库。当然,他不是为了加班严重而如何如何,而是心爱的运营妹子被公司某高层给……(另有一说,虽然有什么内部的QQ、微信截图,但这仍然是谣言,实际上是黑客攻击。)

什么程度的压迫,就会得到什么程度的反抗。

要知道,即使是很努力地去做,也仍然可以出各种问题。而如果要故意捣乱,很多手段,虽然不会引起老板的注意,甚至可以不被认真的代码审查者(reviewer)警觉,但是会客观地影响产品的品质,让用户讨厌一个产品,或者让一个爆款产品最终失败。

反正埋了雷,领了工资,跳下一家便是——要么给股票、期权,要么充分洗脑,或至少给出足够的加班费(几年后的医疗费),否则就是这个后果。

我只能说,就我个人而言,最多辞职,不会故意乱搞。这关乎职业道德,关乎我是否意念通达、心境澄明。(坐等穿越去修真:P)

但是,我不能用自己的道德准绳去要求别人,对吧?

而且,永远不要指望一个人在承受不道德的对待时,仍然能谨守原来的道德。

结语

作为一个软件项目的领导者,你在要求某个程序员加班时,其实就已经在冒险;而如果你经常这么干,不要奇怪为什么项目总是延期,或者一到关键时候,总有突发事件。

只要试验次数够多,可能性再小的事也会发生;而只要试验次数更多,小概率事件也会连续发生。

所以,最理智、客观的观念就是:欲速则不达, 不要相信一个程序员在加班时间写的代码 。

文章转载自 开源中国社区 [http://www.oschina.net]

时间: 2024-08-30 20:29:13

程序员在加班时间写的代码 bug 骤增的相关文章

不要相信一个程序员在加班时间写出的代码这是真的吗。。。。。

作为一个最底层的程序员,我先记录一些只有底层程序员才会知道的事情.如果多年后,我违背自己进入这个行业的初心,走上管理岗位,也能回想起一些禁忌,避免一些错误. 其中最重要的就是这条:不要相信一个程序员在加班时间写出来的代码. (软件工程的学说表明,连正常时间好好写的代码,也不要太相信.不过这不是本文的重点,略过不提.) (不懂代码的人,看到本文中的Java代码可以略过,不影响理解.) 创造力的时限 写代码,与写文章.绘画.思考复杂问题,并没有本质上的区别,都是创造性的活动. 每个人的创造力,都会随

当程序员不相信自己写的代码的时候怎么办?

问题描述 几个月的闲置,让程序员的我开始怀疑自己的能力了,不敢相信自己能胜任工作了咋办? 解决方案 解决方案二:项目空闲期么?学习学习吧解决方案三:是啊!都闲了快三个月了,有时突然感觉(害怕)自己还能做东西?解决方案四:两条出路1.继续闲着,好处是不用做事还有钱拿,缺点是人可能闲疯掉,浪费大好光阴...(闲可能没底,是外包维护项目么-,-我有同事闲过1年半)2.跳槽解决方案五:打酱油的路过解决方案六:闲了三个月,也就意味着老板不会白养你们的,具体的自己看着办吧.解决方案七:学习学习吧忙的时候可能

程序员必知35个jQuery 代码片段_jquery

jQuery如今已经成为Web开发中最流行的JavaScript库,通过jQuery和大量的插件,你可以轻松实现各种绚丽的效果.本文将为你介绍一些jquery实用的技巧,希望可以帮助你更加高效地使用jQuery. 收集的35个 jQuery 小技巧/代码片段,可以帮你快速开发. 1. 禁止右键点击 $(document).ready(function(){ $(document).bind("contextmenu",function(e){ return false; }); });

程序员都应该学写“规范”的代码

在过去的7年半时间里,我带过的软件实习生超过一打,也看到过数以百计的学生和毕业生的档案.我发现很多事情他们都需要学习.或许你会说,我说的不 就是某种特定的技术.算法.数学,或者其他特定形式的知识吗?没错,这的确是需要学习的,但却并不是最重要的事情.他们需要学习的最重要的东西是"自我规 范".这些规范就是:尽可能地写出最简洁的代码:如果代码后期会因为改动而变得凌乱不堪就得重构:尽量删除没用的代码,并添加注释. 我花了很多时间来敦促这些实习生去学习这些内容.我经常会问他们,怎么样才能成为一

程序员每天到底可以写几行代码?

对于特定的人,在大致时间段里他所能写的.确定质量的代码基本上应该是个确定值. 这点似乎显而易见,但事实上大多时候却总是被忽视. 如果项目负责人总是认可上面的基本点,那么任何项目的日程就应该以此为前提,而不是以此为变量. 假设说一个项目被估计为1万行(SLOC),团队平均每人每天可以写100行代码,如果团队中有5个人,那么就应该至少为编码保留20整天.   说到这里,为避免误解,要区分一下编码速度和生产率这两个概念. 项目管理中常用的一个数据被称为生产率,用代码行计算时,会被表示为SLOC/MM.

程序员,为什么千万不要重写代码?

作为 100offer 程序员拍卖的运营,我们常常和用户交流讨论,有一个话题经久不衰:程序员入职新公司后接手已有的代码,怎么处理? 程序员都有一颗工程师的心,所以当他们到一片新的场地想做的第一件事就是,将旧的一切推倒重来.是的,他们决不会满足于简单的增量劳动. 或许这种微妙的心理定位可以解释:为什么程序员进入新项目组后宁愿丢掉旧代码重新写,也不愿意修修补补,他们认为旧代码简直一团糟. 但是,事实上真是这样吗?你之所以认为旧代码一团糟,其实是由编程的一个基本定律决定的,那就是:写代码容易,读代码难

程序员应该避免的 5 种代码注释

你有没有这样的经历:别人审查过你的代码之后给出的注释,你认为是没有必要的?注释代码是为了提高代码的可读性,目的是为了能让其他人更容易理解你的代码. 我特别讨厌这5种注释类型以及制造它们的程序员.希望你不是其中之一. 1.自以为很了不得的程序员 public class Program { static void Main(string[] args) { string message = "Hello World!"; // 07/24/2010 Bob Console.WriteLi

每个程序员需掌握的20个代码命名小贴士_其它综合

代码中到处都需要命名.作为程序员,我们得给类命名,给变量命名,给函数命名,给参数命名,给命名空间命名,等等等等.下面有20条小贴士能帮助你提高你的命名能力. 1.使用能够表达意图的名字 名字得能告诉我们它要做什么,为什么存在,以及是如何工作的.选择能够表达意图的名字,将更有利于我们理解代码. int d; // elapsed time in days int elapsedTimeInDays; int daysSinceCreation; int daysSinceModification;

传奇程序员:8岁写游戏独力开发赚1亿美元

到2013年初,Minecraft的注册用户已经不计其数,付费用户数量也已经超过9000多万.2011年10月,发布了移动版Minecraft,在App Store的付费应用中,仍旧排在前三. 这个如今被越来越多人奉为神作,并让无数人深深沉迷其中的游戏最初却几乎由Markus Persson一个人独力开发,与真名相比,他在网络上的昵称Notch无疑更加有名,许多人将他视作独立游戏开发大神和独立软件圈子的标志性人物. 靠着Minecraft,Notch在2012年的收入超过了1.1亿美元,那么他究