介绍

代码评审是一些人提交一些代码片段，供另外一些人审阅的流程。

代码评审者需要关注什么

代码评审者需要关注：

  • 设计： 代码是否设计良好并且适合你们的系统？
  • 功能性: 代码功能是否和开发者预期一致？这种方式是否对用户友好？
  • 复杂性: 代码能不能更简单？ 其他开发者能否快速理解并在未来很容易地使用这段代码？
  • 测试: 这段代码是否有正确和设计良好的自动化测试样例？
  • 命名: 开发者有没有正确地对变量、类、方法等命名？
  • 注释: 注释是否清晰有用？
  • 代码风格: 代码风格是否遵循【科大国创统一Java开发规范】
  • 文档: 开发者是否更新了相关文档？

    挑选最适合的代码评审者

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

    亲自评审

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