Code Review - 代码审查
概述
系统性地审查代码变更，确保质量、可维护性和正确性。
审查维度
1. 正确性
- 逻辑是否正确？
- 边界情况是否处理？
- 错误处理是否完整？
2. 可读性
- 命名是否清晰？
- 代码结构是否清晰？
- 注释是否有帮助（不是噪音）？
3. 可维护性
- 是否遵循 DRY 原则？
- 是否容易修改和扩展？
- 依赖是否合理？
4. 测试
- 测试覆盖是否充分？
- 测试是否有意义？
- 测试是否可读？
问题分级
| 级别 | 含义 | 处理 |
|---|---|---|
| Critical | 必须修复才能合并 | 阻塞 |
| Important | 应该修复 | 建议修复 |
| Minor | 可以改进 | 可选 |
| Nitpick | 个人偏好 | 仅供参考 |
审查输出格式
## 代码审查报告
### 概览
- 变更: [简述变更内容]
- 审查者: AI
- 日期: YYYY-MM-DD
### 问题
#### Critical
- [ ] **文件:行号** - 问题描述
#### Important
- [ ] **文件:行号** - 问题描述
#### Minor
- [ ] **文件:行号** - 问题描述
### 优点
- [列出代码的优点]
### 结论
[通过/需要修改后通过/不通过]
嵌入式领域扩展
额外检查项:
- 内存分配是否安全？
- 是否有资源泄漏风险？
- 中断上下文是否正确使用？
- 时序要求是否满足？
- 功耗影响是否考虑？