5.1.10  重视代码评审

在进行测试之前，必须进行代码的评审。代码评审分为自动化静态代码检测与人工代码评审，如图5-9所示。首先使用自动化工具对代码进行初步审查，这样可以节约大量时间，之后再进行人工评审。在Java技术领域，自动化代码审查工具一般有Findbugs、PMD、So-narQube、CheckStyle等，具体实施方法请参照本系列教材《Java代码与架构之完美优化——实战经典》一书第2章内容。

图5-9 代码评审流程图

代码出现问题，很大程度上是因为没有重视代码评审。代码是否违反编程规约，逻辑是否清晰，架构是否合理以及潜在Bug等都需要通过代码评审来发现。70%～80%的不良代码都可以在代码评审中发现。如果代码评审良好，那么后续阶段的测试工作等就会大大减少，同时也可做出较高品质的软件产品。

很多读者可能认为80%是一个很高的数字，IBM有一篇报道，其实际代码评审时的成绩就是80%。

实际项目开发中，在没有实施代码评审或者实施力度不够时，可以采取强制措施来强化代码评审工作——项目经理让各组长实施代码评审，不进行评审就不能开始测试。

代码打印评审的重要性

现在，很多公司都提倡无纸办公，但是对于代码评审工作，极力推荐把代码打印出来进行评审，因为通过这种手段发现代码的不良率比在计算机上直接评审要高很多！