这是为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