实践作业3:白盒测试实践(小组作业)每日任务记录5
会议时间:2017年12月15日
会议地点:东九的空教室
主持人:杨潇
参会人员:王晨懿、余晨晨、郑锦波、杨潇、侯欢、汪元
记录人:郑锦波
会议议题:完成代码复审,决胜最后阶段
今天,我们小组在下午上完5、6节课后,开始进行了代码评审会议,小组全体组员都积极参加,通过静态评审发现了更多的缺陷。原本是定于昨天进行会议复审的,但是由于部分同学的实验室有任务,没有时间参加。为了保证评审能够有更好的效果,昨天就决定今天每个人参加会议评审,并且大家都提前做了相应的准备。
小组各成员任务的完成情况和主要困难
姓名 |
当前完成情况 |
当前遇到的困难 |
汪元 |
已经完成了单元测试的工作,即将完成测试执行过程及小结文档 |
无 |
余晨晨 |
根据今天的评审结果和之前使用静态代码检查工具发现的bug进行对比,继续完成后续工作 |
目前进行得比较顺利 |
杨潇 |
已经完成支持评审会议,继续完成会议后的其他内容 |
目前进行得比较顺利 |
侯欢 |
已完成所有任务 |
无 |
王晨懿 |
继续协调整个组的作业的进度,开始着手完成并提交测试总结报告 |
目前进行得比较顺利 |
郑锦波 |
持续跟进作业的完成进度,记录完成情况,跟进博客的更新 |
目前进行得比较顺利 |
代码评审会议的流程
主持人:杨潇
开发者:余晨晨
记录员:侯欢
评审员:汪元、侯欢、王晨懿、郑锦波
1、主持人对本次评审会议的规划
• 制订评审计划
• 捡查入口标准
• 准备评审材料包
• 选择合适的评审员
• 给全体组员分发评审材料
2、评审预备会
• 开发者向评审员简要介绍工作产品和相关资料,包括被测模块的目标、相关实现细节,开发标准等
• 评审员听取开发者和主持人的介绍,对被测模块进行进一步审查。并记录发现的缺陷,反馈给主持人。
• 主持人汇总收集的审阅情况记录表,并判断是否需要增加评审的投入
3、正式召开评审会议
• 主持人: 召开会议。主持会议,控制会议进度,维持会议程序,会后提交评审报告,给出评审结论
• 开发者:讲解工作产品。引导评审员浏览工作产品,共同确认问题
• 评审员:听取讲解,发表意见,指出问题,与作者确定问题、定义问题的严重程度
• 开发者:倾听讲解和评审员的意见,回答提问
• 记录员:记录每个达成共识的缺陷,确保评审员同意对问题的记录,并记录未达成共识的缺陷,作为会议评审的对象。更新审阅情况记录表。
4、修复缺陷
• 开发者:修改工作产品。针对提交的审阅情况记录表,对每个缺陷进行定位、调试和修复,然后提交修复后的工作产品,同时更新审阅情况记录表,在表格中简要说明对每个缺陷的修复过程。
• 主持人:对缺陷清单进行分析和整理,以便于对缺陷检查表进行扩充,在后续的静态测试中,提供给评审员,从而提高后续静态测试的效率。
5、确认修复
• 主持人:指派专人对修复后的工作产品进行确认。查看最终提交的审阅情况记录表,判断工作产品是否符合退出标准。
• 评审员:对照更新后的审阅情况记录表,检查修复后的工作产品,判断是否所有缺陷均得到正确的修复,且未引入新的缺陷。否则打回给作者要求其继续修复。若所有缺陷都己确认修复,则更新审阅情况记录表,并提交给主持人。
代码评审会议中发现的部分缺陷
1、 代码缺少必要的注释,在整个代码的逻辑层中,基本没有必要的注释
2、 代码的风格不统一,比如“{”有时是开行写,有时是在代码后面写的
3、 If语句的内容没有换行,并且和if的同级语句写在了同一行
4、 用户修改的条件未设置边界条件,导致很多不合法的数据都能通过
5、 安全性问题,使用get传输明文密码,导致帐号密码暴露,具体表现为在网址的url中会显示当前用户的帐号密码
代码评审会议中发现的部分优点
1、 对变量的声明都有注释,并且变量命名很规范,易于阅读
2、 单独提取了数据库模块,增加了代码的复用性和可读性
代码评审会议中的场景
1、 每个小组成员都有自己的代码规范文档,并且都对代码规范文档和代码进行了提前学习。
2、 各小组成员通过teamViewer这个软件进行会议交流,开发者余晨晨通过teamviewer进行屏幕共享向大家介绍系统和相关资料
3、 全体小组成员同通过teamviewer进行会议
posted on 2017-12-15 23:09 软工1702班第4组软件测试每日例会记录 阅读(165) 评论(0) 编辑 收藏 举报