使用FxCop做代碼檢查和最佳化
來源:互聯網
上載者:User
前段時間做了一個用到一個微軟的免費測試載入器FxCop,使用後覺得還不錯,能進行一些代碼方面的自動化的測試,同時有協助學習編碼規範和提高代碼執行效率,從而提高代碼的整體品質.
這個工具能測試到的地方有: Design(設計),Globalization(全球化),Interoperability(互通性), Naming(命名),Performance(效能規則),也可以自己加入規則進行測試;
工具還是很好用的,很快就上手了
有幾點小技巧
1 可以自訂錯誤的規則,建議取消掉globalizationrules,securityrules等規則
2點中問題的串連,後可以直接連接到有代碼問題的代碼上
3可以看看breaking的Info注釋,裡面有比較好的解決方案
檢查後修改的代碼有
1 Class的命名規則
2 判斷字元為空白由
if (strStartValue.ToString() == "")
改為
if(strStartValue.Length() == 0))
3比較字串的時候
dr["PrefixM"].ToString().ToLower() == "mm" 改用String.Compare(strPrefix, "yy", true) == 0的方法
4 刪除掉方法中定義而未使用的參數
5 刪除無效的賦值
5 DataReader中未關閉(其他程式中的)
6 提示刪除程式中未使用到的方法
7 提示 catch 需要指定具體的Exception ,這樣可以節約捕獲Exception 時記憶體的使用
8 刪除多餘的using 引用
遇到的問題有
1 有大量的Globalization 要求,但是平時是沒有注意到的,雖然開發的程式要求支援多語言
strMonth = StartDateValue.Month.ToString();
這樣都會提示需要多語言的支援
2 程式未有AssembliesShouldDeclareMinimumSecurity定義
需要做簽名,但是這個一般是平時沒有注意的
3 CLR編寫的預存程序需要有out參數,但是也會提示出來,所以需要制定一套比較詳細的規則
4 在迴圈中有string相加的地方都提示要用stringbuilder這個修改是比較麻煩的,除非能之前注意,