关键字：质量管理
4.6 同行评审的注意事项
为了有效地提高同行评审过程的质量，经常需要对过程数据进行度量（关于软件度量的专题，见本书第7章中相关内容），作为进一步提高过程的依据。
公司有一次组织产品需求的同行评审，会议定在5号上午9：00～11：00进行。开始之前采用邮件形式通知了参会人员，并没有把评审材料发给大家。
会议邀请了两位技术负责人，其他人员都是对技术不是很了解，且不了解评审过程与意义的管理人员，没有安排专门的人员做会议记录。
会议上，大多数管理人员按照个人的喜好与想法来评价软件的优缺点，并且对此软件的开发人员进行评论，提出了偏离评审会议主题的各种意见，使得原本安排2个小时的评审会议时间延长到了4个小时。软件中存在的问题给予了很少的关注。
主持人宣布了会议的主题。作者开始简述自己的产品需求，接下来评审提出自己的意见。
评审员小李说：“关于查询结果排序：查询后的表格应该是动态的，现在FW是固定的，这个需要改进。”
其他人也参与该问题的讨论。“如果继续使用FW提供排序功能，那么需要FW项目组进行修改，FW的负责人小张说说是否可行，打算怎么修改。”
小张开始提出自己的想法以及如何改进，几个同行也都说出自己的想法，有时会遇到不统一的现象，开始解释和说明，等这个问题讨论完了，才发现时间已经过去40分钟。大家继续后边的问题，2个小时过去后，需求评审只进行了一半，会议以没有评审结果而宣告结束，只能下次继续进行，会议中没有任何表格填写。
通过上边的例子，我们看到在评审中发生了5个违反规则的做法：
（1）采用邮件方式通知大家，没有专门通知到个人。
（2）没有预先下发被评审的工作产品和检查单。
（3）会议的焦点不是在确定问题上，而是转到了如何解决问题。针对问题的解决，讨论很多，同行评审会议最主要的是找到和确定哪些是问题，至于如何解决问题，可以在评审会后相关人员继续讨论。
（4）主持人没有经验，没有很好地主持和控制会场局面，当遇到会议跑题的时候，一定要记住会议主题，将讨论的焦点带回来，不然容易越走越远。
（5）没有作缺陷的记录和发现工作量的记录。
同行评审时，需要注意以下几点事项。
4.6.1 同行评审遵循的原则
同行评审有所谓的“123准则”：同行评审准备时间大于开会时间，同行评审期间发现的缺陷数量应该是同行评审准备期间发现的缺陷数量2倍以上，同行评审发现缺陷的效率是测试发现缺陷的3倍。
（1）同行评审需要管理层的支持，如果没有，即使是目标明确的开发组成员也会抵制进行评审。管理层的支持包括建立评审策略和目标，提供资源、时间、培训和激励，并遵守评审小组的决定等。
（2）同行评审是结构化的过程，涉及许多参与人员的角色，在评审专家的选择性上，一定要注意其中的互补性。经验表明，同行评审的参加人员在他相关的技术领域与方向发现缺陷的效率较高，需要为参加人员分配职责，会议参加人员要从不同的技术角度发现缺陷。
（3）对于每种类型的同行评审，应制定通用的工作产品评审检查表，必要时可以进行裁剪以适应特定项目的要求。工作产品评审检查表应涵盖审查计划、准备、实施、结束和报告准则。
（4）评审开始前，评审人应提前准备好自己所关注和将要提出的问题。
（5）评审的重点在于发现问题，而非解决问题，再加上认真细致的准备工作，可以最大程度避免在评审中浪费时间。
（6）对于技术人员工作的审查，应由技术人员进行，管理人员不要参与。但应将评审结果和解决所发现问题的日期通知管理人员。
（7）评审的过程是对事不对人的，例如用“这个假设是错误的”来表述，而不是尖刻地说“你的假设根本不对”。
（8）成功的审查要求所有参与人员精力高度集中，可能会使参与人员十分疲惫。因此，每个审查阶段最好不要超过2小时。对每个人来说，一天最好只参加一个阶段审查。
（9）将评审数据输入到组织度量库中，用于监测评审效果，并管理和跟踪产品质量。相关的度量数据示例有：
在全过程使用同行评审，要占10%的开发工作量；
每20页叙述性文档，需要40人时；
每12页概要设计，需要30人时；
每1000行代码，需要55人时；
使用一段时间后，评价一个项目或一个组织的审查结果需要1人月。
相关阅读：
软件质量管理实践（连载六） 软件质量管理实践（连载五） 软件质量管理实践（连载四）
软件质量管理实践（连载三） 软件质量管理实践（连载二） 软件质量管理实践（连载一）
版权声明：51Testing软件测试网获电子工业出版社授权连载《软件质量管理实践》部分章节，其他个人或单位未经许可，不得对本内容复制、转载或进行镜像。51Testing软件测试网欢迎与业内同行进行有益的合作和交流，如果有任何有关内容方面的合作事宜，请联系我们。
