經(jīng)過(guò)在公司1年多的code review的經(jīng)驗(yàn)回顧：原來(lái)有fisheye，開(kāi)發(fā)提交代碼后旺旺通知測(cè)試工程師，并通過(guò)讀代碼來(lái)了解測(cè)試范圍，并發(fā)現(xiàn)代碼中的錯(cuò)誤。
后來(lái)，最近半年的項(xiàng)目、日常測(cè)試過(guò)程中都是開(kāi)發(fā)提交代碼后，測(cè)試和開(kāi)發(fā)一起借用SVN工具等代碼版本控制工具，或Eclipse 等IDE進(jìn)行Code Review。
這其中的一個(gè)轉(zhuǎn)變就是測(cè)試由被動(dòng)接收消息，到主動(dòng)查看SVN 的log看代碼變動(dòng)。測(cè)試工程師的態(tài)度由被動(dòng)變?yōu)橹鲃?dòng)，是個(gè)不小的進(jìn)步。
目前看來(lái)，進(jìn)行 Code Review 的目的或效果有：
1、讓測(cè)試熟悉所測(cè)產(chǎn)品的業(yè)務(wù)代碼，提升代碼的閱讀能力；
2、提早發(fā)現(xiàn)代碼里面的bug，低成本保障質(zhì)量，防患于未然；
3、提前預(yù)知并評(píng)估并確認(rèn)測(cè)試范圍，減少測(cè)試工作量；
4、促進(jìn)開(kāi)發(fā)、測(cè)試間的溝通、交流和協(xié)作。
功能測(cè)試工程師參加code review提前做的一些準(zhǔn)備：
1、簡(jiǎn)單的編碼規(guī)范
2、Java編程的基本知識(shí)
經(jīng)過(guò)這Code Review的實(shí)踐，感覺(jué)Code Review目前比較適合我們工作的方式是：
階段 | Code Review的方式 |
準(zhǔn)備 | 1、了解開(kāi)發(fā)的UC設(shè)計(jì)，及基本的編碼知識(shí)；
2、了解基本的代碼的編碼規(guī)范；
3、確定code review的范圍：業(yè)務(wù)的核心代碼邏輯。 |
形式 | 項(xiàng)目：會(huì)議室+投影儀。日常：在開(kāi)發(fā)/測(cè)試的位置上即可。 |
參加人員 | PM，PTM，相關(guān)開(kāi)發(fā)工程師、測(cè)試工程師 |
可以采用的方法 | 1、編碼人員講解，其他開(kāi)發(fā)、測(cè)試人員檢查。
2、代碼靜態(tài)檢測(cè)工具：Findbug
3、缺陷檢查表，但是這個(gè)太正式了，不一定需要。 |
注意點(diǎn) | 1、限時(shí)：一般不要超過(guò)1個(gè)小時(shí)為宜；如果量大，最好分批review。
2、不要現(xiàn)場(chǎng)修改代碼，發(fā)現(xiàn)問(wèn)題后，測(cè)試可以直接在bug管理平臺(tái)記錄。 |
產(chǎn)出 | 1、Bug記錄；
2、靜態(tài)分析錯(cuò)誤報(bào)告；
3、結(jié)果：code review 是否通過(guò)。 |
比較合適的，并且目前使用的流程是：
