1.代碼評審的好處
- 提高代碼質量。評審別人的代碼,自己的水平也能提高
- 修復bug的代價最小。自己內部人指出問題,代價最小;其次是測試部門測來;最嚴重時來自用戶反饋。
- 促進團隊之間相互備份。評審別人的代碼的同時,可以強制每個人儘可能熟悉別人的代碼,可以形成團隊成員之間的相互備份。
2.代碼評審的關鍵---流程化
如何做代碼評審?具有可操作性的經驗:
- 規定一個固定的週期去做代碼評審,比如兩週一次。團隊成員儘量在代碼評審前完成自己模塊的代碼,能達到一個可被評審的狀態。
- 一定要有個主審人提前去看代碼,否則效果大打折扣。可以嘗試主審人講解爲主,被評審的人答疑爲輔的形式。
- 被審覈的開發者,最好能主動提出哪些代碼可能存在潛在的問題,以節省時間。
- 會議中提出的疑問點,會議中當場討論。會議上的最終決議形成新的任務記錄,來跟蹤相關程序員修改。
- 會議上大家有機會頭腦風暴。
- 每個人主要從以下3個方面去進行代碼評審
- 變量名和函數名是否準確?
- 是否有明顯的bug?
- 整體設計是否有優化的空間?
代碼平生時提高軟件整體質量的重要手段,也是提高程序員編程素養的捷徑。我們每個人都應該從評審自己開始,進而去評審別人,提高項目質量的同時,也促進大家共同進步。