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