简介
代码审查是除了代码作者之外,其他人检查代码的过程。
非常多大厂都是通过 Code Review 来维护代码和产品质量。
此文档是整理自 Google Code Review 流程和政策的规范说明。
此页面是我们进行 Code Review 流程的概述。本指南还有另外两套文档:
- 如何进行 Code Review:针对代码审查者的详细指南。
- 代码开发者指南:针对 CL 开发者的的详细指南。
代码审查者应该关注哪些方面?
代码审查时应该关注以下方面:
- 设计:代码是否经过精心设计并适合你的系统?
- 功能:代码的行为是否与作者的意图相同?代码是否可以正常响应用户的行为?
- 复杂度:代码能更简单吗?将来其他开发人员能轻松理解并使用此代码吗?
- 测试:代码是否具有正确且设计良好的自动化测试?
- 命名:开发人员是否为变量、类、方法等选择了明确的名称?
- 注释:注释是否清晰有用?
- 风格:代码是否遵守了风格指南?
- 文档:开发人员是否同时更新了相关文档?
参阅如何进行 Code Review 获取更多资料。
选择最合适审查者
一般而言,你希望找到能在合理的时间内回复你的评论的最合适的审查者。
最合适的审查者应该是能彻底了解和审查你代码的人。他们通常是代码的所有者,可能是 OWNERS 文件中的人,也可能不是。有时 CL 的不同部分可能需要不同的人审查。
如果你找到了理想的审查者但他们又没空,那你也至少要抄送他们。
面对面审查
如果你与有资格做代码审查的人一起结对编程了一段代码,那么该代码将被视为已审查。
你还可以进行面对面的代码审查,审查者提问,CL 的开发人员作答。