代码飞检交流.ppt

代码飞检交流 华为通讯质量部 目录 目的和范围 角色定义 飞检流程 飞检过程监控 代码选取 收尾工作 目的和范围 代码飞检是一种突击性的研发内部审计，通过公布飞检结果和共享优秀实践，提升全体开发人员的代码质量意识。 检查编程规范的遵守情况 检查代码缺陷 挖掘代码质量控制的优秀实践 范围：已经通过开发人员Review和开发人员内部测试的代码。 角色定义 实际工作中可以根据大家对工作的熟练程度，同一个人担任多个角色，可参加多个审计小组。 飞检流程1 飞检流程2 飞检流程3 飞检过程监控 过程监控 在整个审计过程中，审计组长向全体审计成员和各人主管定期发送项目进展的邮件，通常每周提交一次项目进展邮件，在重要时间点或重要事情完成时也可随时提交进展报告，如开工会结束或飞检项目范围变更时 组长应及时赞扬表现突出的审计成员，并将赞扬邮件抄送相关人员和其主管，以肯定专家的工作绩效并获得专家的继续支持 审计项目结束时，组长总结本次审计的经验教训，并写作成文档，归档在服务期上 代码选取 根据飞检目的和范围，当前可用的人力资源，一般抽取4~6个产品进行代码飞检，最多不要超过6个。 主审人使用工具“Source Insight”，工程文件中包含被审计产品的全部代码，在整个工程的代码注释中有哪些信誉好的足球投注网站“2007”或“2008” “2009”等字符串，挑选出近期改动过的文件 主审人从近期改动过的文件中，挑选符合下列条件的文件 规模很大的文件或函数 低劣的编程风格，函数/变量命名，排版，注释等 存在大量字符串操作，大量资源操作，大量外部输入的接口，逻辑复杂(3层以上嵌套的if/for/while语句)，大量数据结构操作（树/链表） 收尾工作 审计报告 在访谈进行中，组长就开始写作审计报告的初稿，一般包括被审产品基本信息、优秀实践描述、主要问题描述、建议及后续行动等。 组长与飞检Sponsor沟通审计报告的框架，以明确报告是否内容完整。 组长与各主审人/被审计产品确认报告中的内容是否正确。 组长优化审计报告，并将最终的审计报告归档在服务器上。 审计问题跟踪 最终确认的审计结果通过邮件发送给产品后，组长将审计发现的主要问题录入公司持续改进数据库中进行闭环跟踪。 HUAWEI TECHNOLOGIES CO., LTD. Huawei Confidential 英文标题:32-35pt 颜色: R153 G0 B0 内部使用字体 : FrutigerNext LT Medium 外部使用字体 : Arial 中文标题:30-32pt 颜色: R153 G0 B0 字体:黑体 英文正文:20-22pt 子目录 (2-5级) :18pt 颜色:黑色 内部使用字体 : FrutigerNext LT Regular 外部使用字体 : Arial 中文正文:18-20pt 子目录(2-5级):18pt 颜色:黑色 字体:细黑体 配色参考方案： 建议同一页面内不超过四种颜色，以下是13组配色方案，同一页面内只选择一组使用。（仅供参考） 客户或者合作伙伴的标志放在右上角. HUAWEI TECHNOLOGIES CO., LTD. Huawei Confidential Security Level: 英文标题:40-47pt 副标题:26-30pt 字体颜色:反白 内部使用字体 : FrutigerNext LT Medium 外部使用字体 : Arial 中文标题:35-47pt 字体:黑体 副标题:24-28pt 字体颜色:反白 字体:细黑体 英文目录标题:35-40pt 颜色: R153 G0 B0 内部使用字体 : FrutigerNext LT Medium 外部使用字体 : Arial 中文目录标题:35-40pt 颜色: R153 G0 B0 字体:黑体 英文目录正文:28-30pt 子目录 (2-5级) :20-30pt 颜色:黑色 内部使用字体 : FrutigerNext LT Regular 外部使用字体 : Arial 中文目录正文:28-30pt 子目录(2-5级):20-30pt 颜色:黑色 字体:细黑体 Thank you 参与代码Pre-Review、进行走读、参与问题确认、访谈 工程师，熟悉编程知识 每个审计小组的成员，每个审计小组2-4人 审计专家 人员协调：明确本小组成员具体任务。 过程监控：挑选被审计的源文件， 组织代码走读、收集Pre-Review表单、内部确认本小组飞检结果、汇总整理review summery表单，组织访谈 软件技术专家，精通编程知识、编程规范和审计过程 每个审计小