查看原文
其他

张大胖改Bug

2016-12-16 刘欣 码农翻身

张大胖毕业以后过五关、斩六将,成功拿到了一个知名公司的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

我是一个线程

我是一个Java class

Java:一个帝国的诞生

JDBC诞生记

JDBC后传

一个不安分的JDBC驱动

JSP:一个装配工的没落

Javascript: 一个屌丝的逆袭

Spring本质系列(1) -- 依赖注入

Spring本质系列(2) -- AOP

Http 历险记(上)

Http 历险记(下)—Struts的秘密

三层架构和MVC那点事儿

Java帝国之 Java Bean(上)

Java帝国之 Java Bean(下)

计算机网络

我是一个路由器

我是一个网卡

TCP/IP之大明邮差

TCP/IP之大明内阁

TCP/IP之蓟辽督师

张大胖的socket

IE为什么把Chrome和火狐打伤了?

对浏览器村的第二次采访

节约标兵IE的自述

EMail诞生记

EMail诞生记(下)

数据库

小李的数据库之旅(上)

小李的数据库之旅(下)

张大胖学数据库

数据库村的旺财和小王


你看到的只是冰山一角, 更多精彩文章,尽在“码农翻身” 微信公众号, 回复消息"m"或"目录" 查看更多文章


有心得想和大家分享? 欢迎投稿 ! 我的联系方式:微信:liuxinlehan  QQ: 3340792577



公众号:码农翻身

“码农翻身”公众号由工作15年的前IBM架构师创建,分享编程和职场的经验教训。


您可能也对以下帖子感兴趣

文章有问题?点此查看未经处理的缓存