程式碼複審
標籤: 軟體工程 程式碼複審 結對作業
程式碼複審是軟體開發過程中必不可少的環節,目的在於發現程式碼中潛在的錯誤並在成員之間傳授程式碼經驗,同時也能增進團隊成員之間的相互瞭解。同伴複審是其中相對簡便易行的方式。這次,我的工作是複審我的隊友劉宇帆的詞頻統計程式碼;
複審前的準備:
重新明確此次程式碼的需求,並和開發人員討論程式碼的基本實現流程,執行程式碼用基本資料測試通過;
複審流程:
- 閱讀程式碼,檢視程式碼的實現是否和討論的演算法一致;
- 驗證程式碼是否能符合作業需求
- 從程式碼的可讀性上進行評價;
- 程式碼是否有冗餘的部分;
- 測試程式碼效能,提出改進意見;
複審結果:
- 從演算法實現來看,基本能夠實現當時討論時開發人員的演算法思路;
- 程式碼能支援簡單的資料,但沒有支援大小寫的判斷;
- 程式碼可讀性差,沒有必要的註釋,在沒有開發人員解釋的情況下閱讀程式碼很花時間;
- 程式碼中的冗餘部分較多,沒有用到繼承,有大量類似的程式碼;
- 程式碼效能瓶頸為正規表示式的大量判斷,當時認為正則的使用會話費大量時間,建議重寫匹配的程式碼,後來發現自己寫的匹配程式碼不但實現複雜而且效能很低,只好改進其他方面;
總結:
缺點:
- 在可讀性上還有待提高;
- 沒有全方面支援需求和規格;
- 由於冗餘度較高,在維護上有很大難度;
優點:
- 函數語言程式設計,在整體程式碼結構佈局的方面值得學習;
- 靈活運用了STL,程式碼更加簡潔;