代码审查

一、代码审查的目的：

1、防止Bug混入，从而可以直接提高软件质量。

     ——当然，有人认为这一点恰恰是最不重要的一点。

2、形成一种良性压力，促使团队成员编写更加整洁的代码。

     ——保证代码质量（但对软件质量的提高很可能不是很明显或者要一段时间后其效果才能显现）。 如果我们知道自己正在编写的代码稍后就有同事会对其加以审查，我们的编程态度会更加认真和谨慎。当然，即使没有代码审查，我们也会自觉的编写整洁的代码（程序员的职业素养嘛），但是这种压力通常不够紧迫，会给我们造成不一样的感受。

3、在团队内部传播知识，提高团队成员的整体水平。间接提高软件质量。

4、可以在一定程度上保证软件每一处代码有两个（或以上）人熟悉。降低项目风险。

二、代码审查的前提：

1、团队成员的意愿（文化）。

    大部分成员热情高涨，至少不排斥，特别是要获得项目负责人的支持。

2、统一的编码标准。

     无规矩不成方圆，没有团队认可的统一标准，代码审查将“无法可依”，效果会大打折扣，谁也说服不了谁。

3、团队具有一定规模。

    对任何一个模板需要同时有两个以上的开发人员在做。至少要有两个人熟悉该模块的业务规则，否则只能审查编码风格，而不能真正的理解代码和潜在的设计思路。如果团队规模太小，每个开发人员的任务都是满满的，谁也不会有兴趣和时间去学习别的模块的业务规则，更别说认真的做代码审查了。

三、代码审查的注意事项：

1、轻量。具有可行性，易于操作，不至于影响团队效率。

2、首要的是验证正确性（？但是有另一种观点：检查代码逻辑是否正确不是代码审查的任务和目的——我个人不太同意这种观点）。

3、风格的宽容：对编码风格的评审只限于编码标准，如果要讨论编码标准之外的编码风格，请先使其成为团队的编码标准之后再讨论之。

4、多鼓励少批评。

最近公司新提出一个要求，就这是代码审查，哎，而我们开发基本上是一个跟进一个项目， 其实最好的开发模式是两个人共同开发一个项目，一份代码同时有两个人熟悉，这样有了技术备份，可以提高代码质量和产品质量！

当然这也投入了更多的人力。。。适合一定规模的公司！