Code Review - 代码审查

概述

系统性地审查代码变更，确保质量、可维护性和正确性。

审查维度

1. 正确性

• 逻辑是否正确？
• 边界情况是否处理？
• 错误处理是否完整？

2. 可读性

• 命名是否清晰？
• 代码结构是否清晰？
• 注释是否有帮助（不是噪音）？

3. 可维护性

• 是否遵循 DRY 原则？
• 是否容易修改和扩展？
• 依赖是否合理？

4. 测试

• 测试覆盖是否充分？
• 测试是否有意义？
• 测试是否可读？

问题分级

| 级别 | 含义 | 处理 |

|------|------|------|

| Critical | 必须修复才能合并 | 阻塞 |

| Important | 应该修复 | 建议修复 |

| Minor | 可以改进 | 可选 |

| Nitpick | 个人偏好 | 仅供参考 |

审查输出格式

## 代码审查报告

### 概览
- 变更: [简述变更内容]
- 审查者: AI
- 日期: YYYY-MM-DD

### 问题

#### Critical
- [ ] **文件:行号** - 问题描述

#### Important  
- [ ] **文件:行号** - 问题描述

#### Minor
- [ ] **文件:行号** - 问题描述

### 优点
- [列出代码的优点]

### 结论
[通过/需要修改后通过/不通过]

嵌入式领域扩展

额外检查项:

• 内存分配是否安全？
• 是否有资源泄漏风险？
• 中断上下文是否正确使用？
• 时序要求是否满足？
• 功耗影响是否考虑？