代码重构！你敢吗？

“孩子，阁楼的那扇小门，轻易不要打开哦……”童话里的老人一脸讳莫如深。

那锁在阁楼里的老代码，你敢动吗？非动不可的时候，又要怎么动呢？

作为微信早期员工、腾讯资深架构师、技术 Leader，大飞从高中便开始接触编程，大学通过校招实习生进入腾讯，这么一呆便是十年。算是微信部门的“资深大牛”，做过一线技术小兵，带过团队，做过产品，在工程技术上有丰富的经验。而顶着一系列闪光标签的他在刚转调业务部门的档口，也撞上了代码重构的麻烦事儿……

作者 | 大飞

本文经授权转自大飞码字

今天讲述一个代码重构的经历。

2014 年，我从基础架构部门，转调到业务部门。技术负责人想让我搞定业务系统的稳定性问题。

当时的业务系统确实存在不少问题，不过我初来乍到，对整体系统不熟悉，就想在熟悉一段时间后再动手。没想到，后面是事情自己找上了门。

那是一个周六的早上，我当时不在广州，而是去了深圳的一个同学家。当时跟我同学聊得尽兴，就一直没看手机，间隔了一个多小时后，我打开微信一看，工作群里有几百个未读。看到我们技术负责人的头像一直在闪动，就意识到应该是出大问题了。

原来，是一个核心的业务系统出了一个 bug， 影响到了一个重要的商户。

他们本意是给一个用户推送一条特定消息，消息里面还包含了一些隐私信息。不巧，一个新来的同学因为一个新的需求，修改了那部分的代码，引入了一个 bug , 导致本来是发个一个特定用户的信息，发给了一堆人。

商户相当不满，后来是部门的公关出面，才将事情平息下来，经理那边也因为这个事情，拉我们到办公室批了一顿。

技术负责人也压力山大。我们几个人在会议室里讨论了很久，最后大家都觉得如果要比较好地杜绝此类问题，除了要加强各种测试等措施外，还有一个，就是要重构现有的代码。

因为这个系统是最核心的业务系统之一，而且几经易手，当时的代码已经变得极难维护，里面各种 if else 的分支，还有长达一千行一个的函数，注释不全，文档也不足，要想长期的维护下去，这个技术债是非偿还不可了。

大家面面相觑，虽然知道重构是最好的解决方案，但大家都不想搞呀。

后来，我考虑到，初来这里还是要做些事情才能得到大家的认可，就硬着头皮，把重构的这个任务给接了下来。

确定重构的范围

接下这个任务后，我和项目组的成员就开始分析这个系统。

发现这个系统的业务流程很长，涉及到几十个子系统（微服务），还依赖几个外部门的服务。如果全部重构下来，估计一年都做不完，而且风险极大，重构一年的系统，然后再上线，谁敢呀，而且到那时，说不定黄花菜都凉了。

这样肯定不行。

我们就重新梳理了一遍，把整个系统划分成了三个部分，我们发现中间部分的修改最频繁，出问题的频率也最大，就决定先重构中间流程部分的代码。

项目规划部分，我们对项目进行了分期，中间部分的重构作为第一期，其他两部分可以作为二期、三期项目来做。一个是可以极大地减少压力，使得事情更加容易把握，另一个是间隔一段时间有产出也能给团队带来信心。

设计好验证的方式

当确认好重构的范围后，接下来的事情，就是要考虑如何来验证重构后的代码了。

这个是重构代码最重要的一个部分，如果没办法验证重构代码的正确性，你是不敢上线的，就算硬上了，也会睡不好觉。

一般重构代码的验证，可以采用测试代码，测试用例覆盖的方法（这部分可以参考 《重构》）。但我们发现，要重构的这个部分不能采用这种方式来验证。

因为业务逻辑很复杂，而且涉及到太多的外围系统，一个是测试用例很难覆盖全面，另外一个是没有办法可以很好地隔离外部系统的依赖。

我们分析了整个系统，发现这个系统的功能是，接受商户过来一个请求，然后进行各种权限、角色等的判断，再根据各个参数去各个依赖系统拉取数据，最后组装出一个新的包，再把这个新的包发送到隔壁部门的下游系统。

后来，我们想了双流程验证的方案。

我们将重构部分的代码全部封装起来，然后提供一个新的接口，一个请求进来后，我们分别执行旧的业务逻辑，也将请求发给新接口。在流程的最后，我们将新旧流程构造出的字段，进行逐个字段的对比。新流程只验证正确性，不做实际的输出。

为了保证验证的效果，验证要在线上进行，所以还要再结合后面的灰度流程。

尽一切努力，搞清重构代码的逻辑

当我们确定好验证方式后，接下来就是正式的工作了，重构代码。重写代码本身是不难的，但遇到的麻烦是，几乎没有文档，注释也很少，通过看代码只是搞懂了百分之五十左右的逻辑，还有一大部分的逻辑，无法理清楚。

后来，我们想到一个办法，把代码版本管理系统的 log 全部拉出来。通过 log 我们找到了各部分逻辑不清晰的代码的负责人，然后一个一个地去跟他们聊，跟他们请教。运气好的是，大部分人员都还在，中间还跟产品经理聊了不少，终于，把整个逻辑搞懂了百分之九十几。

因为有了上面的双流程验证和下面灰度逻辑，我们觉得，可以开始上线验证了。

灰度，一定要灰度

接下来，就要开始我们的灰度验证流程了。因为故障的影响很大，所以我们灰度得特别小心。

我们内部有灰度系统，但内部系统的灰度粒度比较大，为了保险我们需要更小粒度的灰度，所以我们自己写了灰度的逻辑代码，直接嵌入到了系统里面。

一开始的时候，极度小心，几乎是一个商户一个商户灰度的。灰度完后，我们每间隔一段时间，就分析一遍 log 和监控，看看有没有隐藏的问题。

最终，我们确实在这个灰度的过程中，发现了不少问题，不过因为涉及的用户很少，都没有造成大的影响。

这种极小范围的灰度，大概持续了一周左右的时间，后面慢慢加快了灰度的速度。大概花了一个月的时间，覆盖了全部的用户。

中间过程几乎没有出现什么大的问题，可以说是比较成功的一次重构。

控制好各方预期

最后一个点，跟技术无关，是关于相关人员的预期，包括上级的预期，同级的预期，下属的预期。

我当时知道这个项目有难度，自己心里也没底，所以跟上级说去试一试，后来谈成可以在过程接受两次中等故障。当然最后结果比预期好，没有一次中等故障，只有过两次小故障。

同级这块，也是跟大家说，尽力去试试，不过确实不是很有把握，也算是降低了他们的预期。

对于下面的兄弟，我是跟大家说，这是一件可以稳固我们团队地位的事情，拼死也要拿下这一仗。后面大家都很齐心，一起完成了这个在当时看来挺难的一个任务。

这个策略，是我第一年工作的时候，我导师告诉我的，内紧外松。这样外面对你的预期比较低，内部却很拼命地做，最后的结果，往往比较容易超出大家的预期。

我觉得这是一个很好的策略。

结语

最后，我们顺利完成了这次的重构任务，也做出了我们在新团队的影响力。后面再来回顾，发现我们做对了不少事情。没有一上来就开干，因为信心不足，反而是小心翼翼，也正是因为信心不太足，同时在不断降低外界的预期，最后一步一步，紧遵流程，获得了不错的结果。

声明：本文经作者授权转载，如需转载请联系原作者。

 热 文 推 荐 

System.out.println("点个在看吧！");
console.log("点个在看吧！");
print("点个在看吧！");
printf("点个在看吧！\n");
cout << "点个在看吧！" << endl;
Console.WriteLine("点个在看吧！");
Response.Write("点个在看吧！");
alert("点个在看吧！")
echo "点个在看吧！"

你点的每个“在看”，我都认真当成了喜欢