關於代碼審查的一點想法

重要性

可以防患於未然,提高系統的穩定性和可維護性,很重要#

常見的方式

方式一

在每次提交代碼時(或前或後),在代碼審查系統中由提交人發起代碼審查流程,由指定的審查者審查本次提交的代碼.
- 優勢: 由於每次提交的代碼都經過了審查,如有問題會及時改正,並由代碼審查系統中的流程保證,如此,代碼庫中的代碼理論上講,都是符合規範的.
- 不足: 爲了保證所有人都嚴格執行上述流程,必然需要硬性規定,給整個編碼環節增加”額外”動作. 由於是規定,而非自發,必然缺乏動力,以至於影響整個審查流程的效果,久而久之,必流於形式.

方式二

要求任何人在任何時候,發現代碼中任何人的問題,以特定方式立即向該人提出意見,並公佈在統一位置,由專人定期檢查意義的修復情況,如未修改,要麼給出不修改的理由,要麼給出修改時間.如有爭議,找經驗豐富者判斷是否需要修改,並在意見下方給出回覆或標記.
- 優勢: 提出意見者在發現問題後,並提出修改意見,顯然能證明其技高一籌,試問有誰不願意比別人強呢?而被提意見者,肯定定會努力提高自身水平,以降低犯錯的機率,有誰願意天天被人指指點點呢?
- 不足: 問題代碼不一定能夠找到主人,要麼代碼記錄沒了,要麼人已經走了; 整個流程不便於監督,目前並未發現此類跟蹤系統;在某些時候,不一定要完美的代碼.

本人推薦第二種.

發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章