代碼復審訓練
阿新 • • 發佈:2018-03-26
數據結構 註意 .com 感覺 測試 規格 常用 html 標準
一、結對,找到一個夥伴進行結對
張劍斌 :http://www.cnblogs.com/janbin/p/8594002.html
二、代碼復審核查表
代碼復審核查表 |
||
概要部分 |
代碼符合需求和規格說明麽? |
符合 |
代碼設計是否考慮周全? |
否 |
|
代碼可讀性如何? |
一般 |
|
有冗余的或重復的代碼嗎? |
無 |
|
代碼的每一行都執行並檢查過了嗎? |
是 |
|
設計規範部分 |
設計是否遵從已知的設計模式或項目中常用的模式? |
不遵循 |
有沒有硬編碼或字符串/數字等存在? |
有 |
|
代碼有沒有依賴於某一平臺? |
沒有 |
|
有沒有無用的代碼可以清除? |
有 |
|
代碼規範部分 |
修改的部分符合代碼標準和風格麽? |
基本符合 |
具體代碼部分 |
數據結構中有沒有用不到的元素? |
無 |
對於調用的外部函數,是否檢查了返回值 |
是 |
|
效能 |
代碼的效能如何? |
一般 |
循環中是否有明顯可優化的部分? |
是 |
|
可讀性 |
代碼可讀性如何?有沒有足夠的註釋? |
可讀性強,但沒有註釋 |
可測試性 |
是否需要更新或創建新的單元測試 |
是 |
代碼復審感想:編寫規範並不是很好,一個註釋都沒有,不易閱讀。當測試郵箱有多處錯誤時,只能報出來一個錯誤,也不易於用戶的修改。通過這次復審,感覺自己平常編程的時候很多地方沒有註意,小項目可能影響不大,但是大項目尤其重視代碼規範,特別是多人開發的時候,你自己的隨心所欲可能會讓其他小夥伴難受,為了大家編程都舒服,在項目開始的時候就要定好一個代碼規範。 |
代碼復審訓練