|
|
51CTO旗下网站
|
|
移动端

讨论 | 接手到垃圾代码,你要不要辞职?

每当程序员接手别的程序员写的代码往往都会吐槽一番:代码写的烂,可读性差,不美观,用的语言有问题......反正任何别人写的代码在他眼中都是“烂代码”,认为自己总能写的比他们好。

作者:小叔不浪何以撩天下来源:闰土小叔|2018-11-19 09:02

 

事情是这样的,前几天在朋友圈,我看到一朋友发表了一条说说:“入职新公司,从重构代码到放弃”,我就问他怎么了?

他说,刚进一家新公司,接手代码太烂,领导让我先熟悉业务逻辑,然后去修复之前项目中遗留的 Bug,实在不行就重构。

关键是,离职的那位仁兄走之前,还跟我在 QQ 上说,老哥辛苦了,我写的很乱真不好意思,但我是故意的。这几天,我都在想,要不要离职?

听完他的遭遇,我想先说一句,离职那位仁兄太狠了!!!

其次,我相信,这个问题很多人都遇到过。每个新人去一家新公司都会觉得公司代码很烂,可能是因为他没被产品虐过。

其实,这种新员工很不错了,一看就是真萌新,重构这种事,老板看不到  KPI,出了事还得自己兜底,还会得罪人,费力不讨好,何必呢!

老前辈的警世良言一定要牢记:重构一时爽,头发不再长。

还有,工作了 10 年的码农,到了新公司,绝不会说这种话。

不过话说回来,能觉得模块代码烂,还能撸起袖子去改的程序员,真是有很难得的一种精神。

我身边 90% 都只是嘴上说,绝对不动手干的。下一次,下一次,下一次,无数的下一次造就了烂代码。

要不要重构?

很多人口中的重构就是重写,而不是去改善现有的代码。

现在的有些工程师太浮躁,动不动就要重构,就跟很多的团队动不动就要写一个框架一样。

说实话,以我个人的经验来说,绝大多数开发人员到新公司后,都会觉得代码很烂。

但通常他不了解业务逻辑是怎么变化的,这种代码是在什么情况下写出来的,有什么特殊的背景(除了真的是乱写的,绝大多数的“烂代码”一般都是有原因的:业务需求改改改,这个需求明天就要上线等等等等),有多少坑(很少有人能在极短时间内把所有的坑都找出来)。

如果贸然去重构,风险非常大。而且再说难听点,就算重构完了,也有可能是一堆新的“烂代码”替换老的“烂代码”。

所以,进了一家新公司,别动不动就重构,先了解项目的业务逻辑。

要不要离职?

在我看来,如果仅仅因为接手代码太烂,就选择离职,那么你跳槽到下一个公司依然会面对同样的现状,因为几乎每个人,都会觉得自己公司的项目代码很烂。

写些烂代码,拯救广大同胞


我们先说说造成这种现象的原因是什么,首先,我们得相信,没有任何一个人故意把自己的代码写的很烂,每个人都想把自己的代码写的很优雅,扩展性很好。

但是可能由于当初水平不够,在当时看似还不错的代码,在别人看来就是所谓的垃圾代码。

我们每个人都在进步,别说别人了,你现在看你三个月之前的代码,可能你都会觉得写的很垃圾,如果你没有这种感觉,只能说你在止步不前。

其次,技术更新换代太快,市场的变化也太快,产品自然也一直在演变,也许在当时看起来还不错的代码,随着时间的推移,功能的更新,代码的堆彻,慢慢就变成后来者眼中的烂代码了。

如何从烂代码中爬出来 ?

假如你真的要接手别人的代码,怎样才能不被玩死呢?

虽然你可能会说,听了很多道理,依然交接不好代码,可作为经常被别人的代码玩得崩溃的老司机,我有些话如鲠在喉,不吐不快。

当你被要求接管要离职的程序员的代码时,如果能注意以下几点,就有可能活着从他的代码里爬出来。

产品需求与业务流程文档

离职程序员之项目交接

产品需求与业务流程文档是你先要找到的,你接手的代码,必然和某个产品需求相对应,必然实现了某个业务流程,先了解产品需求和业务流程,才能更好的读代码。

程序员离职后的代码没人敢接手

假如你的团队就是没文档,OK,也可以要求离职或转移战线的这位程序员把需求描述出来,把业务流程画出来。

测试环境

虽然看着烂,还能正常跑,但又不敢随便修改

了解了产品需求和业务流程,***能体验一下软件,从用户的角度来理解软件的使用。

这个时候你要么需要生产环境,要么需要测试环境。哪个环境不重要,重要的是,你需要一个能 Run,能体验的软件。

业务流程在代码层面的体现

负责交接代码给你的那位同事,要么在办离职,要么已经介入了其他产品,眼下很可能已无心恋战。

但你心里要清楚,只有他才能提供代码层面的东西,比如类图、模块划分说明、数据流图、时序图、状态图等等。

所以,你需要他整理一些文档和图表出来。你可以告诉领导你需要上面的东西,让领导和他沟通,让他在离开之前准备好这些文档给你,并留一些时间以便你熟悉。

读代码,不死不休

读别人代码是一种怎样的体验?

有了产品需求,有了业务流程,有了代码设计相关的文档和图表,接下来你就该死磕代码了:

while(不懂)
{
    读
}

开发环境与调试

新接手了一个代码库,最初调 Bug 的样子

有的产品需要比较复杂的开发环境配置,一定要提前做好,让即将离开的同事辅导你搭建好开发环境,这样你就可以利用“调试”这个强大的武器来快速理解代码了。

调试是接手别人代码时的利器,如果你看不明白一个业务在代码层面是怎么体现的,也看不懂代码之间的调用关系,那***的办法就是调试。

从一个业务的起点所对应的代码开始调试,一步一步跟进去,就能快速理清函数调用链。

树立可实现可衡量的目标

当我以为这是***一个 Bug,改完就可以去吃饭时……

程序员的工作交接,尤其是代码交接,怎样才算顺利完成呢?

这简直就是一个谜!没人说得清楚。所以,你***给自己梳理一些可衡量的、可实现的目标。比如读懂 A、B、C 三个业务流程……

***,找一个 Bug 或者一个新增的功能,带着目的去读代码、修改代码,有目的,有目标,有时间盒,就容易投入,容易读进去,容易掌握与 Bug 或新增功能相关的代码的逻辑与流程。

输出、分享与重构

在新接手的代码库上 Debug

你在读代码时,如果能撇开给你交接工作的程序员提供的文档,按自己的理解,自己绘制类图、数据流图、时序图、关键业务流程对应的函数调用关系链等,就能更快的掌握别人的代码。

如果你还能将你理解到的东西,讲给其他人听,并且讲明白,那 OK,你真的理解了别人交接给你的代码。

再进一步,如果你在理解现有代码的基础上,可以识别出哪些部分实现得逻辑不清晰或有待改善。

然后可以结合业务与自己的理解将其重构,那就真的是完全接手了别人的代码,别人的代码与你的代码就没有差别了,它们终将成为你的代码。

最重要的事

如果你总是看见代码多就发愁,看见代码脏乱差就诅咒埋怨,看见代码逻辑复杂就头疼,搞不清调用关系就放弃,那你可能永远也变不成代码的主人,只能一次又一次被代码蹂躏。

所以,交接代码最重要的事儿,就是:不要被浩渺如烟并且陌生怪诞的代码吓得不敢动弹,现在就开始读,立刻,马上,坚持十分钟,再坚持十分钟,你就能妙悟真谛。

***,当你再碰到这种事儿的时候,请记住老前辈的这句警世良言:那些不能将你击溃的代码,都将成为你成长的垫脚石。

如果你身边有朋友,正在接手别人代码,被虐的想要重构,让他看这篇,普度芸芸众码农,是我义不容辞的责任。

作为程序员,你接手了烂代码会怎么办?扫描下方二维码,关注51CTO技术栈公众号。欢迎在技术栈微信公众号留言探讨。小编将精选出最有价值的三条评论,分别获得 50、30、20 元 的 红 包 奖 励,活动截止时间 11 月 23 号 12 时整。

【编辑推荐】

  1. 最霸气的程序员辞职理由,阅后即焚_技术栈微信半月刊第16期
  2. 那些奇葩的辞职理由!程序员“要命不要钱”最心酸
  3. 应届生入职年薪25W+?深扒互联网行业发展现状!
  4. 程序猿要注意,入职签订竞业限制,离职补偿金应该这么算!
  5. 程序员:改完这个Bug,我就辞职......
【责任编辑:武晓燕 TEL:(010)68476606】

点赞 0
分享:
大家都在看
猜你喜欢

订阅专栏+更多

16招轻松掌握PPT技巧

16招轻松掌握PPT技巧

GET职场加薪技能
共16章 | 晒书包

289人订阅学习

20个局域网建设改造案例

20个局域网建设改造案例

网络搭建技巧
共20章 | 捷哥CCIE

645人订阅学习

WOT2019全球人工智能技术峰会

WOT2019全球人工智能技术峰会

通用技术、应用领域、企业赋能三大章节,13大技术专场,60+国内外一线人工智能精英大咖站台,分享人工智能的平台工具、算法模型、语音视觉等技术主题,助力人工智能落地。
共50章 | WOT峰会

0人订阅学习

读 书 +更多

主流ARM嵌入式系统设计技术与实例精解

本书重点介绍了主流ARM应用系统的开发与实践。全书基于目前较为通用、流行的ARM处理器,介绍了其原理、硬件结构、硬件电路设计与开发和软件...

订阅51CTO邮刊

点击这里查看样刊

订阅51CTO邮刊

51CTO服务号

51CTO播客