namelosw
2020-05-11 08:58:57 +08:00
Code review 最大的目的其实不是找问题。
首先是让分歧能早统一，如果不 review 可能一个月之后才发现，那时候就晚了，想改只能重写了，如果第二天就发现所有事情都来得及。
其次团队互相分享代码，能知道其他人在做什么，接手不至于从头猜。提升巴士系数。
所以最好每天都 Code review，不要攒着。
有新想法，或者发现对方代码做得和自己的代码放一起会有问题这样可以提早发现，提早讨论解决。
不懂对方在做什么也可以让对方解释下，自己觉得别人可能不知道也可以快速解释下。
Code review 的时候可以讨论小问题，风格之类的，但是观点僵持不下，重要的话开会讨论，不重要的话就投票或者找个人让他选就行了。或者干脆再不重要就不用统一。
当然还有写安全问题，设计的问题也要捕捉提出来。
而且不一定提建设性意见，有可能是你发现一个问题，别人没注意，但是你也不知道怎么解决，说出来大家聊聊，不然可能后面就是坑。
最后，当然，政治因素各家不同，自行拿捏。但是不要牺牲太多 Code review 本身的目的，不然还不如不搞。