结对作业1

1.对方代码地址


Coding.net地址：[点我就知道]


2.代码审查表

 

功能模块名称 比赛记分牌的设计与实现
审查人 张庆阳　 审查日期 2019/4/22　
代码名称 比赛记分牌的设计与实现　 代码作者 　张文亮
文件结构
重要性 审查项 结论
  头文件和定义文件的名称是否合理？ 合理　
　 头文件和定义文件的目录结构是否合理？ 合理
　 版权和版本声明是否完整？ 不完整　
重要 头文件是否使用了 ifndef/define/endif 预处理块? 否　
　 头文件中是否只存放“声明”而不存放“定义” 是
　 　 　
程序的版式
重要性 审查项 结论
　 空行是否得体？ 是　
　 代码行内的空格是否得体？ 是　
　 长行拆分是否得体？ 是　
　 “{” 和 “}” 是否各占一行并且对齐于同一列？ 是　
重要 一行代码是否只做一件事？如只定义一个变量，只写一条语句。 是　
重要 If、for、while、do等语句自占一行，不论执行语句多少都要加 “{}”。 是　
重要 在定义变量（或参数）时，是否将修饰符 * 和 ＆ 紧靠变量名？注释是否清晰并且必要？ 是　
重要 注释是否有错误或者可能导致误解？ 否　
重要 类结构的public, protected, private顺序是否在所有的程序中保持一致？ 无类结构　
　 　 　
命名规则
重要性 审查项 结论
重要 命名规则是否与所采用的操作系统或开发工具的风格保持一致？ 是，一致　
　 标识符是否直观且可以拼读？ 是　
　 标识符的长度应当符合“min-length && max-information”原则？ 是　
重要 程序中是否出现相同的局部变量和全部变量？ 是　
　 类名、函数名、变量和参数、常量的书写格式是否遵循一定的规则？ 是　
　 静态变量、全局变量、类的成员变量是否加前缀？ 是　
　 　 　
表达式与基本语句
重要性 审查项 结论
重要 如果代码行中的运算符比较多，是否已经用括号清楚地确定表达式的操作顺序？ 是　
　 是否编写太复杂或者多用途的复合表达式？ 是　
重要 是否将复合表达式与“真正的数学表达式”混淆？ 否　
重要 是否用隐含错误的方式写if语句? 例如 否　
　 （1）将布尔变量直接与TRUE、FALSE或者1、0进行比较。 否　
　 （2）将浮点变量用“==”或“！=”与任何数字比较。 否　
　 （3）将指针变量用“==”或“！=”与NULL比较。 否　
　 如果循环体内存在逻辑判断，并且循环次数很大，是否已经将逻辑判 　
　 断移到循环体的外面？ 是　
重要 Case语句的结尾是否忘了加break？ 无case语句　
重要 是否忘记写switch的default分支？ 无switch语句　
重要 使用goto 语句时是否留下隐患? 例如跳过了某些对象的构造、变量的初始化、重要的计算等。 无goto语句　
　 　 　
常量
重要性 审查项 结论
　 是否使用含义直观的常量来表示那些将在程序中多次出现的数字或字符串？ 是　
　 在C++ 程序中，是否用const常量取代宏常量？ 否　
重要 如果某一常量与其它常量密切相关，是否在定义中包含了这种关系？ 是　
　 是否误解了类中的const数据成员？因为const数据成员只在某个对象 否　
　 生存期内是常量，而对于整个类而言却是可变的。 否　
　 　 　
函数设计
重要性 审查项 结论
　 参数的书写是否完整？不要贪图省事只写参数的类型而省略参数名字。 是　
　 参数命名、顺序是否合理？ 是　
　 参数的个数是否太多？ 是　
　 是否使用类型和数目不确定的参数？ 否　
　 是否省略了函数返回值的类型？ 否　
　 函数名字与返回值类型在语义上是否冲突？ 否　
重要 是否将正常值和错误标志混在一起返回？正常值应当用输出参数获得，而错误标志用return语句返回。 是　
重要 在函数体的“入口处”，是否用assert对参数的有效性进行检查？ 否　
重要 使用滥用了assert？ 例如混淆非法情况与错误情况，后者是必然存在的并且是一定要作出处理的。 否　
重要 return语句是否返回指向“栈内存”的“指针”或者“引用”？ 是　
　 是否使用const提高函数的健壮性？const可以强制保护函数的参数、返回值，甚至函数的定义体。“Use const whenever you need” 否　
　 　 　
内存管理
重要性 审查项 结论
重要 用malloc或new申请内存之后，是否立即检查指针值是否为NULL？（防止使用指针值为NULL的内存） 否　
重要 是否忘记为数组和动态内存赋初值？（防止将未被初始化的内存作为右值使用） 是　
重要 数组或指针的下标是否越界？ 否　
重要 动态内存的申请与释放是否配对？（防止内存泄漏） 否　
重要 是否有效地处理了“内存耗尽”问题？ 否　
重要 是否修改“指向常量的指针”的内容？ 否　
重要 是否出现野指针？例如（1）指针变量没有被初始化;（2）用free或delete释放了内存之后，忘记将指针设置为NULL。 否　
重要 是否将malloc/free 和 new/delete 混淆使用？ 否　
重要 malloc语句是否正确无误？例如字节数是否正确？类型转换是否正 确？ 是　
重要 在创建与释放动态对象数组时，new/delete的语句是否正确无误？ 无　
　 　 　
C++ 函数的高级特性
重要性 审查项 结论
　 重载函数是否有二义性？ 否　
重要 是否混淆了成员函数的重载、覆盖与隐藏？ 否　
　 运算符的重载是否符合制定的编程规范？ 无重载　
　 是否滥用内联函数？例如函数体内的代码比较长，函数体内出现循环。 否　
重要 是否用内联函数取代了宏代码？ 无　
　 　 　
类的构造函数、析构函数和赋值函数
重要性 审查项 结论
重要 是否违背编程规范而让C++ 编译器自动为类产生四个缺省的函数： 　
　 （1）缺省的无参数构造函数； 否　
　 （2）缺省的拷贝构造函数； 无　
　 （3）缺省的析构函数； 无　
　 （4）缺省的赋值函数。 无　
重要 构造函数中是否遗漏了某些初始化工作？ 是　
重要 是否正确地使用构造函数的初始化表？ 否　
重要 析构函数中是否遗漏了某些清除工作？ 否　
　 是否错写、错用了拷贝构造函数和赋值函数？ 否　
重要 赋值函数一般分四个步骤： 　
　 （1）检查自赋值； 无　
　 （2）释放原有内存资源； 无　
　 （3）分配新的内存资源，并复制内容； 无　
　 （4）返回 *this。是否遗漏了重要步骤？ 无　
重要 是否正确地编写了派生类的构造函数、析构函数、赋值函数？ 无　
　 注意事项： 　
　 （1）派生类不可能继承基类的构造函数、析构函数、赋值函数。 无派生　
　 （2）派生类的构造函数应在其初始化表里调用基类的构造函数。 无　
　 （3）基类与派生类的析构函数应该为虚（即加virtual关键字）。 无　
　 （4）在编写派生类的赋值函数时，注意不要忘记对基类的数据成员重新赋值 无　
　 　 　
类的高级特性
重要性 审查项 结论
重要 是否违背了继承和组合的规则？ 否　
　 （1）若在逻辑上B是A的“一种”，并且A的所有功能和属性对B而言都有意义，则允许B继承A的功能和属性。 否　
　 （2）若在逻辑上A是B的“一部分”（a part of），则不允许B从A派生，而是要用A和其它东西组合出B。 否　
　 　 　
其它常见问题
重要性 审查项 结论
重要 数据类型问题： 　
　 （１）变量的数据类型有错误吗？ 否　
　 （２）存在不同数据类型的赋值吗？ 否　
　 （３）存在不同数据类型的比较吗？ 否　
重要 变量值问题： 　
　 （１）变量的初始化或缺省值有错误吗？ 否　
　 （２）变量发生上溢或下溢吗？ 无　
　 （３）变量的精度够吗？ 够　
重要 逻辑判断问题： 　
　 （１）由于精度原因导致比较无效吗？ 否　
　 （２）表达式中的优先级有误吗？ 否　
　 （３）逻辑判断结果颠倒吗？ 否　
重要 循环问题： 　
　 （１）循环终止条件不正确吗？ 　
　 （２）无法正常终止（死循环）吗？ 否　
　 （３）错误地修改循环变量吗？ 否　
　 （４）存在误差累积吗？ 否　
重要 错误处理问题： 　
　 （１）忘记进行错误处理吗？ 否　
　 （２）错误处理程序块一直没有机会被运行？ 否　
　 （３）错误处理程序块本身就有毛病吗？如报告的错误与实际错误不一致，处理方式不正确等等。 否　
　 （４）错误处理程序块是“马后炮”吗？如在被它被调用之前软件已经出错。 否　
重要 文件I/O问题： 　
　 （１）对不存在的或者错误的文件进行操作吗？ 否　
　 （２）文件以不正确的方式打开吗？ 无　
　 （３）文件结束判断不正确吗？ 无　
　 （４）没有正确地关闭文件吗? 无　
　 　 　

3.总结

 

张文亮同学的代码排版很规范，让人看起来很舒服。
在很多细节方面做得都很出色，并且在开头按照分类对变量进行命名，
特别是他采用了宏定义进一步简化了代码，这样的思考值得我学习。
在其他的方面，空格的与括号的缩进页很规范
，每个{和}都各占了一行，缩进很明显
空行也很规范，按照代码块的功能进行空行，这都是细节所在
，代码行内的空格，长行拆分做得也十分得体。

他在每个比较重要的函数和关键语句后面都添加了注释，
对我阅读代码起到了很大的帮助。
这些细节让我从他的代码中学习到了一些提高自己编码效率的方法，让我学到了许多。可见宏定义大大简化了阅读所遇到的障碍，函数参数的命名也很有规则，把字母转化为易懂的英文字符。
而且代码也很紧凑，值得我去学习。

缺点：在部分函数的结构有过于重复和臃肿的情况，比如这一段可以试着写的更加紧凑一点，写成For循环的形式
更能体现C语言的高效性。

 1 #define uChar unsigned char
 2 #define uInt unsigned int
 3 
 4 //定义的常量
 5 #define B00100000 0x20
 6 
 7 //键盘及数码管操控地址
 8 #define    OUTBIT 0x08002//位选择地址
 9 #define OUTSEG 0x08004//段选择地址
10 #define IN 0x08001//键盘读入
11 
12 #define InitMode -5//键盘读入
13 #define InputMode -3//键盘读入
14 #define SearchMode -4//键盘读入
15 
16 
17 void Display(signed int Score,signed int key)//LED显示函数
18 {
19 unsigned char mShowPosition = B00100000; 
20 signed int a = 0;
21 signed int b = 0;
22 if(key<0)
23 {a = (-key)/10;
24 b =    (-key)%10;
25 }
26 else
27 {
28 a = key/10;
29 b =    key%10;
30 }
31 //总分百分位
32 //*DPTR = 0xdf;//1101 1111
33 mPOutbit = mShowPosition;
34 mPOutseg = dispcode[Score/100];
35 mShowPosition >>= 1;//0001 0000
36 delay(2);
37 
38 //总分十分位
39 
40 //*DPTR = 0xef;//1110 1111
41 mPOutbit = mShowPosition;
42 mPOutseg = dispcode[Score/10%10];
43 mShowPosition >>= 1;//0000 1000
44 delay(2);
45 
46 //总分个位
47 
48 //*DPTR = 0xf7;//1111 0111
49 mPOutbit = mShowPosition;
50 mPOutseg = dispcode[Score%10];
51 mShowPosition >>= 1;// 0000 0100
52 delay(2);
53 
54 if(key<0)//当前局分数为负数
55 {
56 mPOutbit = mShowPosition;
57 mPOutseg = 0x40;
58 mShowPosition >>= 1;// 0000 0100
59 delay(2);
60 }
61 else//当前局分数为正数
62 { 
63 mShowPosition >>= 1;// 0000 0100
64 }
65 
66 
67 //当前局分数十分位
68 mPOutbit = mShowPosition;
69 //*DPTR = 0xfd;//1111 1101
70 //if(key<0){key = -key;}
71 mPOutseg = dispcode[a];
72 mShowPosition >>= 1; // 0000 0001
73 delay(2);
74 
75 //当前局分数个位
76 mPOutbit = mShowPosition;
77 //*DPTR = 0xfe;//1111 1110
78 mPOutseg = dispcode[b];
79 //mShowPosition >>= 1;
80 delay(2);
81 
82 }

 

 

当然看别人的代码是一个非常独特的体验，一步步地了解别人的想法，欣赏别人的风格，这些都是独自编程中看不到的。
通过张文亮同学的思路我也学到了很多，找到了自身的一些不足，在以后的任务中，我会和队友同心协力，共同进步。

posted @ 2019-04-22 21:05  檠央  阅读(216)  评论(0)  编辑  收藏  举报