论代码走查的重要性

　　在一个团队中, 若是没有code review, 直接容许开发提交代码到版本库并部署环境, 那么在正式开始测试以前的代码走查就很是有必要了。

　　这里说的走查不是使用工具在持续化集成以前进行代码规范的检查, 而是根据PRD文档, 验证代码的实现是否符合需求描述。

　　在开始测试以前我都会先同步开发的代码, 而后询问开发人员具体有哪些接口涉及到本次功能提测, 以后从每一个接口入手, 查看业务逻辑层与数据库访问层代码, 看其实现是否与需求相符, 并找出一些明显的错误。这样作的目的只有一个, 那就是节省时间。其实这些问题应该在开发提交代码前由code review发现的， 可是因为团队刚刚组建，没有造成一套合理的流程，并且时间很是有限。其实这些恶性bug都是能够在测试阶段发现的，并且是在冒烟测试阶段就能轻易发现，可是问题仍是时间紧，不如先走查一遍代码，将错误处与开发沟通，以后提bug，将具体哪行哪一个位置的问题标出，这样能够大大加快测试进展，起码能够快速使得测试经过冒烟阶段，节约一些时间。

　　下面就举几个栗子说明我在走查阶段发现的几个明显的问题：

　　

 

首先看这段代码：

　　

在数据库读取的时候查询的是UNSETTLED类型的帐单，但查询结果直接赋值给了名为settledBills的引用，好吧，以后的代码就都是错的了。

 

再举个栗子：

　　

这里调用了dubbo服务接口，对接另外一个系统，传入的枚举UNSETTLE_BILL_REPAY，表明偿还未出帐单的意思，可是这个代码是用来偿还已出帐单的service层代码。。。

 

其实这种问题在咱们项目的代码里发现的不少不少，在两个系统对接的时候，两方开发人员几乎不沟通，看哪一个接口像就直接调用了，以后就都推给测试去测了，发现了问题再改，发现不了就呵呵了，不说了，都是泪。。。

总结一下，面对不靠谱、不沟通的开发，几乎没有管理的项目，一切靠测试手动发现问题实在是鸭梨山大啊。