《软件技术评审》PPT课件.ppt

《《软件技术评审》PPT课件.ppt》由会员分享，可在线阅读，更多相关《《软件技术评审》PPT课件.ppt（64页珍藏版）》请在淘文阁 - 分享文档赚钱的网站上搜索。

1、软件技术评审软件技术评审BYPMT05 V1.00Confidentiality level 密级:（内部公开）部门:北研质量部日期:2005/06/012Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散小游戏小游戏:找一找下面的图片有什么不同找一找下面的图片有什么不同不要交流;限时:1分钟3Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散找到多少个不同点找到多少个不同点?

2、!时间到时间到4Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散不同点不同点:共有共有10处处5Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散课程目的课程目的n了解业界软件技术评审方法了解业界软件技术评审方法n掌握我司软件技术评审流程掌握我司软件技术评审流程n学习我司软件技术评审工具学习我司软件技术评审工具n分享多年的实际应用开发经验分享多年的实际应用开发经验6Confi

3、dential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散课程主要内容课程主要内容n软件技术评审方法软件技术评审方法nIPD-CMM软件评审流程软件评审流程n我司常用软件技术评审工具我司常用软件技术评审工具n常见的软件技术评审的误区常见的软件技术评审的误区7Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审软件技术评审n软件技术评审方法软件技术评审方法nIPD-CMM软件技术评审流程

4、软件技术评审流程n华为常用软件技术评审工具华为常用软件技术评审工具n常见的软件技术评审的误区常见的软件技术评审的误区8Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审方法软件技术评审方法n软件技术评审的定义借助他人对软件交付物进行检查发现缺陷或获得改进优化的机会强有力的软件质量保证活动之一CMM中定义为Peer Review9Confidential Information of Huawei,No Spreading without Permission.华为机密，未经

5、许可不得扩散软件技术评审方法软件技术评审方法n软件技术评审的意义降低返工（rework）的成本业界的一些数据：相对于通过review发现缺陷的rework成本，测试发现缺陷的rework成本是其14.5倍,而客户发现缺陷的rework成本是其68倍一家德国软件公司产品发布后发现缺陷的rework成本是在设计阶段的45倍Earlier we find and correct defects,More money and time we will save 10Confidential Information of Huawei,No Spreading without Permission.华为

6、机密，未经许可不得扩散软件技术评审方法软件技术评审方法n常见的软件技术评审形式InspectionWalkthrough4 Eyes Review最正式的最不正式的Inspection4 Eyes ReviewWalkthrough11Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审方法软件技术评审方法nInspection的基本特点定义质量目标由经过培训的团队参加由经过培训的组织者组织（作者不能是组织者）明确参加人员角色和职责有正式的文档规程引导向管理者上报review

7、结果清晰的入口和出口条件对缺陷进行跟踪直至关闭记录过程和质量数据，并根据数据进行review流程和软件开发过程进行改进12Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审方法软件技术评审方法nWalkthrough的基本特点没有明确的质量目标有正式的会议作者就是组织者作者就软件交付物对评审人进行讲解没有正式的文档规程引导不必向管理者上报review结果很少有正式的记录过程和质量数据，无法根据数据进行review流程和软件开发过程进行改进13Confidential Inf

8、ormation of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审方法软件技术评审方法n4 Eyes Review的基本特点临时的、事件驱动的仅仅是为了找出一个bug更多的是团队协助为了作者发现问题提供了另外一种角度最不正式的review14Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审方法软件技术评审方法n3种review形式的比较1评审类型 评审的活动计划准备会议纠正验证InspectionW

9、alkthrough4 Eyes Review15Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审方法软件技术评审方法n3种种review形式的比较形式的比较2要素要素InspectionInspectionWalkthroughWalkthrough4 Eyes Review4 Eyes Review组织者组织者组织者组织者作者作者无无讲解员讲解员专门的讲解员专门的讲解员作者作者无无粒度粒度small chunkssmall chunks由作者自己判断由作者自己判断一段

10、代码、一段文字一段代码、一段文字使用记录员使用记录员YESYESMaybeMaybe无无遵循规程遵循规程YESYESMaybeMaybe无无定义参加人员角色定义参加人员角色YESYESNONONONO使用使用checklistchecklistYESYESNONONONO数据收集和分析数据收集和分析YESYESNONONONO产品评价决策产品评价决策YESYESNONONONO16Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审方法软件技术评审方法n选择合适的review

11、形式基于风险考虑：交付物存在缺陷的可能性以及如果存在缺陷的影响用最正式的形式去review高风险的交付物用最经济的形式去review低风险的交付物用收集数据来判断对某一工作产品，哪种review形式最有效17Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审方法软件技术评审方法n哪些因素会因增加风险指数而需要特别关注复杂的逻辑或算法而其必须精确和最优过度的开发进度压力开发人员技能缺少足够培训或经验产品的关键、基础模块可能被重用的模块或作为模板的模块有众多接口的模块18Con

12、fidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审方法软件技术评审方法n度量分析n缺陷预防19Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散软件技术评审方法软件技术评审方法n更重要的是l形成一种文化拥抱评审，把它作为改进软件质量、提高生产率以及加快个人学习进度的强有力的方法20Confidential Information of Huawei,No Spreadin

13、g without Permission.华为机密，未经许可不得扩散n软件技术评审方法软件技术评审方法nIPD-CMM软件技术评审流程软件技术评审流程n华为常用软件技术评审工具华为常用软件技术评审工具n常见的软件技术评审的误区常见的软件技术评审的误区软件技术评审软件技术评审21Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散IPD-CMM软件技术评审流程软件技术评审流程n角色角色项目经理作者作者讲解员讲解员REVIEWREVIEW人员人员记录员记录员组织者组织者可兼任可兼任不可兼任不

14、可兼任各司其职22Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散IPD-CMM软件技术评审流程n步骤步骤 介绍会议？介绍会议？1.1.计划计划阶段阶段3.3.准备准备阶段阶段2.2.介绍介绍会议会议YN7.7.跟踪跟踪阶段阶段6.6.返工返工阶段阶段YN第三小时会议？第三小时会议？4.Review4.Review会议会议5.5.第三小时第三小时会议会议 入口准则出口准则23Confidential Information of Huawei,No Spreading without

15、 Permission.华为机密，未经许可不得扩散n计划阶段计划阶段IPD-CMM软件技术评审流程l代码：代码：=500=500行行(NBNC)(NBNC)l文档：文档：=40-IPD-CMM IPD-CMM-ReviewlQAM01F01QAM01F01Review Note ReviewReview Note Review通知单通知单 lQAM01F03QAM01F03PreReview Form PreReview Form 预审表单预审表单lQAM01F04C Review Summary Form Review 汇总表单汇总表单 lQAM01C01QAM01C01Review Che

16、cklist ReviewReview Checklist Review查检表查检表 44Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散华为常用软件技术评审工具nCode Review ToolnDocument Review Tooln四象限分析法http:/bjpal http:/bjpal -工具数据库工具数据库 -开发工具开发工具-Review Tool45Confidential Information of Huawei,No Spreading without Per

17、mission.华为机密，未经许可不得扩散n软件技术评审方法软件技术评审方法nIPD-CMM软件技术评审流程软件技术评审流程n华为常用软件技术评审工具华为常用软件技术评审工具n常见的软件技术评审的误区常见的软件技术评审的误区软件技术评审软件技术评审46Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区一：参与者不了解review流程症状症状:参与者对自己在参与者对自己在review中的角色、职责中的角色、职责以及如何进行以及如何进行review中的活动，各

18、有各的理中的活动，各有各的理解；项目组成员对自己的产品哪些应该解；项目组成员对自己的产品哪些应该review、什么时候进行、什么时候进行review、用什么样的、用什么样的review方法根本不清楚；一次方法根本不清楚；一次review上，上，review太多的资料，结果却是两手空空太多的资料，结果却是两手空空.47Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区一：参与者不了解review流程措施措施:培训！培训将最好的团队建设活动，培训！培训将最好的

19、团队建设活动，培训将能确保每个成员都能准确理解培训将能确保每个成员都能准确理解review的过程。一般而言的过程。一般而言4-8个小时的培训基本足个小时的培训基本足够了；但对于正式够了；但对于正式review的组织者而言，可的组织者而言，可能要更多一些。培训内容的准则是遵循公司能要更多一些。培训内容的准则是遵循公司所制定的所制定的review规程。规程。48Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区二：对人不对事症状症状:Review变成了一场人身

20、攻击会、批斗会变成了一场人身攻击会、批斗会.挖苦作者的风格、嘲笑作者的技能。当作者挖苦作者的风格、嘲笑作者的技能。当作者感受到自己受到攻击，他（她）会开始对抗：感受到自己受到攻击，他（她）会开始对抗：拒绝配合；以后不愿意提交产品拒绝配合；以后不愿意提交产品review；甚；甚至寻求报复的机会。至寻求报复的机会。49Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区二：对人不对事措施措施:在进行在进行review前，要强调前，要强调review是发现是发现产

21、品的问题而不是对作者进行批斗。产品的问题而不是对作者进行批斗。review不是给不是给review专家一个机会展示自己比其他专家一个机会展示自己比其他人有多聪明，而是要通过集体的智慧、观察人有多聪明，而是要通过集体的智慧、观察力、经验来发现产品的缺陷，提高产品的质力、经验来发现产品的缺陷，提高产品的质量。当你发现问题的时候，这样表达：量。当你发现问题的时候，这样表达：“我我没有发现这个地方的变量被初始化没有发现这个地方的变量被初始化”也许比也许比“你怎么忘记了把这个变量初始化了你怎么忘记了把这个变量初始化了”要更要更好一些。好一些。50Confidential Information of H

22、uawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区三：没有计划症状症状:对于许多项目而言，对于许多项目而言，review从来不会出从来不会出现在项目现在项目WBS或进度安排中；即使项目计划或进度安排中；即使项目计划上有，也仅仅是以里程碑的形式出现而不是上有，也仅仅是以里程碑的形式出现而不是任务；里程碑是一个以任务；里程碑是一个以“zero-time”定义的。定义的。没有计划，导致当需要没有计划，导致当需要review专家参与的时专家参与的时候，候，review专家已经没有时间了，他已经安专家已经没有时间了，他已经

23、安排其他事情了排其他事情了!51Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区三：没有计划措施措施:做好计划，以任务的形式将做好计划，以任务的形式将review体现体现到项目计划中。要估计出到项目计划中。要估计出review专家个人准专家个人准备的时间、备的时间、review会议的时间、会议的时间、rework的时的时间。间。52Confidential Information of Huawei,No Spreading without Permis

24、sion.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区四：Review会议演变成问题解决会症状症状:开发人员天性都喜好解决问题，喜欢开发人员天性都喜好解决问题，喜欢提出对问题的优雅的解决方案。但这并不是提出对问题的优雅的解决方案。但这并不是review会议所想要的。会议所想要的。Review会议的目的是会议的目的是发现缺陷、确认问题。如果会议演变成问题发现缺陷、确认问题。如果会议演变成问题解决会议，对产品的检验就嘎然而止；争论解决会议，对产品的检验就嘎然而止；争论变得无休止；部分参与者变的无所适从。当变得无休止；部分参与者变的无所适从。当意识到更应该去发现缺陷的时候，时间已经意识到

25、更应该去发现缺陷的时候，时间已经所剩无几所剩无几.于是匆匆结束于是匆匆结束review，很多缺陷，很多缺陷却还在却还在“花丛中笑花丛中笑”.53Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区四：Review会议演变成问题解决会措施措施:开正式的开正式的Review会议的目的是对专家反馈的意会议的目的是对专家反馈的意见进行确认并试图发现更多的问题。正式的见进行确认并试图发现更多的问题。正式的review会议之所以比非正式的更有效，是因为它有组织者；会议之

26、所以比非正式的更有效，是因为它有组织者；当当review进展偏离的正常的轨道，组织者有责任及进展偏离的正常的轨道，组织者有责任及时把会议纠正到正常秩序来。一般而言，对问题的时把会议纠正到正常秩序来。一般而言，对问题的讨论是不可避免的，如果这个问题能在讨论是不可避免的，如果这个问题能在12分钟等分钟等讨论清楚，那么就讨论它；反之，还是把它放在会讨论清楚，那么就讨论它；反之，还是把它放在会议之外解决吧。当然，可能会有这样的情况，某个议之外解决吧。当然，可能会有这样的情况，某个缺陷很严重，严重到不把这个问题解决了，后面的缺陷很严重，严重到不把这个问题解决了，后面的review就无任何意义，那么就把它

27、改成问题解决会就无任何意义，那么就把它改成问题解决会议。议。54Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区五：Review专家还没有准备症状症状:你早上你早上8：45来到工位上打开来到工位上打开Notes，收，收到一个加了到一个加了!的邮件，通知你早上的邮件，通知你早上9：00到到D304参加参加XXXXreview会议，并附件了会议，并附件了10M的资料。你只有的资料。你只有15分钟时间，你能分钟时间，你能review出出问题来吗？你不了解项目的

28、背景，不了项目问题来吗？你不了解项目的背景，不了项目的约束假设，不了解项目的关联，你即使参的约束假设，不了解项目的关联，你即使参加了会议，就能发现问题了吗？加了会议，就能发现问题了吗？55Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区五：Review专家还没有准备措施措施:75的缺陷都是在专家个人准备的时的缺陷都是在专家个人准备的时候发现的，这就是为什么组织者在开会前需候发现的，这就是为什么组织者在开会前需要收集专家个人准备要收集专家个人准备revie

29、w的时间，以确定的时间，以确定会议时候能进行。所以项目经理要做好项目会议时候能进行。所以项目经理要做好项目计划，提前安排好计划，提前安排好review的个人准备时间。的个人准备时间。56Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区六：让不合适的人参加症状症状:如果让一个对产品、项目还不熟悉、如果让一个对产品、项目还不熟悉、技能还比较欠缺的人参与技能还比较欠缺的人参与review，也许唯一，也许唯一收获就他通过收获就他通过review了解了产品，但对工

30、作了解了产品，但对工作产品质量的提高却没有任何贡献。同样，高产品质量的提高却没有任何贡献。同样，高级经理的参与，也有可能导致比较差的级经理的参与，也有可能导致比较差的review效果，因为如果作者感觉到经理仅仅效果，因为如果作者感觉到经理仅仅是在数缺陷个数以对他进行评价的时候，作是在数缺陷个数以对他进行评价的时候，作者就会变的犹豫，也有可能破坏同事之间的者就会变的犹豫，也有可能破坏同事之间的气氛。气氛。57Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区

31、六：让不合适的人参加措施措施:一个一个review一般一般37人效率是最高的。人效率是最高的。review的专家应该包括工作产品的上下游的的专家应该包括工作产品的上下游的人。如：设计文档的人。如：设计文档的review，就应该包括作，就应该包括作者本人，需求文档的作者、编码的责任人、者本人，需求文档的作者、编码的责任人、完成测试的责任人，也应该包括对该领域很完成测试的责任人，也应该包括对该领域很熟悉的人。熟悉的人。58Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审

32、的误区n误区七：过多关注风格，忽略内容症状症状:如果当你浏览如果当你浏览review发现的缺陷列表，发现的缺陷列表，发现大部分缺陷都是风格上的，那么你应该发现大部分缺陷都是风格上的，那么你应该很紧张了：也许很多重要的、逻辑上、算法很紧张了：也许很多重要的、逻辑上、算法上的问题都被忽略了。上的问题都被忽略了。59Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n误区七：过多关注风格，忽略内容措施措施:风格可以是缺陷，但不应该成为风格可以是缺陷，但不应该成为rev

33、iew发现缺陷的主体。作者要在写作之前，即可发现缺陷的主体。作者要在写作之前，即可能接受相关培训，要尽量使用提供的模板；能接受相关培训，要尽量使用提供的模板；作者写完之后，要即可进行自检。作者写完之后，要即可进行自检。60Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散常见的软件技术评审的误区n走出误区走出自我对事不对人仅仅去发现问题，而不要试图去修改控制会议时间（=2Hr.）避免纠缠风格问题，除非它真的影响你尽早的、经常的进行正式或非正式的review61Confidential

34、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.62Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散Questions?63Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散写在最后写在最后在实际工作中，大家可以不断积累经验，指导自己的项目进行高效Review。运用之妙存乎一心小诗一首：你是风儿我是沙，你是蜜蜂我是花。你是梳子我是头发，你是牙膏我是牙刷。64Confidential Information of Huawei,No Spreading without Permission.华为机密，未经许可不得扩散谢谢谢谢