上面談了 Code Review 可以帶來幫助,但與此同時,有些事情不該是由 Code Review 來達成,在這邊展開來討論。
如果是能自動化處理的東西,不該由 Code Review 來做。
舉例來說,縮排要多少個空格、要不要加 ; 到每行程式的尾端、要用單駝峰還是雙駝峰格式。這些東西都可以透過 Linting 工具來達成。工程師的時間寶貴,不該花在這些事上面。如果你發現 Code Review 出現這類討論,那推薦可以主動去設置 Linting 來一勞永逸解決。
Code Review 的核心目的不是幫忙看有沒有 bug。
前面有談到,Code Review 作用之一是確保程式碼庫的品質,但這不代表幫忙看有沒有 bug。事實上,先前微軟有發表一篇研究,發現 Code Review 對於找到 bug 的功用來說不大。要確認沒有 bug,還是需要透過測試來做到,包含自動化測試與人工手動測試。