Review 管理流程

Review管理流程-------质量管理系列培训一目录同行评审过程概述同行评审的目的、对象和意义角色与职责核心代码评审与普通代码评审的区别核心代码评审普通代码评审文档评审与代码评审的区别文档评审《评审记录单》的使用评审问题状态说明同行评审过程概述代码评审核心代码评审普通代码评审文档评审桌面文档评审分发式文档评审同行评审的目的、对象和意义原因尽管人们善于找出自己所犯的部分错误，但是相对于其他任何人而言，大量的各种错误还是很容易逃脱创作者的眼睛，这被称为思维定势。目的同行评审的目的,是在软件项目初期发现那些潜在的缺陷或错误,避免这些错误和缺陷遗漏到项目的后续阶段。同行评审的对象开发过程中的所有工作产品，包括代码、文档等同行评审的作用通过消除工作成果的缺陷而提高产品的质量；越早消除缺陷就越能降低后续开发修改缺陷的人力和时间成本；开发人员能及时地得到同行专家的帮助和指导，无疑会加深对工作成果的理解，更好地预防缺陷。许多缺陷是在早期阶段引入的，发现的越晚，纠正的成本越高：缺陷数量的放大的原因是，每个进入下个步骤的缺陷都可能引起下个步骤中的多个缺陷，导致消初缺成本的剧增。角色与职责作者：在SQA处申请评审id号，通知相关评审人员评审时间地点，按关闭计划修改评审出的问题。对于普通代码的评审，作者自己制定评审问题关闭计划。项目经理：在项目立项的时候确定此项目是否需要进行核心代码评审。协助分析评审中发现的问题，对于核心代码评审，制定关闭计划，对改正的问题进行验证关闭。评审委员会代表（reviewcommitteerepresentative----RCR）：由4~5名技术能力较强的人组成评审委员会，项目经理可以同时是评审委员会成员（但不能担任自己负责项目的评审委员会代表），负责选出必须申请评审的代码范围，指定其他参与评审的成员。参加评审会议，协助评审成员指出评审对象中的问题。评审成员：参加评审会议，提出评审对象中的问题。记录员：记录员可以为作者或其他评审人员，负责记录评审出的问题，此过程输出《评审记录单》（电子版）。记录员负责将《评审记录单》发送给参与评审所有人员。SQA：对于有核心代码评审要求的项目根据开发计划，向项目经理了解开发进度，在开发进展到一定阶段时提醒RCR人员挑出需要评审的核心代码。为申请的review提供reviewID。检查评审过程及《评审记录单》，保证评审按照流程进行，评审达到应有的效果，跟踪评审出的问题的解决情况，保证所有评审问题及时关闭。对《评审记录单》备份整理。核心代码评审与普通代码评审的区别1.参与角色不同：核心代码评审需要评审委员会代表（RCR），项目经理必须参与，而普通代码评审不需要。2.评审驱动方式不同：核心代码由SQA驱动，RCR选择代码范围进入评审。普通代码评审为开发人员自己发起，对自己认为有必要评审的代码组织评审。3.评审过程不同：流程图中详细介绍。4.评审问题的关闭者不同：核心代码评审为项目经理验证关闭评审问题，普通评审为问题提出者验证关闭。注意：与评审过程相关的所有邮件抄送team组相关人员和SQA。保证评审期间team组人员良好沟通。开发工程师可以根据代码改动量的大小自己控制同行评审进行的地点，可以在会议室，也可以选择工位或邮件形式进行，针对改动很小评审也可以通过邮件进行。工位和会议室进行的评审需要有评审记录，输出《评审记录单》。邮件进行代码评审的不需要输出《评审记录单》，且不需要SQA跟踪。核心代码评审流程图核心代码评审普通代码评审流程图普通代码评审文档评审与代码评审的区别不需要评审委员会代表（RCR）参与。评审前无单元测试报告输出有评审要求的文档必须经过评审才可以归档邮件，工位，会议室所有形式的评审都要输出《评审记录单》。通过邮件进行的文档的评审SQA也需要跟踪。文档评审的问题由SQA验证关闭。文档评审流程图文档评审评审文档与参与角色对照表文档编写checklist评审记录单的使用UMPAY_SW_图表_评审记录单_项目_peixun.xls表单使用目的：1.评审记录可以为项目度量依据。为今后计算评审问题发现效率提供数据支持。2.保证评审出的问题不被遗漏，并且全部在受控下改正。3.避免遗忘暂时解决不了的问题（挂起问题）。评审问题状态说明Opened当问题被提交时状态为Opened；Closed确认问题已经修改正确经过验证关闭后，将问题状态置成Closed；挂起不能立刻解决的问题，评审记录单此问题置“挂起”状态，SQA负责定期回顾跟踪。