简介： 代码评审带来的益处不言自明, 但企业业务疾速倒退的诉求与代码评审推动落地两者之间, 往往存在矛盾。在现在疾速倒退的互联网时代，数字化、智能化曾经是根底能力，单纯只靠人肉审查的时代曾经过来了，基于各种自动化查看能力的加持，其实代码评审并没有设想中那么费时费力。明天和大家聊一聊在快节奏的业务现状下基于云效代码治理产品 Codeup 如何更低成本的发展代码评审。
难得请了年假，躺在阳光海浪仙人掌的沙滩上喝着椰汁，忽然接到零碎报警电话，立即跳起来抱着电脑到处找 WIFI 的场景是否似曾相识。
身为技术开发，每逢放假巴不得烧香祈愿线上稳固，如果可能在问题引入前提前扼制危险，就能够释怀享受悠闲的假期了。
而代码评审，就是扼制危险的无效伎俩之一。
代码评审带来的益处不言自明, 但企业业务疾速倒退的诉求与代码评审推动落地两者之间, 往往存在矛盾。在现在疾速倒退的互联网时代，数字化、智能化曾经是根底能力，单纯只靠人肉审查的时代曾经过来了，基于各种自动化查看能力的加持，其实代码评审并没有设想中那么费时费力。明天和大家聊一聊在快节奏的业务现状下基于云效代码治理产品云效 Codeup 如何更低成本的发展代码评审。
话题开始之前，先简略介绍下代码评审的概念。
代码评审，英文名是 Code Review，简称 CR，它是结对编程互相切磋互相学习的形式。肯定频次的 CR 可能晋升咱们的代码品质、促成人才成长。
一、晋升代码品质
所谓代码品质，能够从两个维度来了解，一是可读性，二是缩小缺点。
- 可读性：Code Review 机制的诞生最早是为了保障代码具备良好的可读性。代码是一种资产，并且具备“流通性”，通常会须要多年的保护，并且面临保护人员更替的状况，谁都不心愿本人接手的是一份“天书”一样的代码。应用 CR 的麻利团队更是能取得微小的益处，因为团队的工作是扩散的，通过代码评审能够让团队所有人都可能有机会理解对方的代码内容，有助于促成跨库和整个团队的常识共享，让任何团队成员都能够接管并持续推动整个工程的演进。
- 缩小缺点：Code Review 可能发现除程序逻辑以外的设计、性能、平安、标准等多方面的问题。在这个过程中，除了依赖评审人的业余能力外，也给了咱们更多让自动化和智能化检测伎俩染指的机会，包含对代码标准和平安的自动化查看、基于 AI 的缺点剖析和补丁举荐、合并的危险评估等。
二、促成人才成长
很多团队都由领有不同教训的成员组成，代码评审是一个相互查看谬误，互相学习代码的机会。如果团队的技术骨干人员，能参加到团队日常的架构评审、设计评审以及代码评审中，除了可能升高出错率，缩小设计初期的危险故障外，还能够切切实实的帮忙到其余研发人员的成长。体感更显著的团队会发现团队的开发品质在逐步进步，并且一直在向团队最高程度成员聚拢。
三、两种评审场景
咱们能够基于评审过程的严格水平，把评审分为轻 / 重两类，能够依据本身业务状况抉择最合适的评审形式。
1. 轻评审很多企业管理者会感觉评审会耽搁业务的迭代速度，评审和速度是鱼与熊掌不可兼得的事，当然业务最重要，评审就被天然舍弃了。
对于看重迭代速度的企业，轻评审是一个不错的抉择，它没有强制的规定卡点，不要求评审人必须严格的浏览每一行代码给出评审意见，联合自动化检测的能力，在代码合并入重要分支的时候做一次平安和品质扫描，人力投入可控，能够更加灵便的依据以后业务情况决定是否立刻合并代码，能够小老本的实现评审。
2. 重评审
而对于一些流程严格，或上线代码平安品质要求高的公司，从管理层就要求一系列评审的硬性卡点，包含自动化查看、必须通过的评审人数、评论解决状态等等，其中任何一条不满足都不容许合并，这种状况就须要应用重评审个性的一系列管控能力了。
还有一些企业，不仅对爱护分支的合入强制管控，甚至对每一次提交都有要求，心愿任何推送到服务端的代码都要通过评审，这种场景对评审的要求十分高，而 Codeup 翻新的集中式工作流 Agit-Flow 能够很好的解决这个场景。
接下来咱们先看下惯例的评审流程是什么样子的：
四、惯例评审流程
代码评审次要分为三个阶段：评审开始、评审中和评审完结。
惯例流程中各个阶段存在的次要艰难有：
- 评审开始阶段，对于发起人，须要从大量库成员中抉择适合的评审人，填写必要信息实现评审创立，并告诉评审人及时前来评审。而对于评审人，通常会存在多个评审邀请，须要安顿工作的间隙抉择适宜的评审各个击破或者用固定的时段集中评审。评审人点开评审，顺次浏览代码逻辑，对于比较复杂的嵌套或调用依赖，还须要去本地 IDE 查看外部逻辑；
- 评审过程中，负责的评审人会基于破绽，格调，缺点等维度提出评论。要期待评审发起人收到告诉后修复代码，而后提交再次评审。如此迭代，直到问题被解决，评审人点击通过评审，如果源分支和指标分支有代码抵触的话，须要评审发起人去解决抵触，最初合并代码。
惯例的代码评审流程次要有以下问题：
1. 创立评审麻烦：评审的创立须要手动填写大量信息，很多操作是重复劳动或是无从下手的；
2. 人力投入老本高：最传统的代码评审是结对编程，以及团队圆桌评审，人力的投入不言而喻。代码评审转移到线上后，依然须要多人认真校对、剖析和线下探讨。短少自动化评审伎俩是要害。
3. 评审流程体验差：评审过程中纯文本的代码难以展示代码的粗浅逻辑，代码是平面的，局部改变的办法须要去查看定义和援用能力看出问题，否则只能是走马观花，成果无限，负责的评审人往往须要联合本地 IDE 来配合应用。评审发起人收到评论后，须要去本地批改提交后，再回复评论，门路很长。评审的通过、合并也没有卡点规定，任何有权限的人都能做这些操作，却可能会疏忽评审的问题没有解决，导致本能够提前解决的问题带入线上生产环境。
4. 评审流动状况难评估：管理者经常心愿可能掂量，团队的成员是否真正践行评审，保障评审品质，而不是随便通过评审，只是走个流程。
五、云效智能代码评审
针对惯例代码评审存在的问题，云效 Codeup 通过智能算法和流程管控能力，让评审更加高效。
1. 晋升创立速度创立评审须要填写一堆根底信息，云效 Codeup 致力将用户须要输出的内容压缩到起码：
- 减少了主动填充源、指标分支的性能；
- 反对评审人举荐，基于代码库的历史操作，将最相熟你改变代码的库成员举荐为评审人，让你不便的抉择最适宜的评审者；
- 针对总是须要反复抉择评审人的问题，爱护分支反对设置默认评审人，缩小冗余手工操作。如果你有按目录设置评审人的弱小志愿，还能够应用 CodeOwner 模式（比方 A 目录让甲同学负责，B 目录下的文件由乙同学负责），设置后会将对应目录的代码负责人主动加为评审人。
2. 升高人力投入评审的人力投入是最大的老本，随着自动化扫描能力的减少，人工评审前的机器预评审成为了支流。
云效 Codeup 提供了代码扫描能力，守护代码平安和品质。内置的代码扫描包含【代码规约扫描】、【依赖包破绽扫描】、【敏感信息扫描】、【补丁举荐扫描】，也能够基于云效的 Flow（流水线）配置单元测试和自定义扫描节点，例如【源码破绽检测】、PHP/Python/Go 等常见语言的代码扫描，再将后果关联到评审上。
对于比较简单的评审，自动化测试的保障曾经足够，大大减少了人力和工夫投入老本，同时也防控了缺点的引入。
3. 优化评审流程体验网页端对于浏览简略逻辑的代码十分不便，然而如果存在较多相互援用的函数调用，浏览起来就比拟费劲了。云效 Codeup 针对评审简单状况，反对了网页端的函数援用疾速跳转（咱们称为智能语法服务），防止在网页上艰巨的切换文件视图；对于习惯本地 IDE 看代码的同学，云效 Codeup 也反对了 IDEA 的代码评审插件，不必来回切换编辑器和网页，间接在 IDEA 外面就能够进行代码评审，甚至一键合并代码，十分不便。
另外，通常一个个性可能须要多人合作开发，为了缩小合并代码时的抵触，云效 Codeup 提供了抵触预检测的能力，反对通过网页端的 WebIDE 主动解决抵触并疾速提交。
在评审合作告诉方面，评审的要害动静反对通过站内信、邮件、钉钉的形式及时告诉到评审参与方，防止你等我我等你的难堪，可能更高效的推动评审的进度，更快一步实现迭代。
4. 反对查看评审流动状况 Codeup 针对评审流动，提供了独自的度量报表，能够看到仓库内每次提交是否通过了评审，查看提交和代码行的评审率，每个仓库成员的评审流动参加次数，收到评审邀约的响应速度，如果有同学评审总是迁延，可能他就是迭代的一个阻塞点，兴许应该为他加重工作或者抉择其余评审人帮忙补位；
在评审流动中，咱们也是激励评论的，有问题说进去，无论是疑难还是夸赞，也不便后续的回顾追溯。此外，千行代码评论数也能够作为管理者对评审有效性评估的可视化度量参考。
说了这么多，你当初还感觉代码评审会是业务飞驰的绊脚石吗？好的理念要付出实际，快去试试吧！
更多小提示
理解 Agit-Flow 阿里巴巴集中式工作流，让创立代码评审像提交一样简略：http://t.tb.cn/36VCKdTVaMGVfKFMLFcyuo
- 云效 Codeup 反对本地代码评审插件 Cloud Toolkit，编码间隙高效评审：关上「IntelliJ IDEA」->「Preference」->「Plugins」，搜寻“Alibaba Cloud Toolkit”，点击装置即可；
- 云效 Codeup 是凋谢给所有开发者的收费企业级代码托管产品，收费、收费、收费，重要的事件说三遍；
版权申明： 本文内容由阿里云实名注册用户自发奉献，版权归原作者所有，阿里云开发者社区不领有其著作权，亦不承当相应法律责任。具体规定请查看《阿里云开发者社区用户服务协定》和《阿里云开发者社区知识产权爱护指引》。如果您发现本社区中有涉嫌剽窃的内容，填写侵权投诉表单进行举报，一经查实，本社区将立即删除涉嫌侵权内容。