代码评审是为了什么

早期我们评代码，习惯挑毛病：这里命名不好，那里可以更简洁。结果大家提交前很紧张，评审变成了一种对抗。后来我们调整了关注点，评审才真正开始起作用。

先问对的问题

这段代码解决的是问题本身吗？有没有可能理解错了需求。这比风格问题重要得多。

边界情况想全了吗？空值、并发、权限、失败重试，这些是线上事故的主要来源。

下一个人接手能看懂吗？半年后的自己也算下一个人。

不该在评审里做的事

争论缩进与括号位置。格式交给工具，不占用人的时间。

要求作者按评审者的写法重写。只要实现正确、可读，风格差异可以放过。

在公开评论里否定人的能力。对代码严格，对人保持尊重。

评审的目标不是让代码变成评审者喜欢的样子，而是让它在别人眼里也能站得住。