軟體工程基礎作業-個人項目代碼複審

標籤：blog   http   使用   2014   問題   log   代碼   演算法   程式   

本次我是審閱左少輝同學的個人項目代碼

首先整體來說，左少輝同學的代碼優點很多：

風格不錯，大部分簡練明了，方法與方法之間的關係也是極為清楚，還有不少的注釋，易讀性強，還使用了許多庫函數，簡化了程式，如下

但是對細節仔細的觀察就會注意到這個代碼還是有不少的問題的，具體的說三個：

1.首先在breakWord方法中定義的局部變數過多，有些變數的命名也是讓人看不懂，比如Fr、FN等等。這樣就造成了程式的易讀性降低，同時也降低了程式的簡潔性

2.本程式應該是物件導向的程式，但是在本程式的sortWord方法中面向過程的特點太多，簡直就是麵條狀程式一直走下來

3.在isEqual方法中寫得太繁瑣了，完全可以簡化方法或者是提高演算法的效率

綜上所述 程式中有待提高的部分還是很多

下面給出我的改進建議：

1.提高演算法的效率，這樣能夠寫出更漂亮更簡潔的代碼

2.提高對物件導向語言的認識，這樣就避免再犯寫下面向過程程式的錯誤

3.寫代碼的風格要稍微調整一下，不要命名一些別人看不懂的局部變數，類方法的名稱也要注意，這樣程式的可讀性就強上很多。

軟體工程基礎作業-個人項目代碼複審