「这里应该用 XXX」这种评论,容易变成风格之争 —— 争的是偏好,不是对错。

「这里如果传进来 null 会怎么样?」「这个分支什么时候会走到?」这种提问不一样,它逼着作者把脑子里的假设说出来,而大部分 bug 就藏在这些没说出来的假设里。

所以我自己 review 的时候尽量把结论改成问句。不光是为了不冒犯人,是因为问句真的更容易挖出东西。

#代码评审# #团队协作#
展开
评论