技术(同行)评审 PPT

技术(同行)评审课程目标•理解同行评审；•懂得如何开展同行评审；技术评审(Inspection)的历史•1972M.Fagen提出并在IBM推行；•结果观察:1、在进行任何测试之前发现的错误的百分比1972-7650-65%200175-85%，特别的达到90%以上2、缺陷植入率减少50%3、顾客报告的缺陷减少15X-20X倍4、生产率提高2X倍以上同行评审的目的•同行评审的目的是为了及早地和高效率地从软件工作产品中清除缺陷。一个重要的伴随结果是对软件工作产品及可防止的缺陷得到更好的了解。同行评审的好处•中高层：可以及时掌握项目的进展•项目组：-项目组可以利用组织中最有才干的人，即使他们没有参与项目组也能发挥作用。-令项目成员有一种成就感、参与感和得到认可的感觉，从而保持团队的积极性。-团队成员可以发展他们的技能，而资深成员可以指导那些缺乏经验的人员。-通过评审更加留意缺陷，从而帮助预防缺陷。经验•80％的缺陷，通过同行评审发现。•16％的缺陷，通过测试发现。•4％的缺陷，通过客户发现。8016401020304050607080百分比评审发现测试发现客户发现哲理•审查是过程成功进展的基础评审的意义需求|设计|编码|文档|测试|维护需求|设计|编码|文档|测试|维护如果没有评审发现----原点----混乱区域评审的意义需求|设计|编码|文档|测试|维护需求|设计|编码|文档|测试|维护如果有评审发现----原点----同行评审•小组评审（正规的同行评审）•四眼评审流程申请评审评审计划跟踪返工小组评审会议评审准备启动会议结束联系协调者；确认入口准则；选择审查者；制定评审计划；设计评审视角；准备评审包；设置测量目标；简短（5－30分钟）；沟通评审事宜；评审文档；各自记录条款；最好提前一天；记录发现条款；协调者推动会议；会议时间很重要；整理条款；修改目标文档；协调验证返工；关闭所有条款；职责审查者:尽可能发现并报告条款拥有者:拥有被审查的文档.理解每个条款.协调者:管理过程.为审查收集资料,安排会议以及调整会议流程.记录者:会议中公开记录所有条款.申请评审申请评审拥有者协调者池协调者需要经过同行评审培训；具备推动同行评审整个过程的能力。协调者申请评审评审计划－1评审计划评审计划拥有者审查者池评审计划协调者WPUR支持文档评审包评审计划－2评审计划入口检查拥有者WPUR评审计划协调者选择审查者设计评审视角组织评审包计划评审过程支持文档审查者池评审包评审包数据设计?接口?符合说明?标准?目标文档/代码支持文档指南、标准信封需求工具目标文档启动会议启动会议沟通会议协调者需要让所有的人明白如何开展工作。如何使用评审包；如何记录条款；何时参加会议；解答审查者的其它问题；评审准备评审准备评审准备拥有者；协调者；审查者；评审包评审记录1、记录位置；2、描述条款；3、条款分类；4、引入阶段；注意：要记录评审准备工时！小组评审会议记录会议小组评审会议拥有者；协调者；审查者；评审包评审记录1、阅读目标文档；2、报告发现条款；3、理解发现条款；小组评审会议的基本原则协调者确认所有审查者已准备好；限定讨论，确保拥有者理解问题.焦点在问题上,不在解决方案上.简明并且有建设性,不要忽视任何人所说的.为会议的成功负责.每个参与者：小组评审会议－语言文化可行:只在表扬的时候用“你.如果可能参考其它资料.用“条款”进行描述代替“问题”.用建设性的语调.例子:“我对第20行感到困惑.“你干的好.“基于我们的执行指导方针,这个模块有个执行上的问题“基于我们的模版,缺少了花费/价值信息.“15行与第4页矛盾不可行:轻视拥有者.不让拥有者辩护自己的工作。批评,讥笑,或拒绝任何问题例子:“你应该...“这本应该更好如果换成...“你可以这样修改它...“为什么你这样做?那不是错误!X√小组评审会议－原因分析•如果需要，可以在记录会议之后立即开展原因分析会议。•是否召开原因分析会议由协调者协商决定。•可以使用脑筋风暴法；•因果分析图；返工返工返工拥有者；协调者；评审记录工作产品1、整理条款；2、标识重复条款；3、记录条款状态；跟踪跟踪跟踪拥有者；协调者；评审记录工作产品1、由协调者关闭条款；结束•所有条款都关闭！•拥有者填写同行评审记录表－统计数据。结束非正规的同行评审－四眼评审•只有两个人的评审；•拥有者和审查者；四眼评审评审申请评审返工发布WPUR1、拥有者和审查者一同评审；2、使用支持文档；3、审查者记录条款；4、审查过程不讨论解决方案；1、一同整理条款；2、如果拥有者要求，可以讨论解决方案。3、拥有者根据条款返工；评审过程仍然使用同行评审记录表。注意：记录工时！评审结果度量与分析项目名称项目编号评审的工作产品名称XXX产品需求规格说明书工作产品规模(A4页)15作者项目经理一、会前审查度量评审人员工作单位（部门）、职务、职称评审工作量(小时)评审速度(页/小时)发现缺陷数发现问题数XXX研发中心COS1部,高级工程师2.07.552YYYZZZ合计:2.07.552评审结果度量与分析-cont.二、小组会议评审度量参见小组会议评审人员名单XXX,YYY,ZZZ,…参见小组会议评审人员数量5会议时长(小时)2.0工作量(人时)10缺陷与问题数量非常严重严重一般问题缺陷总数235310度量项定义结果能力基准文档评审的评审速度文档页数/（评审人数*评审会议时长）1.53-5页/小时文档评审的评审效率缺陷数/（评审人数*评审会议时长）1.0文档评审的缺陷密度缺陷数/评审文档页数0.70.5-1.5/页本次技术评审总的工作量(人时)12.0工作量(评审)高低高低缺陷率Scenario2:GOOD/NOTBAD好／不坏Scenario１:BESTCASE最好情况Scenario３:UNSURE不确定Scenario４:WORSTCASE最差情况评审结果度量与分析-cont.评审是否有效,首先判断查出的总缺陷数是否在规定极限内(如0.5-1.5个/页)。如果不在极限内，应结合工作量/结果模型做进一步分析评审结果度量与分析-cont.最好情况－高工作量/低缺陷率：文档／编码在审核之前就很好，团队还花了足够的精力用于ＤＲ／ＣＩ（文档评审／代码审查），从而确保了质量。评审负责人需要判断评审人员是否真的按流程/准则进行。最差情况－低工作量/高缺陷率：高的错误植入，但审查不够严格。到结束审查过程时，可能有更多的缺陷遗留在文档和编码中。需要强调预评审，增加审工作量并改进评审精确性。好/不坏的情况－高工作量/高缺陷率：错误植入率可能很高，但花了较多的精力是一个好的表示，也许是有更多缺陷被清除的原因。需要强调缺陷预防（DP）。不确定的情景－低工作量/低缺陷率：不能确定文档和编码是否更好，因此审查需要的时间较少或审查完成比较匆忙，既而发现的缺陷较少。在这种情况下，我们需要依赖团队的主观评价和其他信息，从而做出较好的决定。领悟同行评审•什么是同行？•同行评审的价值在那里？•如何才能做好同行评审？如何才能做好同行评审•转换思想-帮助拥有者发现问题；•不是挑拥有者的刺；•所有人转换思想，不光是评审组，包括项目经理、高层经理；•同行评审发现的问题不做为考核的数据，只做为分析、改进的数据。问题•项目时间很紧，是否可以进行同行评审？