浅谈iOS开发中code review(代码审查)

代码评审是指在软件开发过程中，通过对源代码进行系统性检查的过程。通常的目的是查找系统缺陷，保证软件总体质量和提升开发者自身水平。代码评审通常有两张类型：正式代码评审(formal code review)，轻量级代码评审(light code review)。

1、为什么要进行代码评审

a、提高质量
b、及早发现缺陷与bug，降低事故成本
c、开发内部知识共享，提高团队整体水平

d、对开发人员来说也是一次开发思想重新重构的过程，可以更好的理解系统

2、代码评审的方式和时间

a、由作者启动评审，展示评审文档(over-the shoulder)。它的优点是开发团队内部可以快速进行评审交流，我个人觉得对于项目架构，后期可扩展性，可维护性等可以启用这样的评审，通过启动这样的评审，一方面评审者可以学习作者的架构思想，另一方面也可以团队内部讨论架构上的不足加以改进。

b、组织评审会议(review meeting)，由团队成员轮流选出自己的评审作品共其它评审者学习。我个人觉得可以在项目结束后进行，通过这种敦促的方式共享好的东西让整体团队一起进步

c、利用评审工具(tool assited code review)   可以参考：http://blog.csdn.net/haoel/article/details/4469462

3、代码评审的层级

a、实现正确性：主要包括是否实现预期功能，是否存在bug，是否存在性能问题等。

b、设计合理性：主要包括实现方法，数据接口，设计模式，扩展性等，是否存在大量重复代码和其它组件是否有重复代码，包括结构设计是否合理，是否存在性能问题等

c、代码可读性：主要包括代码风格，美观性。

实现正确性和设计合理性笔者认为是必须要进行的code review，对于代码可读性，由于每个人的编码风格不一样，建议做最低级别的review，比如注释。