程式碼互審

我已經報警了發表於2014-10-22

  我複審的程式碼是來自於12061174 李靖的單詞搜尋程式碼。

  程式碼能正確實現要求的功能,即從指定的目錄下讀取所有指定型別的文字文件,統計其中符合規定的“單詞”的詞頻並輸出到文字檔案中

  但是,程式碼還是存在著一些不足。

  1.變數的命名不規範,可讀性很差。比如main函式中,使用了d,m之類的名字來命名字串,讀的人完全不知道什麼意思,以後自己再回頭看程式碼也許也會讀不懂。建議將這個變數表達的意思用變數名體現出來,讓人易於理解程式。

  2.程式碼比較冗餘,程式碼的各個部分的封閉性獨立性都比較差。感覺上沒有做到物件導向,而是程式導向的程式,由於程式碼很長,可讀性就比較差。建議寫成物件導向的程式碼,各個部分有明確的輸入引數和指定功能,各自獨立,這樣子對功能擴充套件,debug,增強程式碼可讀性都有好處。

  3.本程式是用c++寫的,對於這種需要做比較多的檔案操作的程式來說,我覺得使用c#更好一些,因為c#整合了很多檔案操作的功能,比如直接迴圈讀取一個目錄和其所有子目錄下所有檔案路徑的函式,本程式用了很大功夫來完成這一功能,而c#只要一個函式即可,而且使用c#也能更好地做到物件導向。

  優點:程式對於文字內容的處理使用了正規表示式,清晰明確。整個程式的思路也比較清晰。

相關文章