程序员如何做「代码审查」：不是挑刺，是学习
代码审查是程序员之间最重要的协作方式。
做得好能提升代码质量、传播知识；做不好就是走过场、浪费时间。
一、代码审查的目标
1. 发现问题
- bug
- 逻辑错误
- 安全漏洞
- 性能问题
2. 传播知识
- 代码逻辑共享
- 团队水平提升
- 减少单点依赖
3. 保证质量
- 代码风格一致
- 设计合理
- 有测试覆盖
二、审查者应该做什么
1. 理解代码在做什么
先看需求，了解这段代码的目的是什么。
不是上来就挑刺。
2. 检查关键点
- 逻辑是否正确
- 边界是否处理
- 错误是否处理
- 性能是否合理
3. 提建设性意见
- 指出问题
- 给出建议
- 说明原因
4. 区分优先级
- blocker：必须改
- suggestion：建议改
- nit：可选改
三、被审查者应该做什么
1. 控制 PR 大小
一次 review 超过 400 行 = 没人会认真看。
拆成小 PR，每次只改一件事。
2. 写清楚 PR 描述
- 这个 PR 改了什么
- 为什么改
- 怎么测试的
3. 响应 feedback
- 及时回复
- 认真考虑
- 有问题就讨论
四、常见错误
❌ 审查太苛刻
代码风格都能挑半天，忽略了核心问题。
❌ 审查太随意
"看起来没问题"——没有真正审查。
❌ PR 太大
一次性 review 几千行代码。
❌ 不响应
提了 feedback，开发者不回也不改。
五、一句话总结
代码审查 = 理解目的 + 检查关键 + 建设性意见 + 区分优先级，核心是提升代码质量和传播知识。