代码审查：程序员内炼之道

http://www.csdn.net/article/2013-01-28/2813977-code-review

 

摘要：“关注并弄清楚桥梁修建细节，否则你建起来的桥梁有可能坍塌。”代码审查更重要的是一种技术分享或者代码共享。程序员如何提升自我修炼之道，欢迎来支招。

代码审查更重要的是一种技术分享或者代码共享。在审查过程中，通过被审查者的讲解、审查人员的质询来发现问题并传播知识。如果审查者做过相关东西并且有更好的解决方案，能够提出来也可以做到一定的技术分享；同时被审查者如果被审查出的问题较多，也可以在一定程度上提高对技术的追求。

此前，我们曾发表过《 高效代码审查的10大经验 》，文章提到：1.代码审查要求团队有良好的文化；2.谨慎的使用审查中问题的发现率作为考评标准；3.控制每次审查的代码数量；4.带着问题去进行审查；5.所有的问题和修改，必须由原作者进行确认；6.利用代码审查激活个体“能动性”；7.在非正式，轻松的环境下进行代码审查；8.提交代码前自我审查，添加对代码的说明；9.实现中记录笔记可以很好的提高问题发现率；10.使用好的工具进行轻量级的代码审查。

开发者可分为资深开发、高级开发、中级开发、初级开发四个级别，针对不同的级别，如何进行代码分配审查工作呢。本文我们将探讨程序员如何提升自我修炼之道。

阅读能力，追踪及支持

1. 提高代码的可读性比如代码的完整性，代码洁净、同行代码审查，通过查看代码指南从而获取一些观点。
2. 改进应用程序追踪和支持性——与任何运行你程序的开发者进行交谈，从对方得知需要改进的地方，吸取好的建议，为代码审查列一份清单。

可靠性和可测性

1. .NET重构：注射质量，可测性以及给现有系统添加架构。
2. 给断开的特定服务器绘制一份草图。
3. 审查所有重试循环和确保他们的行为是合理的。

性能、安全、并发

1.   在性能部分阅读CLRS并做一些实践，Programming pearls（编程珠玑）将会对你有所帮助。
2.  托管代码的安全审查 。
3. 并发——阅读有关.net线程模型（查看文档）；了解异步同步模型以及它的工作原理，尝试在纸上绘画出并发工作时状态，查看访问资源是否有问题。

软件设计：

1. 阅读关于模式方面的书籍。
2. 尝试用不同的模式进行编码。
3. 阅读框架中的依赖注入以及流控制方面的书籍（MVC, MVVM）这个网站涵盖了许多有用的资源，可以参考。

另外，在代码审查期间，使用TODO和FIXME来标记代码是非常有好处的。

各位开发者，你们所在的公司有严格的代码审查机制吗？如果有，欢迎与我们分享经验。

英文出自： MSDN（编译/夏梦竹）