开发者代码审查指南

    Google 通过 Code Review 来维护代码和产品质量。

    此文档是 Google Code Review 流程和政策的规范说明。

    • :针对 CL 开发者的的详细指南。

    代码审查时应该关注以下方面:

    • 设计:代码是否经过精心设计并适合您的系统?
    • 功能:代码的行为是否与作者的意图相同?代码是否可以正常响应用户的行为?
    • 复杂度:代码能更简单吗?将来其他开发人员能轻松理解并使用此代码吗?
    • 测试:代码是否具有正确且设计良好的自动化测试?
    • 注释:评论是否清晰有用?
    • 风格:代码是否遵守了风格指南
    • 文档:开发人员是否同时更新了相关文档?

    参阅 获取更多资料。

    最合适的审查者应该是能彻底了解和审查您代码的人。他们通常是代码的所有者,可能是 OWNERS 文件中的人,也可能不是。有时 CL 的不同部分可能需要不同的人审查。

    如果您找到了理想的审查者但他们又没空,那您也至少要抄送他们。

    面对面审查

    您还可以进行面对面的代码审查,审查者提问,CL 的开发人员作答。