高质量的代码评审（Code Review）应该关注什么？

评审关注的维度

• 正确性：逻辑是否实现需求，异常、并发、边界与空值是否处理到位。
• 可读性：命名是否表意、结构是否清晰、是否易于他人理解与修改。
• 设计：抽象与职责划分是否合理，是否引入不必要的复杂度或耦合。
• 测试：是否有覆盖关键路径与边界的测试，测试本身是否可靠。
• 安全与性能：是否存在注入、越权、敏感信息泄露，或明显的性能瓶颈。

高效评审的实践

• 小批量：每次评审控制在数百行内，过大的 PR 会让评审质量骤降。
• 对事不对人：评论针对代码而非作者，多用提问与建议的语气，区分「必须改」与「可选建议」。
• 自动化先行：把格式、lint、静态检查、测试交给 CI，人只评审机器评不了的设计与逻辑。
• 及时反馈：尽快响应，避免 PR 长期阻塞影响协作节奏。

目标是既保证质量，又传递知识、保持团队协作的健康氛围。