高效代码审查（Code Review）实践指南

Code Review 是提升代码质量的最佳实践：

审查什么？

必须检查：

• 功能正确性
• 安全问题
• 性能影响
• 测试覆盖

重点关注：

• 代码可读性
• 架构一致性
• 错误处理
• 边界条件

不必纠结：

• 个人风格偏好（用 Linter 自动化）
• 变量命名（除非严重误导）

审查流程

1. 提交前：作者自检，确保 CI 通过
2. 小批量：每次 PR < 400 行
3. 及时响应：24 小时内给出反馈
4. 明确结论：LGTM / 需要修改 / 拒绝

给出好建议

❌ "这代码太乱了"
✅ "建议将这段逻辑提取成独立函数，提高可读性"

❌ "为什么要这样写？"
✅ "我注意到这里用了 X 方法，考虑过 Y 方案吗？"

接受反馈

• 不要 defensive
• 理解建议背后的原因
• 不同意见友好讨论

文化：Code Review 是学习机会，不是批评大会。对事不对人。