📈
eng-practices-cn
  • CONTRIBUTEGUIDE
  • README
  • review
    • 紧急情况
    • 代码评审开发者指南
    • developer
      • 写一个好的CL描述
      • 如何处理评审者的评论
      • CL开发者指南 —— 更好的通过代码评审
      • 简短化的CL
    • reviewer
      • 怎样写代码评论
      • 如何做代码评审?
      • 代码评审应该关注什么?
      • 代码评审步骤
      • 处理代码评审中的反驳
      • 代码评审的速度
      • 代码评审标准
Powered by GitBook
On this page
  • 介绍
  • 代码评审者需要关注什么?
  • 挑选最适合的代码评审者
  • 亲自评审
  • 参见

Was this helpful?

  1. review

代码评审开发者指南

Previous紧急情况Nextdeveloper

Last updated 3 years ago

Was this helpful?

介绍

代码评审是一些人提交一些代码片段,供另外一些人审阅的流程。 在谷歌,我们都是通过代码评审来保证代码和产品的质量。 这篇文档是对谷歌代码评审流程和策略的权威描述。 这一页是我们代码评审过程的概述。这篇指南其他有两个大的部分,分别是:

  • : 代码评审者的详细指南。

  • : 提交代码评审的开发者的详细指南。

代码评审者需要关注什么?

代码评审者需要关注:

  • 设计: 代码是否设计良好并且适合你们的系统?

  • 功能性: 代码功能是否和开发者预期一致?这种方式是否对用户友好?

  • 复杂性: 代码能不能更简单? 其他开发者能否快速理解并在未来很容易地使用这段代码?

  • 测试: 这段代码是否有正确和设计良好的自动化测试样例?

  • 命名: 开发者有没有正确地对变量、类、方法等命名?

  • 注释: 注释是否清晰有用?

  • 代码风格: 代码风格是否遵循?

  • 文档: 开发者是否更新了相关文档?

参考 获取更多信息。

挑选最适合的代码评审者

通常而言,你都希望找个一个最合适的评审者在合理的时间里对你的变更作出评审。 最合适的评审者是那些能对你代码做出最全面最准确评价的人,一般情况下都是代码的维护者(他可能在所有者列表里也有可能不在)。 有时这意味着要不同的人阅读不同的部分。 如果你找到理想的评审者但他没有时间,你也至少应该抄送他。

亲自评审

如果你是和某个合格的代码评审者结对写的代码,那么这段代码可以认为已经通过评审了。 你也可以以问答的方式亲自参与代码评审。

参见

: 代码评审者的详细指南。

: 提交变更评审的开发者的详细指南。

评审者指南
开发者指南
谷歌代码风格指南
代码评审者指南
评审者指南
开发者指南