关键词：
软件工程（2018）结对编程第一次作业
本次作业要求对结对编程队友进行一次代码审查。
代码审查表如下
| 功能模块名称 |
图的邻接矩阵输出 |
|
|
|
| 审查人 |
李宗健 |
审查日期 |
2018/4/6 |
|
| 代码名称 |
图的邻接矩阵输出 |
代码作者 |
刘智 |
|
|
|
|
|
|
| 重要性 |
审查项 |
结论 |
|
|
|
头文件和定义文件的名称是否合理？ |
是 |
|
|
|
头文件和定义文件的目录结构是否合理？ |
是 |
|
|
|
版权和版本声明是否完整？ |
否 |
|
|
| 重要 |
头文件是否使用了 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？ |
否 |
|
|
| 重要 |
是否忘记写switch的default分支？ |
否 |
|
|
| 重要 |
使用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问题： |
|
|
|
|
（１）对不存在的或者错误的文件进行操作吗？ |
无 |
|
|
|
（２）文件以不正确的方式打开吗？ |
无 |
|
|
|
（３）文件结束判断不正确吗？ |
无 |
|
|
|
（４）没有正确地关闭文件吗? |
无 |
|
|
对代码进行了测试，测试成功，截图如下。
测试样例（1）：
测试样例（2）：
该代码是有关数据结构与算法中图的相关转换。程序要求输入图的信息（图的类型、图的顶点个数、图的顶点、图的边个数、图的边），输出图的邻接矩阵。核心算法思想是建立“图”这一数据结构（类），将点、边设为图（类）的属性。这样使在图的属性输入时完成点、边的输入，然后根据是否为有向图利用二维数组进行图的类型转换。最终输出二维数组作为图的邻接矩阵。
我刚拿到该代码时，感觉有点亲切。刘智同学代码的注释习惯跟我很像，言简意赅。在大括号方面，也有单个括号占一行的习惯，这点我很欣赏。但是在建立点位的数字信息函数中有个问题，使用链表并指向下一个点位，思想好但是后续程序好像没用到这个指针。在对着审查表审查时我发现有这个指针，之后程序却没用到。当然这个问题在这没什么影响，但是以后的程序设计中定义了无关的变量，积攒多了会影响内存的使用。可能造成这个问题的原因是编程时思路开阔，想到什么实现什么，最后程序也没错，便忘记了刚开始的变量。这就要求我们编程时遵循步骤，将核心算法思想完成后，按照核心算法逐步完善编程，避免想到啥写啥。
关于数组，我在上篇随笔中写到了动态数组的实现，所以我觉得该代码可以优化的地方是改成动态数组，这样既可以避免使用多余内存，又能使数据结构使用更方便。当然，实际生活中的程序根据问题需求会有范围限制，也没有必要追求过于完美。合适的就是最好的。
软件工程（2018）结对编程第一次作业
一.题目及要求(1).首先在同学中找一个同伴，范围不限，可以在1~5班中随意组合，建议尽量不要找同组的成员，女同学尽量找男同学结对，但是不做强制要求；(2).从以往个人完成的项目中选择一个作品，例如：以往的数据结构课... 查看详情
软件工程（2018）结对编程第一次作业
C++代码审查表p.MsoNormal,li.MsoNormal,div.MsoNormalmargin:0cm;margin-bottom:.0001pt;font-size:12.0pt;font-family:宋体h1margin-right:0cm;margin-left:0cm;font-size:24.0pt;font-family:宋体span.1Charfont-fami 查看详情
软件工程（2018）结对编程第一次作业
一.C++代码审查p.MsoNormalmargin:0pt;margin-bottom:.0001pt;font-family:宋体;font-size:12.0000pth1margin-top:5.0000pt;margin-bottom:5.0000pt;font-family:宋体;font-weight:bold;font-size:24.0000ptspan.10font- 查看详情
软件工程（2018）结对编程第一次作业
一.题目要求1.首先在同学中找一个同伴，范围不限，可以在1~5班中随意组合，建议尽量不要找同组的成员，女同学尽量找男同学结对，但是不做强制要求；2.从以往个人完成的项目中选择一个作品，例如：以往的数据结构课程设... 查看详情
软件工程（2018）结对编程第一次作业
功能模块名称 将已有序的子序列合并，得到完全有序的序列审查人 秦彬审查日期 2018.4.06代码名称 归并排序.cpp代码作者 刘傲文件结构重要性审查项结论头文件和定义文件的名称是否合理？是 头文件和定义文件的目... 查看详情
软件工程（2018）结对编程第一次作业(代码片段)
功能模块名称排队候诊系统 审查人张洁审查日期2018/04/06代码名称排队候诊系统代码作者刘子博文件结构 重要性审查项结论 头文件和定义文件的名称是否合理？是 头文件和定义文件的... 查看详情
软件工程（2018）结对编程第一次作业(代码片段)
题目要求(1).首先在同学中找一个同伴，范围不限，可以在1~5班中随意组合，建议尽量不要找同组的成员，女同学尽量找男同学结对，但是不做强制要求；(2).从以往个人完成的项目中选择一个作品，例如：以往的数据结构课程设... 查看详情
软件工程结对编程第一次作业
一、代码审查表参照：C/C++代码审查表代码地址：coding.net结对伙伴：蒲雪冰功能模块名称约瑟夫报数审查人胡泊审查日期2018/4/5代码名称约瑟夫报数代码作者蒲雪冰文件结构重要性审查项结论头文件和定义文件的名称是否合理？... 查看详情
软件工程第一次作业——结对编程
结对编程1500802101羊毛加博客链接：https://www.cnblogs.com/yangzhuo/coding地址：https://coding.net/u/CDYTF/p/diyicizuoyejieduibiancheng/git/tree/master/程序运行功能的截图1.界面 2.整数运算3.分数运算 结对编程心得：实现四则运算的程序没... 查看详情
2021秋软工实践第一次结对编程作业
这个课程属于哪里构建之法-2021秋-福州大学软件工程这个作业要求在哪里2021秋软工实践第一次结对编程作业这个作业的目标学习体验结对编程，学习设计原型模型学号031902110结对小伙伴学号031902114作业博客地址作业博客2021... 查看详情
软件工程（2018）第一次结对作业
结对作业之代码复审一、题目描述??m叉树的定义为：m叉树是指任意一个节点最多由m棵子树构成；m叉树的后序遍历是指：先后序遍历第一棵子树，第二棵，……，第m棵最后遍历根节点；用非递归的算法写出m叉树的后序遍历过程... 查看详情
软件工程（2018）结对编程第2次作业
结对编程作业一.选题驾驶员：汪江峰领航员：张林题目二：现有一新建办公大厦，共有21层，共有四部电梯，所有电梯基本参数如下表所示，其使用规定如下：楼层号为0~20，其中0号为地下一层有楼层限制的电梯不在相应楼层停... 查看详情
结对编程第一次作业
结对编程第一次作业1.代码审查结对伙伴：庞大雷代码地址：https://coding.net/u/pdlpi/p/yufafenxi/git/blob/master/%E8%AF%AD%E6%B3%95%E5%88%86%E6%9E%90.cpp?public=true代码审查表如下:参考文献：C/C++代码审查表功能模块名称图的基本操作审查人胡宜虎... 查看详情
软件工程第二次作业—结对编程
一.作业要求两人组成一队，进行结对编程。从第一次软件工程作业中选出一人的作业作为这次作业的基础。设计一个图形界面（GUI）的四则运算应用程序，可以是PC桌面的图形界面程序、网页程序或者手机程序。设计的应用程序... 查看详情
软件工程（2018）结对编程第2次作业
1.题目要求本次作业要求两个人合作完成，驾驶员和导航员角色自定，鼓励大家在工作期间角色随时互换，这里会布置两个题目，请各组成员根据自己的爱好任选一题。题目1：我们在刚开始上课的时候介绍过一个小学四则运算自... 查看详情
软件工程（2018）结对编程第2次作业
序作为领航员与驾驶员苏阔一组，辅助其完成代码工作。单元测试纵观苏的程序，其对于计算机程序的理解还停留在面向过程式编程层面上，且使用了类静态变量来充当全局变量来在函数间交换信息，导致各部分间耦合过紧。单... 查看详情
软件工程（2018）结对编程第2次作业(代码片段)
题目要求本次作业要求两个人合作完成，驾驶员和导航员角色自定，鼓励大家在工作期间角色随时互换，这里会布置两个题目，请各组成员根据自己的爱好任选一题。题目1：我们在刚开始上课的时候介绍过一个小学四则运算自... 查看详情
结对编程第一次作业(代码片段)
审查代码地址：https://coding.net/u/littlepudding/p/Softwarehomework_PairProgram/git?public=true功能模块名称Floyd最短路算法 审查人吕伟华 审查日期2018.4.5 代码名称Floyd最短路算法 代码作者陈鑫 文件结构重要性 审查项结... 查看详情