我怎么做一次代码评审

先看目标，再看代码

代码评审不是在检查风格，而是在确认改动是否真的解决问题。

我最关注的三件事

• 这个改动是否影响原有行为
• 是否有更简单的实现方式
• 失败路径是否足够清楚

写评语的方式

把“这里写错了”换成“这里如果把条件提前返回，会不会更容易维护？”