实践作业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进行会议