Code Review主要用來(lái)在軟件工程過(guò)程中改進(jìn)代碼質(zhì)量，通過(guò)Code Review可以達(dá)到如下目的目的：
如果做Code Review的人員不能理解Code Review對(duì)項(xiàng)目成敗和代碼質(zhì)量的重要程度，他們的做法可能就會(huì)是應(yīng)付了事。
Code Review人員也不負(fù)責(zé)檢查代碼的功能是否正確，也就是說(shuō)，需要復(fù)查的代碼必須由開(kāi)發(fā)人員或質(zhì)量人員負(fù)責(zé)該代碼的功能的正確性。
這一點(diǎn)也是為了保證Code Review前一些語(yǔ)法和功能問(wèn)題已經(jīng)得到解決，Code Review人員可以將精力集中在代碼的質(zhì)量上。
1.3.3 正確性檢查（Correctness）
代碼是否符合制定的標(biāo)準(zhǔn)
所有的變量都被正確定義和使用
所有的注釋都是準(zhǔn)確的
所有的程序調(diào)用都使用了正確的參數(shù)個(gè)數(shù)
1.3.4 可修改性檢查（Modifiability）
代碼涉及到的常量是否易于修改（如使用配置、定義為類(lèi)常量、使用專(zhuān)門(mén)的常量類(lèi)等）
代碼中是否包含了交叉說(shuō)明或數(shù)據(jù)字典，以描述程序是如何對(duì)變量和常量進(jìn)行訪(fǎng)問(wèn)的
代碼是否只有一個(gè)出口和一個(gè)入口（嚴(yán)重的異常處理除外）
1.3.5 可預(yù)測(cè)性檢查（Predictability）
代碼所用的開(kāi)發(fā)語(yǔ)言是否具有定義良好的語(yǔ)法和語(yǔ)義
是否代碼避免了依賴(lài)于開(kāi)發(fā)語(yǔ)言缺省提供的功能
代碼是否無(wú)意中陷入了死循環(huán)
代碼是否是否避免了無(wú)窮遞歸
1.3.6 健壯性檢查（Robustness）
代碼是否采取措施避免運(yùn)行時(shí)錯(cuò)誤（如數(shù)組邊界溢出、被零除、值越界、堆棧溢出等）
1.3.7 結(jié)構(gòu)性檢查（Structuredness）
程序的每個(gè)功能是否都作為一個(gè)可辯識(shí)的代碼塊存在
循環(huán)是否只有一個(gè)入口
1.3.8 可追溯性檢查（Traceability）
代碼是否對(duì)每個(gè)程序進(jìn)行了唯一標(biāo)識(shí)
是否有一個(gè)交叉引用的框架可以用來(lái)在代碼和開(kāi)發(fā)文檔之間相互對(duì)應(yīng)
代碼是否包括一個(gè)修訂歷史記錄，記錄中對(duì)代碼的修改和原因都有記錄
是否所有的安全功能都有標(biāo)識(shí)
1.3.9 可理解性檢查（Understandability）
注釋是否足夠清晰的描述每個(gè)子程序
是否使用到不明確或不必要的復(fù)雜代碼，它們是否被清楚的注釋
使用一些統(tǒng)一的格式化技巧（如縮進(jìn)、空白等）用來(lái)增強(qiáng)代碼的清晰度
是否在定義命名規(guī)則時(shí)采用了便于記憶，反映類(lèi)型等方法
每個(gè)變量都定義了合法的取值范圍
代碼中的算法是否符合開(kāi)發(fā)文檔中描述的數(shù)學(xué)模型
1.3.10 可驗(yàn)證性檢查（Verifiability）
代碼中的實(shí)現(xiàn)技術(shù)是否便于測(cè)試
1.4 Code Review的步驟
這些是我在平時(shí)工作中的經(jīng)驗(yàn)總結(jié)，目前也是按照這個(gè)步驟在做。
（1）代碼編寫(xiě)者和代碼審核者坐在一起，由代碼編寫(xiě)者按照UC依次講解自己負(fù)責(zé)的代碼和相關(guān)邏輯，從Web層->DAO層；
（2）代碼審核者在此過(guò)程中可以隨時(shí)提出自己的疑問(wèn)，同時(shí)積極發(fā)現(xiàn)隱藏的bug；對(duì)這些bug記錄在案。
（3）代碼講解完畢后，代碼審核者給自己安排幾個(gè)小時(shí)再對(duì)代碼審核一遍。
代碼需要一行一行靜下心看。同時(shí)代碼又要全面的看，以確保代碼整體上設(shè)計(jì)優(yōu)良。
（4）代碼審核者根據(jù)審核的結(jié)果編寫(xiě)“代碼審核報(bào)告”，“審核報(bào)告”中記錄發(fā)現(xiàn)的問(wèn)題及修改建議，然后把“審核報(bào)告”發(fā)送給相關(guān)人員。
（5）代碼編寫(xiě)者根據(jù)“代碼審核報(bào)告”給出的修改意見(jiàn)，修改好代碼，有不清楚的地方可積極向代碼審核者提出。
（6）代碼編寫(xiě)者 bug fix完畢之后給出反饋。
（7）代碼審核者把Code Review中發(fā)現(xiàn)的有價(jià)值的問(wèn)題更新到“代碼審核規(guī)范”的文檔中，對(duì)于特別值得提醒的問(wèn)題可群發(fā)email給所有技術(shù)人員。
提示
Code Review必備的文檔：
“代碼審核規(guī)范”文檔：記錄代碼應(yīng)該遵循的標(biāo)準(zhǔn)。
代碼審核者根據(jù)這些標(biāo)準(zhǔn)來(lái)Code Review代碼，同時(shí)在Code Review過(guò)程中不斷完善該文檔。
2、Code Reivew的執(zhí)行
一個(gè)標(biāo)準(zhǔn)的Code Reivew活動(dòng)應(yīng)該分為三個(gè)階段：
2.1 事前準(zhǔn)備階段
在一次CR前，對(duì)以下內(nèi)容進(jìn)行充分準(zhǔn)備。
2.1.1 CR的對(duì)象
在準(zhǔn)備CR代碼對(duì)象時(shí)，我們要注意代碼的數(shù)量，如果代碼量比較大，要對(duì)代碼進(jìn)行必要的分解，確定其中的關(guān)鍵代碼，對(duì)關(guān)鍵代碼進(jìn)行CR，可以達(dá)到舉一反三的目的。
2.1.2 CR的內(nèi)容
我們對(duì)代碼的審查內(nèi)容很多，如代碼的編寫(xiě)是否規(guī)范（注釋的書(shū)寫(xiě)格式、命名規(guī)范等）、技術(shù)處理規(guī)范（異常處理、日志處理、代碼組織結(jié)構(gòu)等）、業(yè)務(wù)實(shí)現(xiàn)等。我們不能希望通過(guò)一次CR活動(dòng)，完成所有這些內(nèi)容的審查，因此我們必須設(shè)定本次CR活動(dòng)內(nèi)容界限，確定審查重點(diǎn)；
2.1.3 評(píng)審規(guī)范和標(biāo)準(zhǔn)
在CR前設(shè)計(jì)確定評(píng)審規(guī)范和標(biāo)準(zhǔn)是必要，通過(guò)規(guī)范和標(biāo)準(zhǔn)我們?cè)趯彶檫^(guò)程中可以有據(jù)可依，有理可循，而且還可以做到標(biāo)準(zhǔn)統(tǒng)一。
2.1.4 選擇CR活動(dòng)的參與者
在CR開(kāi)始前，必須把本次CR活動(dòng)的對(duì)象、審查內(nèi)容以及審查的規(guī)范和標(biāo)準(zhǔn)通報(bào)給所有的參與者。
2.1.5 選擇CR活動(dòng)的實(shí)施方式。
CR活動(dòng)有很多形式可供我們選擇，我們可以根據(jù)實(shí)際情況選擇桌面式CR、演示講解式CR、一對(duì)一的座位CR等等。
2.2 實(shí)施階段
充分的事前準(zhǔn)備，只是做好CR活動(dòng)的前提，在CR實(shí)施過(guò)程中，我們要做好以下工作。
2.2.1 準(zhǔn)確記錄
對(duì)于CR過(guò)程發(fā)現(xiàn)的問(wèn)題，我們必須清晰準(zhǔn)確的記錄，可以使用問(wèn)題點(diǎn)記錄單，明確記錄的項(xiàng)目和內(nèi)容。
2.2.2 講解與提問(wèn)
CR過(guò)程中，要采用代碼作者講解和審查者提問(wèn)方式。審查者不能只在發(fā)現(xiàn)問(wèn)題時(shí)提問(wèn)，同時(shí)也要根據(jù)本次審查的內(nèi)容要求代碼作者對(duì)某個(gè)特定問(wèn)題的講解。
2.2.3 逐項(xiàng)審查
對(duì)事前確定的審查內(nèi)容，要逐項(xiàng)審查，不能因?yàn)闀r(shí)間不足等因素一掃而過(guò)。
2.2.4 注意氣氛
實(shí)施審查時(shí)，要營(yíng)造一個(gè)討論問(wèn)題、解決問(wèn)題的氛圍，不能把審查會(huì)搞成批判會(huì)，這樣會(huì)影響相關(guān)人員的積極性。
2.3 事后跟蹤跟蹤。
2.3.1 確認(rèn)發(fā)現(xiàn)的問(wèn)題
CR結(jié)束后，對(duì)發(fā)現(xiàn)的問(wèn)題，首先需要確定以下內(nèi)容。
1）問(wèn)題點(diǎn)的難易程度以及影響的范圍；
2）解決問(wèn)題的責(zé)任者和問(wèn)題點(diǎn)修正結(jié)果的確認(rèn)者；
3）解決問(wèn)題點(diǎn)的時(shí)限。
2.3.2 修正問(wèn)題責(zé)任者
對(duì)于修正問(wèn)題責(zé)任者，在問(wèn)題點(diǎn)的修正過(guò)程中，要三方面內(nèi)容的記錄。
1）問(wèn)題點(diǎn)的原因；
2）解決問(wèn)題點(diǎn)的對(duì)策；
3）修正的內(nèi)容。
2.3.3 修正結(jié)果確認(rèn)者
做為修正結(jié)果的確認(rèn)者，必須按照事前約定的時(shí)限及時(shí)的對(duì)修正結(jié)果進(jìn)行全面的確認(rèn)
3、注意事項(xiàng)
3.1 經(jīng)常進(jìn)行Code Review
（1）要Review的代碼越多，那么要重構(gòu)，重寫(xiě)的代碼就會(huì)越多。而越不被程序作者接受的建議也會(huì)越多，唾沫口水戰(zhàn)也會(huì)越多。
（2）程序員代碼寫(xiě)得時(shí)候越長(zhǎng)，程序員就會(huì)在代碼中加入越來(lái)越多的個(gè)人的東西。
（3）越接近軟件發(fā)布的最終期限，代碼也就不能改得太多。
3.2 Code Review不要太正式，而且要短
忘了那個(gè)代碼評(píng)審的Checklist吧，走到你的同事座位跟前，像請(qǐng)師父一樣請(qǐng)他坐到你的電腦面前，然后，花5分鐘給他講講你的代碼，給他另外一個(gè)5分鐘讓他給你的代碼提提意見(jiàn)，這比什么都好。而如果你用了一個(gè)Checklist，讓這個(gè)事情表現(xiàn)得很正式的話(huà)，下面兩件事中必有一件事會(huì)發(fā)生：
（1）只有在Checklist上存在的東西才會(huì)被Review。
（2）Code Reviews 變成了一種禮節(jié)性的東西，你的同事會(huì)裝做很關(guān)心你的代碼，但其實(shí)他心里想著盡快地離開(kāi)你。
只有不正式的Code Review才會(huì)讓你和評(píng)審者放輕松，人只有放松了，才會(huì)表現(xiàn)得很真實(shí)，很真誠(chéng)。記住Review只不過(guò)是一種形式，而只有在相互信任中通過(guò)相互的討論得到了有意義和有建設(shè)性的建議和意見(jiàn)，那才是最實(shí)在的。不然，作者和評(píng)審者的關(guān)系就會(huì)變成小偷和警察的關(guān)系。
3.3 盡可能的讓不同的人Reivew你的代碼
如果可能的話(huà)，不要總是只找一個(gè)人來(lái)Review你的代碼，不同的人有不同的思考方式，有不同的見(jiàn)解，所以，不同的人可以全面的從各個(gè)方面評(píng)論你的代碼。
但不要太多了，人多嘴雜反而適得其反，基本上來(lái)說(shuō)，不要超過(guò)3個(gè)人，這是因?yàn)椋@是一個(gè)可以圍在一起討論的最大人員尺寸。
下面是幾個(gè)優(yōu)點(diǎn)：
（1）從不同的方向評(píng)審代碼總是好的。
（2）會(huì)有更多的人幫你在日后維護(hù)你的代碼。
（3）這也是一個(gè)增加團(tuán)隊(duì)凝聚力的方法。
3.4 保持積極的正面的態(tài)度
程序員最大的問(wèn)題就是“自負(fù)”，尤其當(dāng)我們Reivew別人的代碼的時(shí)候，我已經(jīng)見(jiàn)過(guò)無(wú)數(shù)的場(chǎng)面，程序員在Code Review的時(shí)候，開(kāi)始抨擊別人的代碼，質(zhì)疑別人的能力。太可笑了，我分析了一下，這類(lèi)的程序員其實(shí)并沒(méi)有什么本事，因?yàn)樗麄冎肛?zé)對(duì)方的目的是想告訴大家自己有多么的牛，靠這種手段來(lái)表現(xiàn)自己的程序員，其實(shí)是就是傳說(shuō)中所說(shuō)的“半瓶水”。
所以，無(wú)論是代碼作者，還是評(píng)審者，都需要一種積極向上的正面的態(tài)度，作者需要能夠虛心接受別人的建議，因?yàn)閯e人的建議是為了讓你做得更好；評(píng)審者也需要以一種積極的正面的態(tài)度向作者提意見(jiàn)，因?yàn)槟鞘呛湍阍谝粋€(gè)戰(zhàn)壕里的戰(zhàn)友。記住，你不是一段代碼，你是一個(gè)人！
3.5 學(xué)會(huì)享受Code Reivew
這可能是最重要的一個(gè)提示了，如果你到了一個(gè)人人都喜歡Code Reivew的團(tuán)阿，那么，你會(huì)進(jìn)入到一個(gè)生機(jī)勃勃的地方，在那里，每個(gè)人都能寫(xiě)出質(zhì)量非常好的代碼，在那里，你不需要經(jīng)理的管理，團(tuán)隊(duì)會(huì)自適應(yīng)一切變化，他們相互學(xué)習(xí)，相互幫助，不僅僅是寫(xiě)出好的代碼，而且團(tuán)隊(duì)和其中的每個(gè)人都會(huì)自動(dòng)進(jìn)化，最關(guān)鍵的是，這個(gè)是一個(gè)團(tuán)隊(duì)。