1、质量保证系列课件 同行评审,HSS质量部,2012年3月,目录,测试可以取代评审吗?,为什么要进行评审?,什么是 评审?,什么是评审？,自检不属于 评审的范畴,在软件开发过程中邀请同行对工作产品进行审查，以图尽早查找出工作产品缺陷，进行质量控制的一种质量活动。 Review需要前期准备、计划，安排好时间进度表，而且越早开展对项目越有价值。,为什么要进行评审？,尽早地发现缺陷去除缺陷降低成本提高质量,提高质量 不能依赖评审,不是为了说明 产品没有缺陷,测试可以取代评审吗?,缺陷发现效率：IBM的Santa Teresa实验室发现，采用代码审查发现一个主要缺陷的平均花费为3.5小时，而测试却要花费
2、15-25小时；缺陷发现比例：在单一的测试阶段，只能去除被测工作产品中不超过35%的缺陷；而在设计和代码评审中，能发现50-70%的缺陷，有业务经验的评审者能去除90%的缺陷；代码覆盖程度：评审的覆盖率可以直接达到100%，但是有些代码，测试用例并无法全部覆盖；阻塞影响程度：评审不会出现阻塞，但是测试，可能会出现前一功能未实现好，后一功能也无法测试的现象；代码质量：测试不会体现出代码的清晰度和可维护性，但是评审可以。例如：不充分的注释、可以合并的重复代码等；,若Review发现缺陷的返工成本为1，则 测试发现缺陷的返工成本是其14.5倍； SDV测试后发现缺陷的返工成本是其45倍； 而客户发现
3、缺陷的返工成本是其68倍。,常见的评审形式,由公正的、接受过正式评审技术培训的组织者引导进行的同行检查； 对软件工作产品的一种检查，用于发现或识别软件异常，包括错误和与标准、规格的偏差。,正式评审,分发,又称走读，由产品的设计者或开发人员引导开发组成员和其它相关组成员浏览软件工作产品； 参与者可以提出疑问，或提出可能的错误、缺陷、与开发标准的偏差或其它问题的意见。,走查,又称轮查，产品的设计者或开发人员将要评审的工作产品共享或分发，评审人员以修订标记或批注的方式将意见直接添加到工作产品或其复件上。,评审对象和评审人员的选择,各阶段的工作产品及最终的工作产品都可以作为评审对象。 主要包括： SO
4、W、系统规格 项目计划、配置管理计划、测试计划、资料计划、质量保证计划 软件需求 软件设计 代码 单元测试用例 ST测试用例 SDV测试策略、 SDV测试方案、 SDV测试用例、SDV测试报告 资料文档,一个工作产品的评审必须包括如下人员： 前一阶段工作产品的作者 需要根据被评审工作产品开展后续工作的人 和被评审工作产品有依赖关系的相关组人员 作者的同行,目录,标准评审过程,1、计划阶段,PM在团队工作计划中制订Review计划； PM指定组织者规划本次评审，组织者应该熟悉评审流程，必要时由QA协助； 作者在评审启动前完成自检 文档完成自检，符合自检checklist要求 代码完成自检，源码编
5、译没有错误，通过代码检查工具检查 组织者和作者一起确定评审工作产品及评审重点； 组织者在评审启动时，就必须明确介绍会议(如果有必要)、预审、评审会议的进度安排，并知会评审参与人员； 在预审开始之前，作者将评审相关的工作产品提交给组织者(纳入配置库中)，由组织者发送给评审参与人员。,建议评审规模： 代码：=500行 文档：=40页 评审资料内容太多时，应分成几次评审内容,2、介绍会议,评审者向组织者提出申请组织者裁决是否有必要召开介绍会议若召开则,3、准备阶段,评审工作产品、发现缺陷 填写 预审表单 反馈给组织者,评审人员,组织者,汇总 预审表单 中的问题到 汇总表单 中（纳入配置库中） 裁决是
6、否需要增加预审投入,4、评审会议,组织者召开评审会议 组织者应控制评审会议的时间，评审会议时间不超过2小时大家共同确认问题 汇总表单中记录的问题 会上发现的问题 争执不下时组织者应做出裁决对已确认的问题进行分类记录员记录所有的问题及分类 发给组织者组织者更新 汇总表单，补充会议上发现的问题,召开评审会议前，组织者需要确保 预审工作 足够充分 否则 需要重新安排评审会议时间,5、第三小时会议,组织者决定是否召开第三小时会议（也可根据作者要求）会议内容 大家对 汇总表单中未解决的问题给出决议 大家对 汇总表单中已确认的问题讨论解决方案 记录员进行记录组织者更新 汇总表单,6、返工阶段,作者修改BU
7、G,汇总表单,修改后的工作产品,更新后的汇总表单 注明修改情况,7、跟踪阶段,跟踪人 （一般是问题发现者）,对照汇总表单验证工作产品中的缺陷所有接受的问题都已经被正确修改所有错误或者讨论的问题都已经写明原因所有缺陷信息填写完整且正确不正确、未实施的返工返回给作者继续修改更新汇总表单,目录,常用评审流程,评审准备,执行评审,跟踪返工,1、作者准备好工作产品，并上传配置库 2、作者发邮件通知评审人员进行评审,1、评审人员评审工作产品并导出PreReview表单 2、将PreReview表单和打批注的文档纳入配置库中,1、评审负责人（由PM指定）将所有预审表单导入到Summary表单，并放入配置库中
8、 2、评审负责人组织作者和评审人员确认所有评审出的问题是否是问题，并且填写在Summary表单中 3、作者修改问题，并将结果更新至Summary表单中 4、评审人员确认问题是否已经修改，并且将确认结果更新至Summary表单中 5、评审负责人检查评审Summary表单填写的完整性,评审注意事项,各评审人员输出评审稿时，必须使用PreReview Form，点击 按钮，生成预审表单，各评审人员还必须完善预审表单上的相关信息，包括如下信息：,各评审人员要跟踪各自的评审问题是否已修改，并更新Summary表单最后一列,评审负责人要检查评审Summary表单的完整性，并填写如下信息：,并给出评审结论，
9、填写在如下位置：,目录,评审角色,评审角色职责,目录,评审常见问题,没有评审计划,评审会议中过多争论占用大量时间,问题修改后跟踪不力,没有使用CHECKLIST做为指导,评审会议偏离主题和重点,评审常见问题解答,评审常见问题解答,评审常见问题解答,附录：组织级评审资料 规程 VER02P01-Review规程.doc VAL01P02-缺陷管理规程.doc（文档中有缺陷分类和严重级别描述）指导书 ISS-项目评审指导书.doc表单 VER02F01-PreReview Form.xls VER02F02-Review Summary Form.xls 工作产品自检Checklist 工作产品评审Checklist,谢 谢！,
