給 C# 開發者的代碼審查清單

原文地址:http://blog.jobbole.com/46255/

這是爲C#開發者準備的通用性代碼審查清單,可以當做開發過程中的參考。這是爲了確保在編碼過程中,大部分通用編碼指導原則都能注意到。對於新手和缺乏經驗(0到3年工作經驗)的開發者,參考這份清單編碼會很幫助。


清單

1. 確保沒有任何警告(warnings)。

2.如果先執行Code Analysis(啓用所有Microsoft Rules)再消除所有警告就更好了。

3. 去掉所有沒有用到的usings。編碼過程中去掉多餘代碼是個好習慣。(參考:msdn

4. 在合理的地方檢查對象是否爲’null’,避免運行的時候出現Null Reference Exception。

5. 始終遵循命名規範。一般而言變量參數使用駝峯命名法,方法名和類名使用Pascal命名法。(參考:msdn

6. 請確保你瞭解SOLID原則。

根據維基百科定義:在程序設計領域,SOLID (單一功能、開閉原則、里氏替換、接口隔離以及依賴反轉)是由羅伯特·C·馬丁在21世紀早期引入的記憶術首字母縮略字,指代了面向對象編程和麪向對象設計的五個基本原則。當這些原則被一起應用時,它們使得一個程序員開發一個容易進行軟件維護和擴展的系統變得更加可能。SOLID所包含的原則是通過引發編程者進行軟件源代碼的代碼重構進行軟件的代碼異味清掃,從而使得軟件清晰可讀以及可擴展時可以應用的指南。SOLID被典型的應用在測試驅動開發上,並且是敏捷開發以及自適應軟件開發的基本原則的重要組成部分。參考:wiki/SOLID_(面向對象設計)

7. 代碼可重用性:如果一塊代碼已經被使用超過一次,或者你希望將來使用它,請提取成一個方法。將重複的工作做成通用的方法放在相關的類中,這樣一旦你完成別人就可以使用了。將常用功能開發成用戶控件,這樣可以跨項目重用它們。(參考: 、 

8. 代碼一致性:比方說,Int32寫成int,String寫成string,應該在代碼裏保持統一形式。不能一會二寫成int一會兒寫成Int32。

9. 代碼可讀性:代碼應該是可維護的,便於其他開發者理解。(參考:msdn

10. 釋放非託管資源,比如文件I/O,網絡資源等。一旦使用結束就應該釋放它們。如果你想一旦超出使用範圍就自動釋放對象,可以使用usings將非託管代碼括起來。參考:msdn

11. 合理實現異常處理(try/catch和finally塊)和異常記錄。參考:msdn

12. 確保代碼中方法的行數不要過多,不超過30到40行。

13. 及時用代碼管理工具check-in/check-out代碼。(比如TFS) 參考:codeproject.com

14. 相互審查代碼:和你的同事交換代碼,實現內部審查。

15. 單元測試:編寫開發測試用例完成單元測試,確保代碼被送到QA以前,基本測試完成。參考:msdn

16. 儘量避免for/foreach循環嵌套和if條件嵌套。

17. 如果代碼只會使用一次,請使用匿名類型。參考:msdn

18. 儘量使用LINQ查詢和Lambda表達式,增加可讀性。參考:msdn

19. 合理使用var、object和dynamic關鍵字。由於很多開發者會感到困惑或者知道的很少,會覺得它們有些相似,故而交換使用,這是要避免的。參考:blogs.msdn

20. 使用訪問限定符(private, public, protected, internal, protected internal)限定每個方法、類或變量的需要範圍。比方說如果一個類只會在程序集內使用,那麼定義成internal就足夠了。參考:msdn

21. 在需要保持解耦的地方使用接口,有些設計模式的出現也是由於接口的使用。參考:msdn

22. 按照用法和需要將類定義爲sealed、static或abstract。參考:msdn

23. 如果需要多次串聯,請使用Stringbuilder代替string,這可以節省堆內存。

24. 檢查是否有不可能執行的代碼,如果有,請修改。

25. 在每個方法前註釋,說明它的用法、輸入類型和返回值類型信息。

26. 使用類似Silverlight Spy的工具,檢查和操控Silverlight應用在運行時對XMAL的渲染,以此來改善效率。這可以在設計執行XAML時,節省大量退回和來回修改的時間。

27. 使用filddler工具通過檢查HTTP/網絡流量和帶寬,來跟蹤web應用和服務的性能。

28. 如果你想確認Visual Studio以外的方法,請使用WCFTestClient.exe工具,或者裝載它的進程到Visual Studio來進行調試。

29. 在任何合理的地方使用constants和readonly。參考:/msdnmsdn

30. 儘量避免強制轉換和類型轉換,因爲會造成性能損失。參考:msdn

31. 對於你想提供自定義信息的類,請重載ToString(來自Object類)。參考:msdn

32. 避免直接從其他代碼中ctrl+c/ctrl+v。一直建議還是自己用手敲,即使你已經找到相關代碼。這樣可以鍛鍊自己寫代碼能力,還能正確理解那段代碼的用法。最終你永遠都不會忘記那段代碼。

33. 保持閱讀書籍和文章的良好習慣,遵循大神們的實踐指導。(比如微軟專家和一些著名的專家,Martin Fowler, Kent Beck, Jeffrey Ritcher, Ward Cunningham, Scott Hanselman, Scott Guthrie, Donald E Knuth.)

34. 確認代碼是否有內存泄漏。如果有,請確保已修正。參考:blogs.msdn.com

35. 儘可能參加專家們組織的技術研討會,可以接觸到最新的軟件趨勢、技術和最佳實踐

36. 要透徹理解OOP概念,並儘可能在代碼裏實現。

37. 知道項目設計架構,可以從整體上理解程序的執行流程。

38. 採取必要措施阻止避免任何交叉腳本攻擊、SQL注入和其他安全漏洞。

39. 永遠記得將保密和敏感信息加密(通過使用好的加密算法),比如保存到數據庫的密碼和保存在web.config文件中的連接字符,要避免被非認證的用戶操縱。

40. 避免對已知類型(原始類型)使用默認關鍵字,比如int, decimal, bool等。多數情況下,如果不確定是值類型還是引用類型,就使用泛型類型(T)。參考:msdn

41. 微軟(在代碼分析條例和指導中)並不推薦使用’out’和’ref’,這些關鍵字是通過引用傳參,請注意,’ref’參數在傳入被調用方法之前,應當在調用方法中先初始化,但’out’參數就不是這樣。參考:msdn

原文鏈接: Mohammed Hameed   翻譯: 伯樂在線 伯樂在線讀者
譯文鏈接: http://blog.jobbole.com/46255/



發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章