代碼review是質量保證(QA)的手段之一,但不是用來替代測試的,特別是自測。
一個軟件項目的質量定義並不是代碼review的職責,換句話說,良好的質量定義是代碼review發揮效果的必要前提。
代碼review到底要review哪些東西?
- 代碼風格
- 代碼結構(架構與設計)
- 核心邏輯
想要通過代碼review來檢測每一行代碼,並確保檢查出所有問題是不可能的,它更側重於處理核心且明顯的問題。
誰來review?
這個要看開發組採取的review形式,一般分爲獨立review和集中review,前者指定要review的人,後者開發組都在場集中review。
獨立review適合項目進度沒那麼緊張,且人員配置充足的情況,可以對每一筆提交進行review。
集中review則相反,比較適合人員和時間有限的情況,一般每週一次,需要投屏。
如何review?
明確需求(case) -> 代碼結構 -> 核心邏輯 -> 代碼風格。
review過程產生的問題,要記錄下來,一般review工具(比如gerrit)提供的有記錄方式,review結束後要推動問題的修改落實。