1.首先我覺得我們所有開發人員要弄明白 現在Code Review 的目的 ,凡事不弄明白目的,無法做好完成一件事情,個人覺得有以下一些目的:
a)可以在項目早期就能夠發現代碼中的BUG ,提測後可以盡快的釋放開發資源;
b)同時可以達到知識共享 ,避免我們所有開發人員犯一些很常見,很普通低級的錯誤 ;
c)保證項目組人員的良好溝通 ,項目的代碼更容易維護
大家還有希望補充上
2.Code Review 很容易變得沒有意義或是流于形式,進入 Code Review 個人覺得以下幾點肯定得弄明确:
a) 我們是否了解了 Code Review 的概念和 Code Review 将做什麼,這點都不明白,做法可能就會是應付了事。
b) 我們的代碼是否已經正确的 build , build 的目的使得代碼已經不存在基本文法錯誤 ,我們總不希望review人員浪費在檢查連編譯都通不過的代碼上吧。
c) 我們 Review 人員是否了解了代碼 ,做複查的人員需要對該代碼有一個基本的了解,其本功能是什麼,具體的業務是怎樣的,這樣才能采取針對性的檢查
3 .具體檢查點
1 完整性檢查
代碼是否完全實作了設計文檔中提出的功能需求
代碼中是否存在任何沒有定義或沒有引用到的變量、常數或資料類型
2一緻性檢查
代碼的邏輯是否符合設計文檔
代碼中使用的格式、符号、結構等風格是否保持一緻
3正确性檢查
代碼是否符合制定的标準
所有的變量都被正确定義和使用
所有的注釋都是準确的
4 可修改性檢查
代碼涉及到的常量是否易于修改 ( 如使用配置、定義為類常量、使用專門的常量類等 )