CodeReview数字化管理工程实践-京东云开发者社区

## 一、CodeReview（以下简称CR）到底能给团队带来什么？
通过代码质量把控，让大家培养良好的编码习惯，并且逐步形成个人的编码风格，使个人能力得到提升，从而带动整个团队的能力提升。

一般来说，很多团队在CR前期重点会是找问题（代码规范、潜在缺陷、BUG，代码设计等等），而后期随着问题的逐渐减少和习惯的逐步养成，工程师交流文化的营造将转化成重点，中期当有大批新人加入时，问题找茬将又上升为重点，如此复始。

总结一下，大多数情况下，找问题会是CR活动启动的初衷，但越到后期它更大的意义将演变成工程师交流土壤的培育和人员成长的促进。

Code Review在软件工程中是最为重要的提高代码质量，保障业务逻辑正确，同时建立工程师文化的环节。通过高质量的code review可以带来如下好处：

a.通过交叉代码审查，减少代码基本问题。

b.通过业务逻辑审查，确保业务理解不一致导致的逻辑问题。

c.高级别技术人员通过代码审查，提高被审查技术人员的编码能力和业务理解能力。

d.通过Code review，建立代码审查员的权威地位，建立技术为先的工程师文化。

目前技术发展部大多数的code review还是线下进行，code review数量没有覆盖所有的代码提交，同时，线下的code review对代码审查员的时间精力占用较大，效果有限。一些团队也是用线上的工具进行code review，但是工具和流程没有统一的标准，即不能够形成统一的数字化的效果指标，也导致工程师在跨团队共建时需要学习和适应不同的code review标准和流程，增加工作成本。基于此，供应链技术部启动建立严格，标准的code review流程，制定code review的标准和要求，统一code review的线上化工具，并基于工具形成数字化的沉淀作为质量监控的一个重要指标。过程中，供应链技术部也和中台技术部质量与效能团队，在线上code review工具和流程上达成一致，作为统一标准推广到技术发展部，同时，业务工具提供方（coding）建立合作关系，在Coding的基础上共建需要的能力。

## 二、CR不规范引发的反思
线下的方式最大的弊端就是不可控，会因为各种理由被迫跳过或者是降低要求：

a.已经提测了，然后开始找人做CR，发现问题修复后，测试同学前面的工作都白做了。

b.再有马上要上线了，然后急忙招人做CR，这个时候发现的问题就不得不对质量作出一些让步，因为承诺了业务交付时间。

c.逃避CR，直接上线，完全无法得到有效的监控。

诸如此类的案例还有很多，我们因为无法规范化，可视化对于质量做出了太多的让步。

## 三、开启数字化之旅
从20年Q4，供应链技术部就启动了线上化的探索之旅，先后尝试了reviewBoard、gitLib、coding对各个平台也做了一定的分析，对比结果如下：

![](//img1.jcloudcs.com/developer.jdcloud.com/d2e03861-dfaf-4a33-acad-dcf64c8a54f920210521130934.png)

最终选用了coding平台做为我们的线上化的方案，过程中也与coding团队的同学进行了深入沟通，对方也给出了大力的支持，并进行了一定的适配改造。

整体推进线上化的同时，我们也在积极探索自动化分析的方法，通过与coding运维团队沟通进行一些功能优化支撑，同时为了满足个性化的需求，我们也自主开发统计看板工具，目前已经可以自动统计一些指标进行每周复盘分析，团队维度主要观测：CR次数、打回次数、点评次数。个人维度分为评审人维度和发起人维度进行观测。评审人维度观测：总评审次数、打回次数、点评次数；发起人维度观测：发起评审次数、代码变动量、被打回次数；同时，还有评审人单人评审次数最多占比，规避指定人评审带来的思维固化无法发现一些常规的问题。

## 四、线上化CR模式的优势
通过线上化的模式，整体代码质量得到的非常大的提升，同时由于可视化的管控，是CR这项工作可以得到顺利的开展。

a.规避了紧急提交CR，将问题暴露在开发阶段。之前线下模式经常有测试阶段CR导致修改后增加了回归验证的工作量；更有甚者上线前CR，无奈的质量智能像现实低头。

b.降低了单次CR代码量，由原来的CR2小时修改5分钟，变为提交人主动做代码分层，分部提交，代码结构更加清晰易懂。

c.解决了CR过程质量无法监控的问题，通过系统流程卡控，真正做到了100% CR线上化。

## 五、数字化管控，助力质量持续提升
首先，我们进行整体的coding代码库迁移（这里特别感谢coding团队的同学，协助做统一的代码库迁移，节省了大量的人工操作）并统一进行代码评审设置。

然后，通过自建的看板检查所有的项目是否按照规范开启了评审规则：

![](//img1.jcloudcs.com/developer.jdcloud.com/621f9ded-500c-4065-bb4f-f27b8dc1f09c20210521172712.png)

接下来就是CR过程管理及质量提升了，通过建立统一指标进行CR质量监控，使CR质量数字化、可监控化。目前每周的打回率维持在25%左右，如下图：

![](//img1.jcloudcs.com/developer.jdcloud.com/75e27153-77c9-4a87-b239-fe75a423678720210521172732.png)

同时，我们还关注notes的质量，推进CR过程质量的持续提升。过程中，我们也发现越来越多的同学在有意识的做高质量的CR，比如：

![](//img1.jcloudcs.com/developer.jdcloud.com/3fbe7063-4676-45cb-aa9f-da2538ecc0f420210521171743.png)

![](//img1.jcloudcs.com/developer.jdcloud.com/87251bcd-113c-4ea0-8210-c95299c45aed20210521171753.png)

![](//img1.jcloudcs.com/developer.jdcloud.com/a7384e28-d5fd-4378-b771-e7ff8f7ea92620210521171802.png)

他们不单单是再看代码的规范性，更多的在关注性能、代码分层设计等方面的问题，这也是我们所期望看到的。在我们CR质量建设、提升的过程中，也会在每周的质量周会上对优秀的CR案例进行分享。各三级也会进行双周CR分享会。

目前，整个供应链技术部CR线上化已经在稳定运行了，并在4月27日我们组织了「供应链技术部金码奖&伯乐奖的颁奖会」表彰在CR活动中表现突出的同学，同时激励同学们更多的参与到代码质量提升的活动中来。

## 后记：
整个供应链技术部如火如荼的进行CR线上化、数字化推广的过程中，我们也持续不断的完善我们的机制。同时，我们也在整个技术发展部推广CR线上化的方案，现在已经在大件和快递进行推广试运行，过程中我们也在建设统一的过程质量看板，帮助其他团队快速建立代码质量提升体系，让更多的人能够参与到质量建设中来。

------------

###### 自猿其说Tech-JDL京东物流技术发展部
###### 作者：供应链技术部-到仓研发组 刘海鸣  供应链技术部-测试组 张雪

------------