学生选课系统-代码评审.doc

一、代码评审检查表： NO 检查项 是否符合 1 变量生存周期合适 O 2 代码不仅包含单元测试用例，而且测试均已通过。 X 3 代码包含了恰当的注释和代码文档 O 4 代码符合代码格式规范 O 5 代码有健全的出错处理子程序 X 6 删除了多余语句 O 7 编译没有出错 O 8 连接没有出错 O 9 消除了所有警告 O 10 性能问题有考虑 X 11 安全问题有考虑 O 12 不存在溢出 O 13 不存在内存泄露 O 14 优先使用可重用组件或库中相同功能的函数来替换自己的实现 X 15 出现了恰当的转移语句 X 16 变量初始化没有异常，没有引用未初始化变量 O 17 变量类型定义没有出错 O 18 代码逻辑达到最佳编码 X 19 功能要求和代码实现一致 O 20 二、代码评审实验心得： 2.1代码评审定义： 代码评审，是一种静态的软件测试方法。它针对软件的源代码，通过走查、评审等方式，来有效的发现软件代码缺陷。 2.2代码评审优点： 有利于预防bug，为改善代码质量打下坚实的基础。在项目的早期发现缺陷，可以将损失降至最低。评审的过程也是重新梳理开发思路的过程，双方都加深了对系统的理解。有利于促进团队沟通、促进知识共享、共同提高。 2.3代码评审实现过程及心得体会： 在针对源程序，即学生选课管理系统源代码的基础上，结合代码评审的相关概念、思想，还有参考了相关知识。有针对性的设计了，一个评审检查表。按照代码评审的要求，逐步按评审检查表的款项。逐条，执行检查。严格的按照审查要求，严肃的将检查结果记录在册。 代码评审的过程中，按照一定的速度执行代码检查。检查代码的编程风格，书写格式，变量定义、生存周期、初始化，代码注释是否合适等款项。每一条检查项目严格执行，这个过程十分枯燥。但必须仔细，认真对待，不容许出现一丝错误。 代码评审，是我们以前重来没有经历过的事情，对于这些事情。我们既充满了好奇，也有些忐忑不安。在实验的过程中，我们一边查找资料，一边根据书上所学、以及老师所讲。一边学，一边用。在这样的状态下，历经苦难，终于完成了这个过程。在这个过程中，我学到了很多，在以前都不会注意的东西。学会了以一种严谨的态度来认真执行，学会了有问题就同伙伴，交流、讨论。 在将来，或许我会从事软件行业。相信到那个时候，在做软件测试的时候，这个时候的一段经历，会对我很有帮助。 三、源代码： #includestdio.h #includestring.h #includewindows.h #define SCMAX 6/*每人选课最大数目-1*/ #define TEMAX 4 /*老师能教的最大课程数目-1*/ #define STU 1000/*最大学生人数*/ #define CNO 200 /*课程最大数目*/ #define Tea 200 /*老师最大编号*/ #define CPMAX 80 /*规定每门课人数最大值*/ struct Course{ int Cno;/*课程号*/ char Cname[20];/*课程名*/ double time;/*上课时间*/ char place[20];/*上课地点*/ char teacher[10];/*上课老师*/ int Tno; /*老师编号*/ int pn ; /*学生人数*/ }Cou[CPMAX]={ {1,数学,109.40, 5#5110, 张三,1,4}, {2,语文,210.30,6#6110, 李四,2,1}, {3,英语,307.30, 15#15217,王五,3,1}, {4,经济,715.40,6#6510, 张三,1,4}, {5,防身术,10.00,8#8321,李四,2,0}, {6,电脑游戏,10.00,5#5310,王五,3,0} }; /*课程初始化 课程编号、上课时间、上课地点、老师、老师编号、课程人数*/ struct STUDENT{ int no; char Sname[10]; }S[6]={{1,尹新荣},{2,李明},{3,李阳},{4,刘成立},{5,艾斯},{6,司丽芳}}; int Student[STU][SCMAX]={{1,1,0,0,0},{2,1,0,0,0},{3,3,0,0,0},{4,1,2,3,4},{5,1,0,0,0} ,{6,0,0,0,0}}; int Teacher[Tea][SCMAX]={{1,1,4,0,0},{2,2,5,0,0},{3,3,6,0,0}}; /*+++++++++