AntiPattern: Batch Code Review

所谓批量code review是指一段时间, 通常是一个迭代或几周之后, 团队坐在一起集中code review. 这种方式有几个问题:有一些通过code review能发现的bug因此而潜伏下来, 有可能被发布, 就算运气好发布前通过各种测试被发现, 依然带来一些overhead像发bug, 定位问题, 修bug等额外的工作.
有一些通过code review能共享出来的知识因此而很长时间之后才被共享, 而在此期间, 团队其他人由于不了解这些知识, 有可能犯已经犯过的错误, 或走一些弯路, 造成效率的降低
有一些通过code review能得到改善的设计因此而被推迟, 造成技术债的累计, 提高维护成本
有一些故事已淡忘…
如果是通过版本控制系统的diff来做code review, 则按顺序review时很多中间版本都已经过时, 不得不费力解释最新版本是什么样子, 给团队带来理解上的困难, 甚至造成错误的印象
code review meeting将持续很长时间, 每个人都很疲惫
然而最关键的问题是: 批量code review得出来的Action, 很难得到执行, 因为已经时过境迁, 团队手头有新的高优先级的任务, 很难回过头去改已经标记为"Done"的任务. 而Action得不到执行, code review的意义已经消失了一半.Daily code review is a better solution.