今天對(duì)同事寫的代碼做codereview。
在粗略看了后,沒(méi)有發(fā)現(xiàn)什么邏輯和常規(guī)錯(cuò)誤。但是,當(dāng)提交測(cè)試部測(cè)試后,個(gè)別計(jì)算機(jī)藍(lán)蘋了。(注:藍(lán)屏正是由于codereview的那個(gè)驅(qū)動(dòng)程序工程導(dǎo)致的)對(duì)此,研發(fā)經(jīng)理大為惱火。所以讓我不得不重新審視一下codereview。
codereview本意識(shí)用來(lái)提高代碼質(zhì)量的手段,那么有什么方式來(lái)保證codereview的質(zhì)量呢?
codereview實(shí)施的前提是什么?
codereview的內(nèi)容是什么?
codereview通常是編碼當(dāng)事人將代碼提交到代碼服務(wù)器上的最后一個(gè)環(huán)節(jié),這就要求review的代碼需要滿足以下幾個(gè)條件:
1、代碼已經(jīng)通過(guò)內(nèi)部測(cè)試,意味著主要邏輯應(yīng)該沒(méi)有問(wèn)題!
2、符合團(tuán)隊(duì)制定的編碼規(guī)范
除了代碼這個(gè)必要條件外,編碼人需要 條理清晰 地將代碼講述一遍!并且應(yīng)該明確:復(fù)審人對(duì)代碼的建議并不見(jiàn)得是最佳的,所以有必要對(duì)關(guān)鍵點(diǎn)同程序經(jīng)理討論決定!