评审流程.docx

1、评审流程评审规程编 写兰洋编写 时间2013-8-1审 批审批 时间2013-8-18文档版本V1.0未经允许，不可全部或部分发表、复制、使用于任何目的。目录第1章 引言 31.1 文档用途 31.2 阅读对象 31.3 名词术语 31.3.1 同行评审 3第2章 评审定义 42.1 评审的目的 42.2 评审分类 42.2.1 审查（Inspection） 42.2.2 小组评审（ Team Review ） 42.2.3 走查（ Walkthrough ） 52.2.4 结对编程（ Pair Programming ） 52.2.5 桌查（ Deskcheck ） 52.2.6 同行桌查（

2、 Peer Deskcheck ） 52.2.7 轮查（ Passaround ） 6第3章 正式评审流程 73.1 角色和职责 73.2 准备评审 73.2.1 概述 73.2.2 主要步骤 73.3 个人审查 83.3.1 概述 83.3.2 主要步骤 83.4 会议评审 83.4.1 概述 83.4.2 主要步骤 83.4.3 评审通过标准 93.5 缺陷追踪 103.5.1 概述 103.5.2 主要步骤 10第4章 非正式评审流程 11第5章 选择合适的评审方法 125.1 选择原则 125.2 针对不同目标建议的方法 125.3 几种主要评审的参加人员 12 文档修订摘要：版本作者

3、说明第1章 引言1.1 文档用途本文档为评审活动的指导文档。主要用来说明评审的方法和流程。它为一个组织按照正确的方法进行评审活动提供了指导。1.2 阅读对象 项目组全体 EPG组成员 高级经理1.3 名词术语1.3.1 同行评审Peer Review，在工作产品的开发者中对工作产品进行评审，用以识别并消除缺陷。第2章 评审定义2.1 评审的目的评审的目的是保证所选择的工作产品能满足确定的需求，同时发现其中可能存在的缺陷。2.2 评审分类按照评审的不同目的，可以进行如下分类：评审类型目的举例教育评审让其他项目相关人员来督促与项目相关的技术论题。管理、预备、或准入评审向高层经理提供信息，以帮助他们

4、做出如下决策：发布产品、继续（或取消）开发项目、批准（或拒绝）提案、改变项目范围、调整资源、改变承诺。项目计划书的评审同行评审寻找工作产品中的缺陷和改进契机产品总体方案评审Code Review状态评审向项目经理和其他项目成员提供最新的项目状态信息，包括里程碑的进度、所遇的问题、识别的或受控的风险。项目状态报告（里程碑）的评审项目总结评审对最近完成的项目或阶段进行评审，让未来的项目吸取经验。项目总结报告评审上述的同行评审是应用比较广泛的一种方法，包括审查、小组评审、走查、结对编程、桌查、同行桌查和轮查。下面对同行评审的这几种主要类型进行较详细介绍。2.2.1 审查（Inspection） 是一

5、种正式、严格的同行评审发生，遵循同行评审的流程 作者不可以作为评审的主持人，需由指定的主席担当 审查要求每次宣读或描述产品的一小部分，然后大家提出评审意见 需要使用标准的评审检查单，并收集评审数据2.2.2 小组评审（ Team Review ） 是一种“轻型的审查”。是有计划的、结构化的。但没有审查正式和严格 作者可以作为小组评审的主持人（审查不允许） 小组评审是由主持人询问评审者这部分是否有问题（审查要求每次宣读或描述产品的一小部分） 小组评审方法每小时发现的错误只有审查的2/3 小组评审适用于不需要严格审查过程的工作产品。由于没有审查正式，小组评审可以节省出一些会议时间来讨论解决问题的思

6、路，并对技术方法达成共识2.2.3 走查（ Walkthrough ） 是一种非正式的评审。由产品的作者将该产品向一组同事介绍，并希望他们给出问题反馈 走查之所以不正式，是因为它通常不按照事先预定的过程进行，不制订详细的准出条件，但为了度量，需要记录相关的走查数据。 走查中，产品作者起主导作用，而其他评审角色则通常没有定义 走查的目的是为了发现代码中隐藏比较深的逻辑方面的错误，当测试无法定位问题时，或出现不稳定等问题时，可以对主要模块进行走查。 审查每检查1000行代码所发现的错误比走查多50% 走查也适用于当评审的首要目的是使别人了解产品时： 设计方案 测试文档 用户界面设计2.2.4 结对

7、编程（ Pair Programming ） 属于一种流行的软件开发“敏捷方法” (agile methodology)，又称为极限编程(extreme programming) 在结对编程过程中，两个开发者在一个工作站上同时操作同一个程序。这种方法有利于交流并且允许每个人的观点进行持续的、非正式的评审 两位小组成员都非常熟悉每一段代码，可减少因人事变动所带来的知识流失。由于搭档的实时评审，结对者可以迅速纠正错误 结对编程开发的软件相比个人开发的软件而言，其质量更加卓越，而且开发时间更少 因其无结构性，属于非正式的评审方法，但更象一种软件开发方法 在大多数情况下，它不能简单替换传统的同行评审2

8、.2.5 桌查（Deskcheck） 在程序编写的初期，程序员常在两次编译之间仔细检查源代码以保证程序的正常进行 是PSP的一部分 是一种自评审，不属于同行评审2.2.6 同行桌查（Peer Deskcheck） 除作者外只有一个人对工作产品进行检查 完全依靠评审者本身的知识、技能和自律 是最便宜的评审方法，特别是有技术高超的同事 同行桌查所发现的错误只是评审者最擅长寻找的 作者本人既不能回答评审者的问题，也听不到那些有助于发现其他错误的讨论 是开始形成评审文化的好方法2.2.7 轮查（Passaround） 又成为分配审查方法，由多人组成的并行同行桌查 作者将产品副本发给几位评审员并收集整理

9、他们的意见 可以将待评审的产品放入共享文件夹，评审者在规定时间内以注释方式提出建议，作者负责汇总和过滤意见，并对产品进行修改 允许每位评审者看到别人的意见，可减少冗余 有助于缓解同行桌查的两个主要风险 评审者不能及时反馈 评审效果差第3章 正式评审流程同行评审中的审查和小组评审均属正式评审，这里主要介绍正式评审的流程。其他类型的评审流程可以参考正式评审的流程进行理解。3.1 角色和职责在正式评审活动中，涉及的角色有评审发起人、作者、评审委员会/评委、评审委员会主席、PPQA、记录人员。他们的主要职责如下：角色职责评审发起人评审发起人负责评审文档发放，会议通知，确定评审委员会主席。作者待评审工作

10、成果的开发者，可能是一个人或是小组。在评审期间，作者答复评审小组的问题，并与评审小组共同查找缺陷、商讨缺陷解决方案。评审会议结束后，作者应当及时消除工作成果中的缺陷。评审委员会/评委阅读评审文档和资料，参加评审会，发表评审意见。评审委员会成员一般由如下角色组成： 作者（讲述被评审文档） 评审主席 有相关经验的经理、技术负责人、总工程师（尤其是重要技术类的评审）等 产品的直接受益人和要求者（如，需求的评审需要客户参加） 项目组相关人员（主要技术负责人、测试负责人、配置管理员等）评审委员会主席主持召开评审会议，控制评审会议进程，给出评审结论。QA检查评审流程是否符合要求；检查工作产品是否符合要求记

11、录人员负责会议记录，发送评审结果报告3.2 准备评审3.2.1 概述评审发起人提出评审要求，制定评审计划，通知相关人员，及其他相关准备工作。3.2.2 主要步骤1. 发起人制定同行评审计划，计划内容包括： 评审会主席、参加评审的人员名单、会议记录人员 评审内容及文档名称 评审地点 评审要求2. 同行评审计划只需包括以上内容即可，不强制要求使用固定模板。3. 对于不同评审人员，发起人可以给出不同的评审侧重方面的建议。涉及到多个部门和项目的评审时，评审组成员中应包括其他部门和项目的代表。4. 评审发起人参考各过程提供的评审检查表模板，裁剪评审检查表及评审标准。5. 发起人将检查表以及评审文件及其上

12、游输入文件分发给相关人员，让相关人员了解评审要求，阅读评审文档。6. 由QA审核被评审文档与过程的符合情况。3.3 个人审查3.3.1 概述个人审查是正式评审前必要的准备工作。评委各自仔细阅读评审文件，并将填好内容的评审检查表提交给评审主席。评审主席做简要的检查，以确定是否如期举行会议。3.3.2 主要步骤1. 评委仔细阅读评审文件，使用评审检查表以一致的评审准则审查工作产品，做好评审会议的准备工作。对工作产品有不明确的地方或者看起来存在缺陷的地方，评委应在评审检查表中记录缺陷或问题并加以必要的说明。2. 评委将填好内容的评审检查表提交给评审主席。3. 评审主席应对检查单中的工作量和缺陷数据做

13、简要的检查，核实评委在个别审查中确实投入了足够充分的时间和精力。4. 如果准备工作不充分，则应该推迟评审会议。3.4 会议评审3.4.1 概述会议评审过程主要按以下步骤进行：主持人宣讲、作者介绍工作成果、识别缺陷和答辩、对缺陷达成一致、会议结束决议。3.4.2 主要步骤1. 评审主席在所有评委都已提交符合要求的评审检查表后召开评审会议。由计划中明确的会议记录人对会议的过程予以记录。2. 在会议上，评审主席首先宣讲本次评审的议程、责任、原则、时间限制等，然后作者简要介绍工作产品。3. 作者回答评委的问题，采用逐项审议或者逐段审议的方式，确认个别审查中所发现的缺陷和问题。双方对每个缺陷达成共识。4

14、. 在会议结束之前，由记录人说明在会议中发现和确认了哪些缺陷和问题，还有哪些缺陷和问题需要进一步讨论和审查，由评审主席进行总结，给出评审是否通过的判定。5. 会后，会议记录人完整填写AIQCS系统中的各项信息（问题、工时等），形成评审结果报告，发送给与会者。评审报告中的问题类型分为如下几种： 严重：说明错误、引起系统功能严重错误的 一般：不会严重影响功能但由于会引起异议或含糊不清也必须进行修改的； 改进：不影响功能描述，但希望进一步完善能得到更好效果的； 新问题：不是评审对象（文档）本身的问题，但和评审对象有关； 需要澄清：需要作者在会后查询资料后给大家一个澄清的；6. 对于必须参加评审会议却

15、因为特殊原因未能到会的评审参与人可以在评审报告正式发布之前以邮件的方式表明自己对评审的意见，并抄送给所有参加评审的人员。其邮件确认也同到会者意见一致看待，与评审报告同时存档。3.4.3 评审通过标准评审结论有三种：【】完全通过，进入下一阶段【】基本通过，需做修改【】不通过，需处理评审出的问题，再评审评审是否通过由评审主席根据如下规则进行判断：评审结论判断标准完全通过，进入下一阶段 没有严重问题，所有疑问均已澄清，或 没有一般问题和建议，对内容的正确性得到评委们认可基本通过，需做修改。（不需要再进行正式评审，通过轮查的方式进行后续评审） 每个工作产品（文档）中的缺陷3个严重问题，或 解决方案已讨

16、论清楚，作者已明确如何修改即可满足要求不通过，需修改评审对象（文档），再评审 每个工作产品（文档）中的缺陷3个严重问题，或 工作产品内容要发生较大变化，评委们无法判定作者后续的修改能否满足要求注：在实际应用中，评审主席可以根据实际情况或评审对象的特殊性，不完全采纳上述判断标准而采取例外放行的方法。需要在评审结论中给予说明。3.5 缺陷追踪3.5.1 概述经过同行评审，识别工作产品的缺陷后，项目人员要进行缺陷的修改。项目经理进行缺陷的追踪，以保证所识别的产品缺陷已全部得到解决。3.5.2 主要步骤1. 评审主席负责对评审会上发现的缺陷是否修改进行跟踪。2. 缺陷由作者解决，而问题则可以由不同人员

17、解决。在会议结束之前明确问题负责人及问题解决的最后期限，以便跟踪所存在的问题解决情况。3. 评审会上确认的责任人应修改识别出的所有缺陷，项目经理负责跟踪缺陷修改情况，PPQA负责验证缺陷是否被修改。必要时重新召开评审会议，重新评审经过修改的工作产品，或讨论对待解决问题的研究分析结果（工作方式与评审会议相同）。第4章 非正式评审流程对非正式评审不需要召集评审会。作者将待审批的产品提交给相关的评审人员进行审批，评审人员对提交审批的产品必须在指定的时间范围内进行评审、反馈意见。在指定时间内没有批准或否决的产品将被认为是批准了。如果产品没被批准，应将问题反馈给产品提交者。如果问题解决了，产品应重新提交

18、审批。重新提交的产品只有在对原有问题的解决不令人满意的情况下，才能被再次否决，从而排除无休止的审批循环，否则可能会严重影响项目的进度情况。第5章 选择合适的评审方法5.1 选择原则 组织应该选择最便宜的评审方法来将产品的风险降低到可接受的程度 基本原则：对高风险的工作产品使用正式评审，对低风险的构件使用相对便宜的技术 在需求开发阶段，先使用一系列的轮查，使参与评审的客户以非正式的方式评审越来越大的需求规格说明 将由多个分析员编写的各部分规格说明汇总后，再采用正式评审，并再一次邀请关键用户参与评审 在何时使用何种评审方法，最佳途径就是在组织中记录下评审的有效性和效率数据5.2 针对不同目标建议的

19、方法评审活动审查小组评审走查结对编程同行桌查轮查项目级活动有关项目计划及子计划的评审有关需求规格说明书的评审有关总体设计规格说明书的评审测试Case评审有关产品缺陷的评审Code Review有关发布的评审培训其他组员以熟悉产品组织级活动组织级各种计划的评审各种结果报告的评审组织PCB方案和报告5.3 几种主要评审的参加人员评审项目经理测试经理高级经理需求分析总工程师设计工程师开发工程师测试工程师QA配置管理员客户记录员市场部售前项目计划评审VVVVVVVVV测试计划评审VVVVV配置计划评审VVV需求规划说明评审VVVVVVVVVV总体设计方案评审VVVVVVVVV详细设计方案评审VVVVVCode ReviewV发布评审VVVVVVVV