1、软件技术评审,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
2、 Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,不同点:共有10处,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,课程目的,了解业界软件技术评审方法 掌握我司软件技术评审流程 学习我司软件技术评审工具 分享多年的实际应用开发经验,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可
3、不得扩散,课程主要内容,软件技术评审方法 IPD-CMM软件评审流程 我司常用软件技术评审工具 常见的软件技术评审的误区,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,软件技术评审,软件技术评审方法 IPD-CMM软件技术评审流程 华为常用软件技术评审工具 常见的软件技术评审的误区,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,软件技术评审方法,软件技术评审
4、的定义 借助他人对软件交付物进行检查 发现缺陷或获得改进优化的机会 强有力的软件质量保证活动之一 CMM中定义为Peer Review,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,软件技术评审方法,软件技术评审的意义 降低返工（rework）的成本 业界的一些数据： 相对于通过review发现缺陷的rework成本，测试发现缺陷的rework成本是其14.5倍,而客户发现缺陷的rework成本是其68倍一家德国软件公司 产品发布后发现缺陷的rework成本是在设计阶段的45
5、倍 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
6、Spreading without Permission. 华为机密，未经许可不得扩散,软件技术评审方法,Inspection的基本特点 定义质量目标 由经过培训的团队参加 由经过培训的组织者组织（作者不能是组织者） 明确参加人员角色和职责 有正式的文档规程引导 向管理者上报review结果 清晰的入口和出口条件 对缺陷进行跟踪直至关闭 记录过程和质量数据，并根据数据进行review流程和软件开发过程进行改进,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,软件技术评审方法,W
7、alkthrough的基本特点 没有明确的质量目标 有正式的会议 作者就是组织者 作者就软件交付物对评审人进行讲解 没有正式的文档规程引导 不必向管理者上报review结果 很少有正式的记录过程和质量数据，无法根据数据进行review流程和软件开发过程进行改进,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,软件技术评审方法,4 Eyes Review的基本特点 临时的、事件驱动的 仅仅是为了找出一个bug 更多的是团队协助 为了作者发现问题提供了另外一种角度 最不正式的rev
8、iew,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,软件技术评审方法,3种review形式的比较1,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,软件技术评审方法,3种review形式的比较2,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散
9、,软件技术评审方法,选择合适的review形式 基于风险考虑：交付物存在缺陷的可能性以及如果存在缺陷的影响 用最正式的形式去review高风险的交付物 用最经济的形式去review低风险的交付物 用收集数据来判断对某一工作产品，哪种review形式最有效,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. 华为机密，未经许可不得扩散,软件技术评审方法,更重要的是 形成一种文化拥抱评审，把它作为改进软件质量、提高生产率以及加快个人学习进度的强有力的方法,Confidential Informat
11、ion 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.
12、华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,步骤,出口 准则,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,计划阶段,IPD-CMM软件技术评审流程,项目经理指定组织者 作者自检工作产品 组织者规划本次Review检查入口准则准备Review包(工作产品/参考资料Review表单/查检表)指定Review人员（3-6人） 组织者将Review包、Review通知单发给相关人员,Confidential Information of Huawei, No Spr
13、eading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,介绍工作产品及相关资料,介绍会议,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,准备阶段,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,Review会议,Confidential I
14、nformation 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. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,跟踪,收到Re
15、view表单和工作产品 对照Review表单、验证工作产品中的缺陷更改正确 若缺陷未修改正确，返回给作者继续修改 更新Review表单、并分发给相关人员,？出口准则是否满足,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,职责：项目经理,制定Review计划、明确日程 选择合适的组织者和Review人员,Confidential Information of Huawei, No Spreading without Permission. 华为机
16、密，未经许可不得扩散,IPD-CMM软件技术评审流程,职责：作者,确保工作产品已准备好 在介绍会议上讲解工作产品 在Review会议结束后修改已确认的所有缺陷,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,职责：组织者1,组织管理会议，以防无休止的争论,裁决有异议的问题,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-C
17、MM软件技术评审流程,职责：组织者2,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,职责：组织者3,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,职责：讲解员,在Review会议上通过讲解来引导Review团队浏览工作产品。,Confidential Information of Huawei,
18、 No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,职责：记录员,负责记录Review会议上的信息。,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,职责：review人员,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,裁
19、减原则,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,裁减原则,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,北研所客户化的一些review流程制定 组织者提前对评审对象进行分解，有针对性的分配给不同专家 与评审专家沟通，获得承诺。 请用下面的表格作为通知发给相关人，记得每次评审的这个表格抄送给
20、EPG 和项目的QA。,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,IPD-CMM软件技术评审流程,北研所客户化的一些review流程制定 评审后，组织者把每个评审专家的实际预审投入，发现的问题详细的记录在Review Summery表单中，发给EPG和项目QA，由他们对这次评审做个分析和评价。这可是天上掉馅饼，何乐不为呢？,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未
21、经许可不得扩散,软件技术评审方法 IPD-CMM软件技术评审流程 华为常用软件技术评审工具 常见的软件技术评审的误区,软件技术评审,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,华为常用软件技术评审工具,IPD-CMMv3.0 Review Form,http:/bjpal - IPD-CMM - Review,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,
22、华为常用软件技术评审工具,Code Review Tool Document Review Tool 四象限分析法,http:/bjpal - 工具数据库 开发工具-Review Tool,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,软件技术评审方法 IPD-CMM软件技术评审流程 华为常用软件技术评审工具 常见的软件技术评审的误区,软件技术评审,Confidential Information of Huawei, No Spreading without Permiss
23、ion. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区一：参与者不了解review流程 症状:参与者对自己在review中的角色、职责以及如何进行review中的活动，各有各的理解；项目组成员对自己的产品哪些应该review、什么时候进行review、用什么样的review方法根本不清楚；一次review上，review太多的资料，结果却是两手空空,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区一：参与者不了解review流程 措施
24、:培训！培训将最好的团队建设活动，培训将能确保每个成员都能准确理解review的过程。一般而言4-8个小时的培训基本足够了；但对于正式review的组织者而言，可能要更多一些。培训内容的准则是遵循公司所制定的review规程。,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区二：对人不对事 症状:Review变成了一场人身攻击会、批斗会.挖苦作者的风格、嘲笑作者的技能。当作者感受到自己受到攻击，他（她）会开始对抗：拒绝配合；以后不愿意提交产品rev
25、iew；甚至寻求报复的机会。,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区二：对人不对事 措施:在进行review前，要强调review是发现产品的问题而不是对作者进行批斗。review不是给review专家一个机会展示自己比其他人有多聪明，而是要通过集体的智慧、观察力、经验来发现产品的缺陷，提高产品的质量。当你发现问题的时候，这样表达：“我没有发现这个地方的变量被初始化”也许比“你怎么忘记了把这个变量初始化了”要更好一些。,Confident
26、ial Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区三：没有计划 症状:对于许多项目而言，review从来不会出现在项目WBS或进度安排中；即使项目计划上有，也仅仅是以里程碑的形式出现而不是任务；里程碑是一个以“zero-time”定义的。没有计划，导致当需要review专家参与的时候，review专家已经没有时间了，他已经安排其他事情了!,Confidential Information of Huawei, No Spreading without Permissi
27、on. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区三：没有计划 措施:做好计划，以任务的形式将review体现到项目计划中。要估计出review专家个人准备的时间、review会议的时间、rework的时间。,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区四：Review会议演变成问题解决会 症状:开发人员天性都喜好解决问题，喜欢提出对问题的优雅的解决方案。但这并不是review会议所想要的。Review会议的目的是发现缺陷、确认
28、问题。如果会议演变成问题解决会议，对产品的检验就嘎然而止；争论变得无休止；部分参与者变的无所适从。当意识到更应该去发现缺陷的时候，时间已经所剩无几.于是匆匆结束review，很多缺陷却还在“花丛中笑”.,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区四：Review会议演变成问题解决会 措施:开正式的Review会议的目的是对专家反馈的意见进行确认并试图发现更多的问题。正式的review会议之所以比非正式的更有效，是因为它有组织者；当review
29、进展偏离的正常的轨道，组织者有责任及时把会议纠正到正常秩序来。一般而言，对问题的讨论是不可避免的，如果这个问题能在12分钟等讨论清楚，那么就讨论它；反之，还是把它放在会议之外解决吧。当然，可能会有这样的情况，某个缺陷很严重，严重到不把这个问题解决了，后面的review就无任何意义，那么就把它改成问题解决会议。,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区五：Review专家还没有准备 症状:你早上8：45来到工位上打开Notes，收到一个加了“
30、!“的邮件，通知你早上9：00到D304参加XXXXreview会议，并附件了10M的资料。你只有15分钟时间，你能review出问题来吗？你不了解项目的背景，不了项目的约束假设，不了解项目的关联，你即使参加了会议，就能发现问题了吗？,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区五：Review专家还没有准备 措施:75的缺陷都是在专家个人准备的时候发现的，这就是为什么组织者在开会前需要收集专家个人准备review的时间，以确定会议时候能进行。
31、所以项目经理要做好项目计划，提前安排好review的个人准备时间。,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区六：让不合适的人参加 症状:如果让一个对产品、项目还不熟悉、技能还比较欠缺的人参与review，也许唯一收获就他通过review了解了产品，但对工作产品质量的提高却没有任何贡献。同样，高级经理的参与，也有可能导致比较差的review效果，因为如果作者感觉到经理仅仅是在数缺陷个数以对他进行评价的时候，作者就会变的犹豫，也有可能破坏同事之
32、间的气氛。,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区六：让不合适的人参加 措施:一个review一般37人效率是最高的。review的专家应该包括工作产品的上下游的人。如：设计文档的review，就应该包括作者本人，需求文档的作者、编码的责任人、完成测试的责任人，也应该包括对该领域很熟悉的人。,Confidential Information of Huawei, No Spreading without Permission. 华为机密，
33、未经许可不得扩散,常见的软件技术评审的误区,误区七：过多关注风格，忽略内容 症状:如果当你浏览review发现的缺陷列表，发现大部分缺陷都是风格上的，那么你应该很紧张了：也许很多重要的、逻辑上、算法上的问题都被忽略了。,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,误区七：过多关注风格，忽略内容 措施:风格可以是缺陷，但不应该成为review发现缺陷的主体。作者要在写作之前，即可能接受相关培训，要尽量使用提供的模板；作者写完之后，要即可进行自检。,C
34、onfidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,常见的软件技术评审的误区,走出误区 走出自我 对事不对人 仅仅去发现问题，而不要试图去修改 控制会议时间（= 2Hr.） 避免纠缠风格问题，除非它真的影响你 尽早的、经常的进行正式或非正式的review,Confidential Information of Huawei, No Spreading without Permission. 华为机密，未经许可不得扩散,总结,Wow! Peer Review definitely is
35、 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. 华为机密，未经许可不得扩散,谢谢,
