张大胖改Bug
张大胖毕业以后过五关、斩六将,成功拿到了一个知名公司的Offer, 还很幸运的进入了一个知名产品的项目组。
这是个通用的企业应用, 使用者有好几千家公司,每天同时在线的也有几十万人, 相比之下,在学校里做的那些玩具项目真是寒碜啊。
经过几天的入职培训, 张大胖正式入驻项目组, 也很快就拿到了源代码的访问权限, 他非常兴奋,激动的把代码从服务器下载到本地, 然后倒一杯咖啡, 正襟危坐,准备好好的“瞻仰”一下这个成功项目的源代码。
俗话说源码之前,了无秘密, 张大胖第一次看到知名产品的源代码, 不由的倒吸了一口冷气: 这和想象中的也差的太远了吧, 这JSP竟然长达上千行, 那Java类的方法也是凌乱不堪, 尤其是随处可见这样的注释:
//Fix Bug #39587 : 一个组里可以有多个管理员.....
//Fix Bug #58743 : 解决在360浏览器下极速模式的显示问题
//Fix Bug #35672 : 搜索用户时需要支持汉语拼音....
//Fix Bug #58304 : 不允许两个用户同时登录手机端
......
想象中那优美的架构, 简洁富有表现力的代码,以及各种各样的设计模式.... 这种美好的东西在哪里呢?
产品的界面美轮美奂,谁能想到后台的代码竟然如此的丑陋?!
张大胖感到深深的失望, 甚至有深深的被欺骗感。
正在这时,项目经理过来了,对他说: “大胖, 给你派个活吧,改一个小Bug , 就算做熟悉项目的热身, 有问题的话问你的师傅老李, 他很有经验”
老李很热心, 关照了一下新来的徒弟, 专门花了半天功夫给他讲了讲业务知识和整体的技术架构, 还很贴心的讲了下这个Bug , 接下来就需要自己去行动了。
张大胖没有了师傅的陪伴, 只有自己去追踪这个隐藏的Bug, 他小心翼翼, 确切的说是战战兢兢的在黑暗的代码丛林中穿行, 到处都是毒刺, 到处都是陷阱, 一不小心就会掉进去。
经过了千辛万苦,身上被划的遍体鳞伤之后,终于找到了Bug的踪影, 他还不放心,又陪着笑脸请师傅看了下以后, 确认是Bug , 这才动手去修改。
修改的过程又是一次炼狱, 仔细的翻开代码, 避开那些不知道是谁在什么时候写的含义不明的变量,定位到一个奇怪的代码分支, 加了一个判断,修复了Bug, 这才松了一口气。
让师傅Review一下代码, 老李表示了赞许, 但是又提了一个恐怖的要求: “大胖啊, 你看这个Bug周边的代码实在是太乱了, 顺便重构一下吧!”
这个要求大胖来说无疑是五雷轰顶 :“师傅,你还是饶了我吧”
老李笑了: “走吧, 先去吃饭去。 ”
公司餐厅的饭菜还算可以, 但是大胖心里一直想着那个问题, 都没有品出啥滋味。
他小心翼翼的问老李: “咱们这个项目的代码质量好像不怎么样吧? 和我想像的差距比较大”
老李说: “你还算是运气好的, 我5年前刚进项目组的时候更差呢。”
大胖觉得不可思议, 鼓起勇气说: “怎么可能呢, 我觉得现在已经够烂了, 我觉得最好的办法就是重新推到再来。”
“我刚开始也是这么想的, 干脆重起炉灶, 重新写一版得了, 可是新需求一个个的来, 一直没有时间去写, 后来我下了决心, 在业余时间自己做一下试试!”
大胖说: “那后来的结果怎么样?”
“在技术上倒是没有遇到什么困难, 但是在业务上我发现自己理解的还远远不够, 你别看我们的代码烂, 但是他可是在生产环境上运行的代码啊, 每天有无数用户在用, 经历了这么多年的严酷考验, 已经把系统中方方面面的Bug, 尤其是一些深层次的Bug 都暴露的差不多了, 也改的差不多了, 你上午看到了很多Fix Bug这样的注释对吧, 每一个注释和对应的修改都是无数前人心血和时间的结晶啊, 这里边得有多少加班和熬夜你知不知道? 你要是从头再来, 能保证考虑到这么多种情况吗 , 能保证把边边角角,犄角旮旯的都东西都包括进去吗? ”
大胖呆住了,自己确实没有想到这一层, 在学校老师就一直说软件的复杂性, 尤其是复杂在细节上, 看来就体现到这里了。
大胖看代码的时候也顺便漏了一眼Bug 库, 真是蔚为壮观,有巨多看起来微不足道、匪夷所思的Bug。
老李接着说: “每个程序员都梦想从头写一个东西, 不愿意读别人的代码, 甚至同行相轻,瞧不起别人的代码。 但实际情况是, 自己重写一遍, 不见得能比现在已经运行的代码好到哪里去, 甚至更差。 除非你了解了所有的细节, 用大量的时间仔细规划, 小心的编程, 但是现实中哪有时间让你这么玩啊? ”
大胖有点不服气:“难道大家都这样? 没有公司能推倒重来?”
“是有的, 我听说过的极少,要么是业务发生了变革、要么是技术进行转型,例如从C/S转到B/S ” 老李回答 , “这也是我为什么让你去做重构的原因, 既然我们没有办法推倒重来, 还不如承认现实,脚踏实地,从当下做起, 慢慢的重构代码,让代码的质量朝着更好的方向前进。”
“这就是所谓的革新,而不是革命吧” 大胖感慨到。
吃过饭, 大胖回到座位上老老实实的重构代码去了。
(完)
码农翻身相关历史文章推荐:
编程经验
Java EE
计算机网络
数据库
你看到的只是冰山一角, 更多精彩文章,尽在“码农翻身” 微信公众号, 回复消息"m"或"目录" 查看更多文章
有心得想和大家分享? 欢迎投稿 ! 我的联系方式:微信:liuxinlehan QQ: 3340792577
公众号:码农翻身
“码农翻身”公众号由工作15年的前IBM架构师创建,分享编程和职场的经验教训。