程式碼複審1234跟我一起來

UDvoid發表於2014-10-23

程式碼複審

標籤: 軟體工程 程式碼複審 結對作業


程式碼複審是軟體開發過程中必不可少的環節,目的在於發現程式碼中潛在的錯誤並在成員之間傳授程式碼經驗,同時也能增進團隊成員之間的相互瞭解。同伴複審是其中相對簡便易行的方式。這次,我的工作是複審我的隊友劉宇帆的詞頻統計程式碼;

複審前的準備:

重新明確此次程式碼的需求,並和開發人員討論程式碼的基本實現流程,執行程式碼用基本資料測試通過;

複審流程:

  1. 閱讀程式碼,檢視程式碼的實現是否和討論的演算法一致;
  2. 驗證程式碼是否能符合作業需求
  3. 從程式碼的可讀性上進行評價;
  4. 程式碼是否有冗餘的部分;
  5. 測試程式碼效能,提出改進意見;

複審結果:

  1. 從演算法實現來看,基本能夠實現當時討論時開發人員的演算法思路;
  2. 程式碼能支援簡單的資料,但沒有支援大小寫的判斷;
  3. 程式碼可讀性差,沒有必要的註釋,在沒有開發人員解釋的情況下閱讀程式碼很花時間;
  4. 程式碼中的冗餘部分較多,沒有用到繼承,有大量類似的程式碼;
  5. 程式碼效能瓶頸為正規表示式的大量判斷,當時認為正則的使用會話費大量時間,建議重寫匹配的程式碼,後來發現自己寫的匹配程式碼不但實現複雜而且效能很低,只好改進其他方面;

總結:

缺點:

  • 在可讀性上還有待提高;
  • 沒有全方面支援需求和規格;
  • 由於冗餘度較高,在維護上有很大難度;

優點:

  • 函數語言程式設計,在整體程式碼結構佈局的方面值得學習;
  • 靈活運用了STL,程式碼更加簡潔;

相關文章