Code Review（代碼審查）是保證項(xiàng)目中代碼質(zhì)量非常重要的一個(gè)環(huán)節(jié)，其主要工作是：
1、發(fā)現(xiàn)代碼中的bug；
2、從代碼的易維護(hù)性、可擴(kuò)展性角度考察代碼的質(zhì)量，提出修改建議。 1、代碼中的bug主要會(huì)出現(xiàn)在下列兩個(gè)地方：
(1) 與商業(yè)邏輯無關(guān)的bug。
比如，系統(tǒng)中打開的流／文件／連接等沒有及時(shí)關(guān)閉；或是存在thread safe問題，或是存在性能低下問題等，這類問題對(duì)有經(jīng)驗(yàn)的開發(fā)人員是比較容易發(fā)現(xiàn)的。
2、與商業(yè)邏輯相關(guān)的bug。
這類bug是非常隱蔽的，如果有對(duì)產(chǎn)品不熟悉的人參與該產(chǎn)品的項(xiàng)目開發(fā)，容易出現(xiàn)這類的bug。為了避免這類bug的出現(xiàn)，我們除了在Use Case和Test Case中詳細(xì)描述以正確指導(dǎo)開發(fā)人員并在測(cè)試時(shí)能及時(shí)發(fā)現(xiàn)它之外，Code Review也是不可缺少的保證環(huán)節(jié)。
我們希望代碼的審核者對(duì)產(chǎn)品非常熟悉。
3、什么樣的人承擔(dān)代碼審核者Code Reviewer？
(1)、比較熟悉相關(guān)商業(yè)邏輯。
(2)、有豐富的編程經(jīng)驗(yàn)。
兩者缺一不可。
4、代碼Code Review的步驟，這些是我在平時(shí)工作中的經(jīng)驗(yàn)總結(jié)，目前也是按照這個(gè)步驟在做。
(1)、代碼編寫者和代碼審核者坐在一起，由代碼編寫者按照UC依次講解自己負(fù)責(zé)的代碼和相關(guān)邏輯，從Web層->DAO層；
(2)、代碼審核者在此過程中可以隨時(shí)提出自己的疑問，同時(shí)積極發(fā)現(xiàn)隱藏的bug；對(duì)這些bug記錄在案。
(3)、代碼講解完畢后，代碼審核者給自己安排幾個(gè)小時(shí)再對(duì)代碼審核一遍。
代碼需要一行一行靜下心看。同時(shí)代碼又要全面的看，以確保代碼整體上設(shè)計(jì)優(yōu)良。
(4)、代碼審核者根據(jù)審核的結(jié)果編寫“代碼審核報(bào)告”，“審核報(bào)告”中記錄發(fā)現(xiàn)的問題及修改建議，然后把“審核報(bào)告”發(fā)送給相關(guān)人員。
(5)、代碼編寫者根據(jù)“代碼審核報(bào)告”給出的修改意見，修改好代碼，有不清楚的地方可積極向代碼審核者提出。
(6)、代碼編寫者 bug fix完畢之后給出反饋。
(7)、代碼審核者把Code Review中發(fā)現(xiàn)的有價(jià)值的問題更新到"代碼審核規(guī)范"的文檔中，對(duì)于特別值得提醒的問題可群發(fā)email給所有技術(shù)人員。
5、責(zé)任：
代碼編寫者，代碼審核者共同對(duì)代碼的質(zhì)量承擔(dān)責(zé)任。這樣才能保證Code Review不是走過場(chǎng)，其中代碼編寫者承擔(dān)主要責(zé)任，代碼審核者承擔(dān)次要責(zé)任。
6、Code Review必備的文檔：
“代碼審核規(guī)范”文檔：記錄代碼應(yīng)該遵循的標(biāo)準(zhǔn)。代碼審核者根據(jù)這些標(biāo)準(zhǔn)來Code Review代碼，同時(shí)在Code Review過程中不斷完善該文檔。