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.