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

一、代码评审检查表： NO 检查项 是否符合 1 变量生存周期合适 O 2 代 X 3 代 4 代码符合代码格式规范 O 5 代码有健全的出错处理子程序 X 6 删除了多余语句 O 7 编译没有出错 O 8 连接没有出错 O 9 消除了所有警告 O 10 性能 X 11 安全 12 不存在溢出 O 13 不存在内存泄露 O 14 优先使用可重用组件或库中相同功能的函数来替换自己的实现 X 15 出现了恰当的转移语句 X 16 变量初始化没有异常，没有引用未初始化变量 O 17 变量类型定义没有出错 O 18 代码逻辑达到最佳编码 X 19 功能要求和代码实现一致 O 20 二、代码评审实验心得： 2.1代码评审定义： 代码评审，是一种静态的软件测试方法。它针对软件的源代码，通过走查、评审等方式，来有效的发现软件代码缺陷。 2.2代码评审优点： 有利于预防bug，为改善代码质量打下坚实的基础。在项目的早期发现缺陷，将损失降至最低评审的过程也是重新梳理思路的过程，双方都加深了对系统的理解促进团队沟通、促进知识共享、共同提高#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}}; /*++++++++++++++++++++++++++++++++++++++++++++++++++++++学生++++++++++++++++++++++++++++++++++++++++++++++++++++*/ void student(); void manger(); void jiemian(); void Add(); void change(); void select(int XH); void DropC(int XH) ; void teacher(); void manger(); void search(int XH) /*课程查询*/ { int j,i; char k[10]; printf(您已选择的课程为：\n); printf(课程号 课程名 上课时间 上课地点 上课老师 老师编号 课程人数\n); for(j=1;j=4;j++) { if(Student[XH][j]==0) printf(空\n); else { printf(%3d %10s %9.2f ,Cou[Student[XH][j]-1].Cno