代码审查清单：提升代码质量的 20 个检查点

功能正确性

• 代码是否实现需求
• 边界条件是否处理
• 错误处理是否完善
• 是否有单元测试覆盖

代码设计

• 函数是否单一职责
• 类/模块职责是否清晰
• 是否有重复代码（DRY）
• 依赖注入是否合理

可读性

• 命名是否有意义
• 代码是否有注释（解释为什么而非是什么）
• 函数长度是否合理（< 50 行）
• 代码格式是否统一

性能

• 是否有 N+1 查询
• 循环内是否有耗时操作
• 是否有内存泄漏风险
• 数据库查询是否有索引

安全性

• 输入是否验证
• SQL 是否参数化（防注入）
• 敏感信息是否加密
• 权限校验是否完善

可维护性

• 配置是否外置
• 日志是否充分
• 是否有文档
• 是否向后兼容

审查技巧

给审查者

• 关注逻辑而非风格（风格交给 linter）
• 提出问题而非命令
• 肯定好的代码
• 一次审查不要太多代码（< 400 行）

给作者

• 小批量提交
• 自审后再提交审查
• 及时响应反馈
• 不要个人情绪化

自动化辅助

# 代码风格检查
npm run lint

# 类型检查
npm run type-check

# 测试
npm test

# 安全扫描
npm audit

代码审查是团队学习和质量保障的最佳实践。