CodeReview的本质分析 - 编程一生
内容提要
代码评审是持续集成的重要环节，需人工介入，以避免设计理解偏差、促进知识传播、检查逻辑正确性。实施时应找了解代码的人评审，使用检查清单，并提前约定提交规范，确保评审高效、代码质量可控。
延伸解读
代码评审在持续集成中的定位
文章将代码评审（CR）明确为持续集成（CI）的一个环节，并指出CI的意义价值（如快速失败、减少风险、自动化重复流程等）自然成为CR的目标。CR通过设置卡点，只有评审通过的代码才能合并到主干或分支，从而在自动化流程中嵌入人工控制，确保代码质量。
人工介入的必要性
尽管CI原则是尽量自动化，但CR强烈需要人工介入。文章列举了三个原因：避免设计理解偏差（即使详细传达，开发实现仍可能不一致）、促进知识传播（评审中的沟通是相互学习的过程）、检查逻辑正确性（线上问题代价高昂）。这些是自动化工具难以完全替代的。
实施代码评审的关键做法
文章从三个方面给出建议：找对人（推荐结对编程，因为双方都了解代码逻辑，能发现更深层问题）、用对法（使用检查清单，涵盖注释、单测覆盖率、模板修改、提交范围等具体项）、讲原则（提前约定提交规范，如一次提交只做一个功能，约定后严格执行，避免情绪冲突）。
Q&A
CodeReview在持续集成中扮演什么角色？
CodeReview是持续集成（CI）的一个环节，需要人工介入，通过设置卡点确保只有评审通过的代码才能合并到主干或分支。
为什么CodeReview需要人工介入而不是完全自动化？
因为人工介入可以避免设计理解偏差、促进知识传播、检查逻辑正确性，这些是自动化难以替代的。
进行CodeReview时应该找什么样的人来评审？
应该找了解代码的人来评审，比如结对编程的伙伴，因为不了解代码的人可能发现不了重要的逻辑问题。
CodeReview的检查清单（checklist）通常包含哪些内容？
检查清单包括：注意文件是否有测试注释、单测覆盖率是否增加或持平、是否缺少必要注释、velocity模板三文件同时修改、一次提交不允许多个功能、修改范围限于本次功能、不允许全局format、所有修改必须有单测、façade的POJO必须序列化等。
如何让CodeReview更高效？
提前约定提交规范（如一次提交只做一个功能），使用检查清单，并找了解代码的人评审，确保评审高效。
CodeReview能带来哪些好处？
可以避免设计理解偏差、促进知识传播、检查逻辑正确性，从而提升代码质量和团队能力。
