研发过程之代码评审
在研发过程中，为了保证代码质量，很多团队都会使用代码评审，或者叫做代码Review。一般情况下，代码Review都会采用集体Review的形式。
集体Review
形式： 一个团队，大家坐在一间会议室里面。一人进行讲解自己的业务和代码实现，团队的其他成员进行Review，提出问题，发现问题和补充不足。
这样可以达到以下的目的：
1.Linus定律 "只要有足够多的眼睛，就可让所有问题浮现"。每个人思考和理解事情的角度不同，这样发现问题也不尽相同，人数越多，发现问题也就越全面。
2.在进行补充和提出建议的过程中，可以达到组内的开发规范和标准的传播，使大家明白统一标准。
3.在讲解的过程中，和大家提出问题，发现问题的过程中，可以达到经验和知识的共享。
缺陷：耗时过长，在开发时间较短的时候，会造成时间压力过大。虽然能减少后期成本，但是对于当时来说，短期的压力。
因为集体Review耗时过长，往往会耽误大家时间，特别是项目时间紧张的时候，项目负责人往往顶不住压力，选择不Review，或者选择自己一个人进行Review，把问题发给大家。
负责人Review
形式：项目负责人进行checkout所有代码，进行逐个Review，把发现的问题email或者当面告知相关人员。
这样可以达到以下的效果：
1.避免员工设计或者代码出现问题。
2.可以达到项目负责人的开发规范和标准的传播，使大家了解其标准。
缺陷：对项目负责人要求较高，并且耗用其精力过多，使其无法处理更多关键的事项。同时因为一个人进行Review，无法对问题进行全面细致的分析，容易遗漏问题。
后来借鉴了结对编程的思想，把结对编程改为了结对Review，取得了一定的效果。
结对Review
形式：项目组成员进行结对，两人一组，互相Review对方的代码。每次进行轮换Review的对象。
这种形式，中和了负责人Review和集体Review的优点，但是同时也中和了上面两个的缺点，达到的是一种均衡。
效果如下：
1）两个人互相Review的时候，可以达到这二人业务和经验交流。
2）Review的过程中可以发现bug，并且在讲的过程中，讲述人也可以发现自身的bug。
3）加快Review进度，相对于集体Review来说，可以大幅提高Review效率，同时保证一定的效果。
4）可以达到组织知识的充分共享，组内的人会和每个人进行结对Review，可以全面传播组内人员的经验和知识。
缺陷：1）如果两人经验都不足的话，技术交流的效果不明显。但可以通过和另外一人的交流时得到补充。所以建议最开始的时候，指定结对Review人员，一名经验丰富者带一名经验较低人员。
2）缺少足够多的眼睛，有些bug会被遗漏。
3）需要Review的双方都要有责任心，如果任何一方不负责的话，都会产生问题遗漏。
上面三种Review各有其优点和缺点，建议项目上根据自己的情况进行选择，或者组合使用。现在建议的方式是每周1~2次结对Review，每2周一次集体Review，研发负责人根据情况进行抽查Review，或者在项目初期和后期进行加强Review。
敏捷
研发过程之代码评审
posted @ 2012-04-03 15:29 kaka思言丝语录 阅读(128) | 评论 (0) 编辑
每日立会之漠不关心
posted @ 2012-04-03 14:49 kaka思言丝语录 阅读(733) | 评论 (1) 编辑
每日立会之不含糊
posted @ 2012-04-03 13:57 kaka思言丝语录 阅读(916) | 评论 (3) 编辑