您好!
欢迎来到京东云开发者社区
登录
首页
博文
课程
大赛
工具
用户中心
开源
首页
博文
课程
大赛
工具
开源
更多
用户中心
开发者社区
>
博文
>
CodeReview数字化管理工程实践
分享
打开微信扫码分享
点击前往QQ分享
点击前往微博分享
点击复制链接
CodeReview数字化管理工程实践
自猿其说Tech
2021-05-21
IP归属:未知
187640浏览
敏捷
计算机编程
## 一、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京东物流技术发展部 ###### 作者:供应链技术部-到仓研发组 刘海鸣 供应链技术部-测试组 张雪 ------------
原创文章,需联系作者,授权转载
上一篇:产品经理的禅-产品经理的定位、日常工作和修炼
下一篇:配运基础数据缓存瘦身实践
相关文章
浅谈对敏捷的认识
架构研究:研发敏捷与中台架构(论前台bp研发敏捷)
敏捷实践 — 估算
自猿其说Tech
文章数
426
阅读量
2149957
作者其他文章
01
深入JDK中的Optional
本文将从Optional所解决的问题开始,逐层解剖,由浅入深,文中会出现Optioanl方法之间的对比,实践,误用情况分析,优缺点等。与大家一起,对这项Java8中的新特性,进行理解和深入。
01
Taro小程序跨端开发入门实战
为了让小程序开发更简单,更高效,我们采用 Taro 作为首选框架,我们将使用 Taro 的实践经验整理了出来,主要内容围绕着什么是 Taro,为什么用 Taro,以及 Taro 如何使用(正确使用的姿势),还有 Taro 背后的一些设计思想来进行展开,让大家能够对 Taro 有个完整的认识。
01
Flutter For Web实践
Flutter For Web 已经发布一年多时间,它的发布意味着我们可以真正地使用一套代码、一套资源部署整个大前端系统(包括:iOS、Android、Web)。渠道研发组经过一段时间的探索,使用Flutter For Web技术开发了移动端可视化编程平台—Flutter乐高,在这里希望和大家分享下使用Flutter For Web实践过程和踩坑实践
01
配运基础数据缓存瘦身实践
在基础数据的常规能力当中,数据的存取是最基础也是最重要的能力,为了整体提高数据的读取能力,缓存技术在基础数据的场景中得到了广泛的使用,下面会重点展示一下配运组近期针对数据缓存做的瘦身实践。
自猿其说Tech
文章数
426
阅读量
2149957
作者其他文章
01
深入JDK中的Optional
01
Taro小程序跨端开发入门实战
01
Flutter For Web实践
01
配运基础数据缓存瘦身实践
添加企业微信
获取1V1专业服务
扫码关注
京东云开发者公众号