進行代碼複審訓練

1、結對，找到一個夥伴進行結對；（在做業中標註本身的夥伴博客連接）

      http://www.cnblogs.com/chenyali/p/8591900.html

2、各自對本身的夥伴上週進行的「單元測試」練習所完成的代碼進行復審，造成「代碼複審檢查表」。

一、概要部分

(1)代碼符合需求和規格說明麼？

    大部分符合。

(2)代碼設計是否考慮周全？

   不夠周全，代碼中判斷出了問題，卻不輸出錯誤緣由，而是統一輸出「郵箱格式錯誤」。

(3)代碼可讀性如何？

   代碼簡單明瞭，可讀性良好。

(4)有冗餘的或重複的代碼嗎?

   if判斷語句冗餘。

(5)代碼的每一行都執行並檢查過了嗎？

  都檢查過了，沒什麼問題。

二、設計規範部分

(1)設計是否聽從已知的設計模式或項目中經常使用的模式？

   代碼遵循了已知的設計模式和在項目中的經常使用模式。

(2)有沒有硬編碼或字符串存在？

   有數字和字符串的存在。

(3)代碼有沒有依賴於某一平臺？

   java沒有依賴於某一平臺，可移植性高。

(4)有沒有無用的代碼能夠清除？

    沒有無用的代碼能夠清除。

三、代碼規範部分

    大部分仍是符合代碼標準的。

四、具體代碼部分

(1)數據結構中有沒有用不到的元素？

     數據結構中沒有用不到的元素。

(2)對於調用的外部函數，是否檢查了返回值？

    檢查了返回值。

五、效能

(1)代碼的效能如何？

    達到了具體任務要求的程度。

(2)循環中是否有明顯可優化的部分？

    有的，if判斷語句過多，能夠合併一塊兒。

六、可讀性

    可讀性良好。

七、可測試性

是否須要更新或建立新的單元測試？

    目前不須要更新或建立新的單元測試。

八、代碼複審感想

    代碼在可複用的基礎上，儘可能簡短一點。由於對於郵箱的判斷，通常都用在用戶的註冊上，因此我以爲只須要用最簡單又快捷的代碼判斷是否正確就能夠了。