代码审查 Checklist：提升代码质量的 15 个要点

代码规范

• 命名清晰有意义（变量、函数、类）
• 代码格式统一（缩进、空格、换行）
• 注释恰当（解释为什么，不是做什么）
• 函数长度合理（< 50 行为佳）

功能正确性

• 满足需求规格
• 边界条件处理（空值、极限值）
• 错误处理完善
• 无安全漏洞（SQL 注入、XSS 等）

性能考虑

• 无明显的性能问题
• 数据库查询优化（避免 N+1）
• 缓存使用合理
• 无内存泄漏风险

可维护性

• 代码可复用（DRY 原则）
• 职责单一（SRP 原则）
• 依赖关系清晰
• 有单元测试覆盖

审查建议写法

## 🐛 Bug
- 第 23 行：空指针风险，建议添加 null 检查

## 💡 建议
- 函数 extractData 太长，可以拆分为 validate 和 parse

## ✅ 亮点
- 错误处理很完善，值得学习！

审查心态

• 对事不对人
• 多问"为什么"，少说"应该"
• 先肯定优点，再提改进建议
• 一次审查聚焦重点问题

好的代码审查是团队成长的最佳途径！