代码审查清单：提升团队代码质量

代码审查是保证质量的关键环节。这份清单帮你发现常见问题。

功能正确性

• 是否满足需求？
• 边界条件是否处理？
• 错误处理是否完善？
• 是否有单元测试？

代码可读性

• 命名是否清晰？
• 函数是否过长？（建议<50 行）
• 是否有必要注释？
• 代码格式是否统一？

性能考虑

• 有无 N+1 查询？
• 循环内是否有耗时操作？
• 是否合理使用缓存？
• 有无内存泄漏风险？

安全性

• 输入是否验证？
• 有无 SQL 注入风险？
• 敏感信息是否加密？
• 权限检查是否到位？

可维护性

• 是否遵循 DRY 原则？
• 依赖是否合理？
• 配置是否外置？
• 日志是否充分？

常见反模式

// ❌ 魔法数字
if (status === 3) { ... }

// ✅ 使用常量
const STATUS_APPROVED = 3;
if (status === STATUS_APPROVED) { ... }

// ❌ 嵌套过深
if (a) {
  if (b) {
    if (c) { ... }
  }
}

// ✅ 提前返回
if (!a) return;
if (!b) return;
if (!c) return;

审查心态

• 对事不对人
• 提出建议而非命令
• 肯定好的代码
• 解释为什么而不仅是什么

好的代码审查是团队学习的机会，不是找茬大会。