编者按：本文源自阿里云云效团队出品的《阿里巴巴 DevOps 实际指南》，扫描上方二维码或返回：https://developer.aliyun.com/…，下载完整版电子书，理解阿里十年 DevOps 实践经验。
代码评审，英文名是 Code Review，简称 CR，它是结对编程互相切磋互相学习的形式。庄重地讲，CR 可能晋升代码品质、促成人才成长、造就技术情怀。
首先，代码也是一种资产且具备“流通性”，通常会须要 3 到 5 年的保护。过程中将面临保护人员更替、其他人参考援用的状况，谁都不心愿本人接手或参考的是一份“天书”一样的代码。所以通过 CodeReview 可能发现除程序逻辑以外的设计、性能、平安、标准等多方面的问题。
其次，CodeReview 是一个相互查看谬误，互相学习代码用法的机会。如果团队的外围骨干人员，能参加到团队日常的架构评审、设计评审以及代码评审中，肯定能够切切实实地帮忙到其余研发人员的成长，不论是新人的融入，还是处于瓶颈期的其余研发人员。咱们冀望看到 CodeReview 能够促使团队 ” 战斗力 ” 的晋升。同时 CodeReview 因为波及到研发之间就某一个具体的问题或场景交互式的探讨，所以也成为了工程师们晋升编程能力的最佳路径。
最初，CodeReview 是一种文化，通过一个个小团队外部的评审实际逐步造成为 IT 企业的一种开发者文化。当一个企业领有了特定的技术文化，这对人才的吸引和成长是至关重要的。
传统评审流程的问题
如上图所示，代码评审次要分为三个阶段：评审开始、评审中、评审完结。
- 评审开始阶段，发起人须要从大量库成员中抉择最相熟改变代码的若干人作为评审人，抉择开发分支和指标分支，填写评审形容信息，建成评审，评审告诉会发送给受邀的评审人。一个评审人通常会收到多个评审邀请，他须要安顿工作的间隙抉择适宜的评审各个击破或者用固定的时段集中评审。评审人点开评审，顺次浏览代码逻辑，对于比较复杂的嵌套或调用依赖，须要去本地 IDE 查看外部逻辑和影响范畴。
- 评审过程中，负责的评审人会基于破绽异样，代码格调，性能缺点，反复代码等维度提出评论。评审发起人收到评论告诉后，会从新扫视问题代码，批改或重构部分问题行，再次提交并更新评审。如此迭代，直到问题被解决。
- 评审完结后，评审人点击通过评审，如果源分支和指标分支有代码抵触的话，须要评审发起人去解决抵触，最初合并代码。
综上所述，传统的代码评审流程次要有以下几个问题：
- 创立速度慢，评审的创立须要手动填写大量信息：抉择分支，填写形容，抉择评审人，关联工作项等。
- 人力投入大，最传统的代码评审是结对编程，以及团队圆桌评审，人力的投入不言而喻。
- 评审内容杂，不标准的评审会存在变更文件过多，变更版本错乱，多个需要混合等景象，导致评审的代码难以了解，评审过程无从下手，文件间逻辑难以关联，文件跨度过长遗记前文逻辑等问题。
- 成果评估难，作为管理者经常在评审问题上犯难，他们认为评审是一个重要且正向的流程，促成代码品质和团队合作。然而团队的成员是否能真正践行评审保障评审品质，而不是随便通过走个流程，是管理者困惑且急需晓得的。同时管理者也想晓得评审流程的投入是否能带来足够的回报。
智能代码评审
针对传统评审流程面临的问题，阿里巴巴的代码平台团队借助智能化和流程管控的伎俩，晋升评审效率，如下图所示：
具体从以下几个方面进行优化：
- 通过对代码库维度的评审数据进行智能剖析，主动实现创立评审过程中的信息填充从而晋升评审创立的效率，同时还提供以客户端命令行的形式疾速创立评审。
- 通过丰盛的代码自动化检测工具链，尽可能减少人工 review 的内容。例如通过阿里巴巴 Java 规约检测工具主动发现不合乎代码标准的代码并给出修复倡议。
- 通过大评审拆分，针对蕴含文件数过多的评审按文件相关度进行智能拆分，将一个超大的评审拆分成若干个小评审，并依据评审的内容和以后评审人的工作饱和度智能抉择适合的评审人进行评审。这样通过精简评审内容来进步评审的有效性，防止蜻蜓点水的模式评审。
- 评审尽管依附人工，但整个流程是数字化的，提供了丰盛的代码报表数据，帮忙管理者更好的治理和优化评审流程，领导管理者对评审带来的成果进行无效的评估。
评审人举荐
通过智能算法，主动举荐最合适的评审人。对于大型利用，和跨团队共建的中台利用，主动举荐能够节约大量的沟通老本，疾速推动评审流程。
评审耗时预估
通过提交数据分析以后一次评审预计须要多长时间能够实现，帮忙评审人依据以后评审所需的工夫来合理安排本人的工作。此外，对于一个评审人同时领有多个代码评审的场景，还能够依据评审耗时来安顿评审工作的优先级。
git-repo 评审工具
git-repo 是由阿里巴巴开发的一款客户端工具，能够从客户端间接发动代码评审，实现在客户端疾速创立并在页面端疾速关上浏览。git-repo 并不会扭转 git 用户的应用习惯，而是提供了对 git 命令的扩大。
传统的代码评审工作模式，代码贡献者要将代码推送到集体 / 个性分支，再通过在浏览器页面上发动创立合并申请，整个过程要经验多个步骤，开发者要切换到不同的工具能力实现。而应用 git-repo，一个用户只有领有仓库的读取权限，就能够在本地工作区中执行上面的一条命令，将代码以合并申请的形式奉献到服务端。
总结
在阿里巴巴外部，从上自下都在推广代码评审文化，要求每个开发者不仅要对本人的代码负责，也要作为代码评审人帮忙其余共事一起把控代码的公布品质。想要做好一个评审，除了须要好用的工具辅助之外，还须要开发者留神以下几点：
- 做好提交，行将提交做小做好。写小提交就是将问题解耦：“Do one thing and do it well”。每个提交管制在只批改一个到几个文件，每次只批改几行到几十行。每个提交应该是一个残缺的性能，可能是批改某个 bug 或实现某个小需要的开发。
- 充沛形容，对于代码评审形容应介绍本次改变的需要背景，改变范畴及影响点。一段清晰的评审形容能让 reviewer 充沛了解需要背景，疾速开始评审，升高沟通老本。
- 数字化度量，通过数据洞察取得团队品质状况，有策略的晋升团队技术能力。
【对于云效】
云原生时代一站式 DevOps 平台，数十万企业都在用。反对公共云、专有云和混合云多种部署状态，通过云原生新技术和研发新模式，助力翻新守业和数字化转型企业疾速实现研发麻利和组织麻利，打造“双敏”组织，实现多倍效力晋升。
立刻体验