软件技术评审.ppt

,单击此处编辑母版标题样式,单击此处编辑母版文本样式,第二级,第三级,第四级,第五级,*,软件技术评审,BYPMT05,V1.00,Confidentiality level,密级,:,（,内部公开,）,部门,:,北研质量部日期,:2005/06/01,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,小游戏,:,找一找下面的图片有什么不同,不要交流,;,限时,:1,分钟,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,找到多少个不同点,?,!,时间到,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,不同点,:,共有,10,处,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,课程目的,了解业界软件技术评审方法,掌握我司软件技术评审流程,学习我司软件技术评审工具,分享多年的实际应用开发经验,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,课程主要内容,软件技术评审方法,IPD-CMM,软件评审流程,我司常用软件技术评审工具,常见的软件技术评审的误区,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审,软件技术评审方法,IPD-CMM,软件技术评审流程,华为常用软件技术评审工具,常见的软件技术评审的误区,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,软件技术评审的定义,借助他人对软件交付物进行检查,发现缺陷或获得改进优化的机会,强有力的软件质量保证活动之一,CMM,中定义为,Peer Review,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,软件技术评审的意义,降低返工（,rework,）的成本,业界的一些数据：,相对于通过,review,发现缺陷的,rework,成本，测试发现缺陷的,rework,成本是其,14.5,倍,而客户发现缺陷的,rework,成本是其,68,倍一家德国软件公司,产品发布后发现缺陷的,rework,成本是在设计阶段的,45,倍,Earlier we find and correct defects,More money and time we will save,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,常见的软件技术评审形式,Inspection,Walkthrough,4 Eyes Review,最正式的,最不正式的,Inspection,4 Eyes Review,Walkthrough,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,Inspection,的基本特点,定义质量目标,由经过培训的团队参加,由经过培训的组织者组织（作者不能是组织者）,明确参加人员角色和职责,有正式的文档规程引导,向管理者上报,review,结果,清晰的入口和出口条件,对缺陷进行跟踪直至关闭,记录过程和质量数据，并根据数据进行,review,流程和软件开发过程进行改进,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,Walkthrough,的基本特点,没有明确的质量目标,有正式的会议,作者就是组织者,作者就软件交付物对评审人进行讲解,没有正式的文档规程引导,不必向管理者上报,review,结果,很少有正式的记录过程和质量数据，无法根据数据进行,review,流程和软件开发过程进行改进,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,4 Eyes Review,的基本特点,临时的、事件驱动的,仅仅是为了找出一个,bug,更多的是团队协助,为了作者发现问题提供了另外一种角度,最不正式的,review,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,3,种,review,形式的比较,1,评审类型 评审的活动,计划,准备,会议,纠正,验证,Inspection,Walkthrough,4 Eyes Review,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,3,种,review,形式的比较,2,要素,Inspection,Walkthrough,4 Eyes Review,组织者,组织者,作者,无,讲解员,专门的讲解员,作者,无,粒度,small chunks,由作者自己判断,一段代码、一段文字,使用记录员,YES,Maybe,无,遵循规程,YES,Maybe,无,定义参加人员角色,YES,NO,NO,使用,checklist,YES,NO,NO,数据收集和分析,YES,NO,NO,产品评价决策,YES,NO,NO,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,选择合适的,review,形式,基于风险考虑：交付物存在缺陷的可能性以及如果存在缺陷的影响,用最正式的形式去,review,高风险的交付物,用最经济的形式去,review,低风险的交付物,用收集数据来判断对某一工作产品，哪种,review,形式最有效,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,哪些因素会因增加风险指数而需要特别关注,复杂的逻辑或算法而其必须精确和最优,过度的开发进度压力,开发人员技能缺少足够培训或经验,产品的关键、基础模块,可能被重用的模块或作为模板的模块,有众多接口的模块,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,度量分析,缺陷预防,及时对问题进行分析和总结,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,更重要的是,形成一种文化拥抱评审，把它作为改进软件质量、提高生产率以及加快个人学习进度的强有力的方法,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,IPD-CMM,软件技术评审流程,华为常用软件技术评审工具,常见的软件技术评审的误区,软件技术评审,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,IPD-CMM,软件技术评审流程,角色,项目经理,作者,讲解员,REVIEW,人员,记录员,组织者,可兼任,不可兼任,各司其职,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,IPD-CMM,软件技术评审流程,步骤,介绍会议？,1.,计划,阶段,3.,准备,阶段,2.,介绍,会议,Y,N,7.,跟踪,阶段,6.,返工,阶段,Y,N,第三小时会议？,4.Review,会议,5.,第三小时会议,入口,准则,出口,准则,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,计划阶段,IPD-CMM,软件技术评审流程,代码：,=500,行,(NBNC),文档：,IPD-CMM,-,Re,view,QAM01F01,Review Note Review,通知单,QAM01F03,PreReview Form,预审表单,QAM01F04C,Review Summary Form Review,汇总表单,QAM01C01,Review Checklist Review,查检表,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,华为常用软件技术评审工具,Code Review Tool,Document Review Tool,四象限分析法,bjpal,-,工具数据库,-,开发工具,-,Review Tool,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,软件技术评审方法,IPD-CMM,软件技术评审流程,华为常用软件技术评审工具,常见的软件技术评审的误区,软件技术评审,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区一：参与者不了解,review,流程,症状,:,参与者对自己在,review,中的角色、职责以及如何进行,review,中的活动，各有各的理解；项目组成员对自己的产品哪些应该,review,、什么时候进行,review,、用什么样的,review,方法根本不清楚；一次,review,上，,review,太多的资料，结果却是两手空空,.,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区一：参与者不了解,review,流程,措施,:,培训！培训将最好的团队建设活动，培训将能确保每个成员都能准确理解,review,的过程。一般而言,4-8,个小时的培训基本足够了；但对于正式,review,的组织者而言，可能要更多一些。培训内容的准则是遵循公司所制定的,review,规程。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区二：对人不对事,症状,:,Review,变成了一场人身攻击会、批斗会,.,挖苦作者的风格、嘲笑作者的技能。当作者感受到自己受到攻击，他（她）会开始对抗：拒绝配合；以后不愿意提交产品,review,；甚至寻求报复的机会。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区二：对人不对事,措施,:,在进行,review,前，要强调,review,是发现产品的问题而不是对作者进行批斗。,review,不是给,review,专家一个机会展示自己比其他人有多聪明，而是要通过集体的智慧、观察力、经验来发现产品的缺陷，提高产品的质量。当你发现问题的时候，这样表达：“我没有发现这个地方的变量被初始化”也许比“你怎么忘记了把这个变量初始化了”要更好一些。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区三：没有计划,症状,:,对于许多项目而言，,review,从来不会出现在项目,WBS,或进度安排中；即使项目计划上有，也仅仅是以里程碑的形式出现而不是任务；里程碑是一个以“,zero-time”,定义的。没有计划，导致当需要,review,专家参与的时候，,review,专家已经没有时间了，他已经安排其他事情了,!,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区三：没有计划,措施,:,做好计划，以任务的形式将,review,体现到项目计划中。要估计出,review,专家个人准备的时间、,review,会议的时间、,rework,的时间。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区四：,Review,会议演变成问题解决会,症状,:,开发人员天性都喜好解决问题，喜欢提出对问题的优雅的解决方案。但这并不是,review,会议所想要的。,Review,会议的目的是发现缺陷、确认问题。如果会议演变成问题解决会议，对产品的检验就嘎然而止；争论变得无休止；部分参与者变的无所适从。当意识到更应该去发现缺陷的时候，时间已经所剩无几,.,于是匆匆结束,review,，很多缺陷却还在“花丛中笑”,.,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区四：,Review,会议演变成问题解决会,措施,:,开正式的,Review,会议的目的是对专家反馈的意见进行确认并试图发现更多的问题。正式的,review,会议之所以比非正式的更有效，是因为它有组织者；当,review,进展偏离的正常的轨道，组织者有责任及时把会议纠正到正常秩序来。一般而言，对问题的讨论是不可避免的，如果这个问题能在,1,2,分钟等讨论清楚，那么就讨论它；反之，还是把它放在会议之外解决吧。当然，可能会有这样的情况，某个缺陷很严重，严重到不把这个问题解决了，后面的,review,就无任何意义，那么就把它改成问题解决会议。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区五：,Review,专家还没有准备,症状,:,你早上,8,：,45,来到工位上打开,Notes,，收到一个加了,!,的邮件，通知你早上,9,：,00,到,D304,参加,XXXXreview,会议，并附件了,10M,的资料。你只有,15,分钟时间，你能,review,出问题来吗？你不了解项目的背景，不了项目的约束假设，不了解项目的关联，你即使参加了会议，就能发现问题了吗？,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区五：,Review,专家还没有准备,措施,:,75,的缺陷都是在专家个人准备的时候发现的，这就是为什么组织者在开会前需要收集专家个人准备,review,的时间，以确定会议时候能进行。所以项目经理要做好项目计划，提前安排好,review,的个人准备时间。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区六：让不合适的人参加,症状,:,如果让一个对产品、项目还不熟悉、技能还比较欠缺的人参与,review,，也许唯一收获就他通过,review,了解了产品，但对工作产品质量的提高却没有任何贡献。同样，高级经理的参与，也有可能导致比较差的,review,效果，因为如果作者感觉到经理仅仅是在数缺陷个数以对他进行评价的时候，作者就会变的犹豫，也有可能破坏同事之间的气氛。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区六：让不合适的人参加,措施,:,一个,review,一般,3,7,人效率是最高的。,review,的专家应该包括工作产品的上下游的人。如：设计文档的,review,，就应该包括作者本人，需求文档的作者、编码的责任人、完成测试的责任人，也应该包括对该领域很熟悉的人。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区七：过多关注风格，忽略内容,症状,:,如果当你浏览,review,发现的缺陷列表，发现大部分缺陷都是风格上的，那么你应该很紧张了：也许很多重要的、逻辑上、算法上的问题都被忽略了。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区七：过多关注风格，忽略内容,措施,:,风格可以是缺陷，但不应该成为,review,发现缺陷的主体。作者要在写作之前，即可能接受相关培训，要尽量使用提供的模板；作者写完之后，要即可进行自检。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,常见的软件技术评审的误区,走出误区,走出自我,对事不对人,仅仅去发现问题，而不要试图去修改,控制会议时间（,=2Hr.,）,避免纠缠风格问题，除非它真的影响你,尽早的、经常的进行正式或非正式的,review,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,总结,Wow!,Peer Review,definitely,is one of the most powerful software quality tools available.,I would never work in a team that did not perform them.,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,Questions?,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,写在最后,在实际工作中，大家可以不断积累经验，指导自己的项目进行高效,Review,。运用之妙存乎一心,小诗一首：,你是风儿我是沙，,你是蜜蜂我是花。,你是梳子我是头发，,你是牙膏我是牙刷。,Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散,谢谢,