任務說明:
一、結對,找到一個伙伴進行結對;(在作業中標注自己的伙伴博客鏈接)
王小梅:?http://www.cnblogs.com/CURRY-30/p/8570290.html
二、各自對自己的伙伴上周進行的“單元測試”練習所完成的代碼進行復審,形成“代碼復審檢查表”。
代碼復審檢查表 | ||
概要部分 | 代碼符合需求和規格說明么? | 基本符合。 |
代碼設計是否考慮周全? | 不是很周全。 | |
代碼可讀性如何? | 較簡單。 | |
有冗余的或重復的代碼嗎? | 沒有。 | |
代碼的每一行都執行并檢查過了嗎? | 是的。 | |
設計規范部分 | 設計是否遵從已知的設計模式或項目中常用的模式? | 是的, |
有沒有硬編碼或字符串存在? | 沒有。 | |
代碼有沒有依賴于某一平臺? | 沒有。 | |
有沒有無用的代碼可以清除? | 沒有。 | |
代碼規范部分 | 修改的部分符合代碼標準和風格么? | 代碼基本符合規范標準和風格。 |
具體代碼部分 | 數據結構中有沒有用不到的元素? | 沒有。 |
對于調用的外部函數,是否檢查了返回值? | 是的。 | |
效能 | 代碼的效能如何? | 基本達到了任務要求。 |
循環中是否有明顯可優化的部分? | 沒有明顯可優化部分。 | |
可讀性 | 代碼可讀性如何?有沒有足夠的注釋? | 可讀性中等。 |
可測試性 | 是否需要更新或創建新的單元測試? | 需要,代碼存在部分問題,測試時會報錯但仍能顯示結果。 |
8、代碼復審感想?
個人認為,代碼復審這一步在這個流程中是非常重要的,通過復審可以找出很多的錯誤,也可以完善之前的漏洞。而且復審自己也可以學到很多知識,通過對別人代碼的檢查,從總學到了別人對代碼的理解,取其精華去其糟粕。而且檢查出錯誤的同時,也可以給自己一個教訓,以免下次不會犯同樣的錯誤。