2020软件工程第四次作业

博客信息沈阳航空航天大学计算机学院2020软件工程作业
作业要求 https://edu.cnblogs.com/campus/sau/Computer1701-1705/homework/10685
课程目标 熟悉一个“高质量”软件的开发过程
作业目标 结对编程练习

一、题目要求
     我们在刚开始上课的时候介绍过一个小学四则运算自动生成程序的例子,请实现它,要求:
        1.能够自动生成四则运算练习题
        2.可以定制题目数量
        3.用户可以选择运算符
        4.用户设置最大数(如十以内、百以内等)
        5.用户选择是否有括号、是否有小数
        6.用户选择输出方式(如输出到文件、打印机等)
        7.最好能提供图形用户界面(根据自己能力选做,以完成上述功能为主)
二、合作分工
      驾驶员 :刘博文   完成全部代码工作,程序基本实现全部要求功能;对导航员在本次编程工作中起到的作用给出客观评价;
      领航员 :王宏宇   辅助驾驶员完成全部代码工作,并且为关键函数选用合适的覆盖标准设计测试用例,并编写代码进行单元自动测试;
                                   利用上次结对编程作业的代码检查表对驾驶员的本次工作进行评价,并同时填写PSP 0.1工作记录表
三、代码审查
      小学四则运算自动生成程序 源代码
      小学四则运算自动生成程序 单元测试代码
      代码运行结果:

 

 

      代码审查表

功能模块名称

小学四则运算自动生成程序

审查人

王宏宇

 代码作者

 刘博文

文件结构

重要性      

审查项

结论

                

头文件和定义文件的名称是否合理?

是 

 

头文件和定义文件的目录结构是否合理?

是 

 

版权和版本声明是否完整?

重要

头文件是否使用了 ifndef/define/endif 预处理块?

 

头文件中是否只存放声明而不存放定义

是 

 

 

 

程序的版式

重要性      

审查项

结论

 

空行是否得体?

是 

 

代码行内的空格是否得体?

是 

 

长行拆分是否得体?

是 

 

“{” “}” 是否各占一行并且对齐于同一列?

是 

重要

一行代码是否只做一件事?如只定义一个变量,只写一条语句。

是 

重要

Ifforwhiledo等语句自占一行,不论执行语句多少都要加 “{}”

重要

在定义变量(或参数)时,是否将修饰符 * 和 & 紧靠变量名?注释是否清晰并且必要?

否 

重要

注释是否有错误或者可能导致误解?

否 

重要

类结构的public, protected, private顺序是否在所有的程序中保持一致?

 

 

 

命名规则 

重要性      

审查项

结论

重要

命名规则是否与所采用的操作系统或开发工具的风格保持一致?

是 

 

标识符是否直观且可以拼读?

是 

 

标识符的长度应当符合“min-length && max-information”原则?

是 

重要

程序中是否出现相同的局部变量和全部变量?

是 

 

类名、函数名、变量和参数、常量的书写格式是否遵循一定的规则?

是 

 

静态变量、全局变量、类的成员变量是否加前缀?

否 

 

 

 

表达式与基本语句 

重要性      

审查项

结论

重要

如果代码行中的运算符比较多,是否已经用括号清楚地确定表达式的操作顺序?

是 

 

是否编写太复杂或者多用途的复合表达式?

否 

重要

是否将复合表达式与真正的数学表达式混淆?

否 

重要

是否用隐含错误的方式写if语句? 例如

 

 

1)将布尔变量直接与TRUEFALSE或者10进行比较。

否 

 

2)将浮点变量用“==”=”与任何数字比较。

否 

 

3)将指针变量用“==”=”NULL比较。

否 

 

如果循环体内存在逻辑判断,并且循环次数很大,是否已经将逻辑判

 

 

断移到循环体的外面?

否 

重要

Case语句的结尾是否忘了加break

否 

重要

是否忘记写switchdefault分支?

否 

重要

使用goto 语句时是否留下隐患? 例如跳过了某些对象的构造、变量的初始化、重要的计算等。

 

 

 

常量 

重要性      

审查项

结论

 

是否使用含义直观的常量来表示那些将在程序中多次出现的数字或字符串?

是 

 

C++ 程序中,是否用const常量取代宏常量?

是 

重要

如果某一常量与其它常量密切相关,是否在定义中包含了这种关系?

否 

 

是否误解了类中的const数据成员?因为const数据成员只在某个对象

否 

 

生存期内是常量,而对于整个类而言却是可变的。

否 

 

 

 

函数设计 

重要性      

审查项

结论

 

参数的书写是否完整?不要贪图省事只写参数的类型而省略参数名字。

是 

 

参数命名、顺序是否合理?

是 

 

参数的个数是否太多?

是 

 

是否使用类型和数目不确定的参数?

否 

 

是否省略了函数返回值的类型?

是 

 

函数名字与返回值类型在语义上是否冲突?

否 

重要

是否将正常值和错误标志混在一起返回?正常值应当用输出参数获得,而错误标志用return语句返回。

否 

重要

在函数体的入口处,是否用assert对参数的有效性进行检查?

否 

重要

使用滥用了assert? 例如混淆非法情况与错误情况,后者是必然存在的并且是一定要作出处理的。

否 

重要

return语句是否返回指向栈内存指针或者引用

否 

 

是否使用const提高函数的健壮性?const可以强制保护函数的参数、返回值,甚至函数的定义体。“Use const whenever you need”

否 

 

 

 

内存管理 

重要性      

审查项

结论

重要

mallocnew申请内存之后,是否立即检查指针值是否为NULL?(防止使用指针值为NULL的内存)

否 

重要

是否忘记为数组和动态内存赋初值?(防止将未被初始化的内存作为右值使用)

否 

重要

数组或指针的下标是否越界?

否 

重要

动态内存的申请与释放是否配对?(防止内存泄漏)

重要

是否有效地处理了内存耗尽问题?

重要

是否修改指向常量的指针的内容?

否 

重要

是否出现野指针?例如(1)指针变量没有被初始化;2)用freedelete释放了内存之后,忘记将指针设置为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)若在逻辑上BA一种,并且A的所有功能和属性对B而言都有意义,则允许B继承A的功能和属性。

无 

 

2)若在逻辑上AB一部分a part of),则不允许BA派生,而是要用A和其它东西组合出B

无 

 

 

 

其它常见问题 

重要性      

审查项

结论

重要

数据类型问题:

 

 

(1)变量的数据类型有错误吗?

否 

 

(2)存在不同数据类型的赋值吗?

 

(3)存在不同数据类型的比较吗?

否 

重要

变量值问题:

 

 

(1)变量的初始化或缺省值有错误吗?

否 

 

(2)变量发生上溢或下溢吗?

否 

 

(3)变量的精度够吗?        

是 

重要

逻辑判断问题:

 

 

(1)由于精度原因导致比较无效吗?

否 

 

(2)表达式中的优先级有误吗?

否 

 

(3)逻辑判断结果颠倒吗?        

否 

重要

循环问题:

 

 

(1)循环终止条件不正确吗?

否 

 

(2)无法正常终止(死循环)吗?

否 

 

(3)错误地修改循环变量吗?

否 

 

(4)存在误差累积吗?        

否 

重要

错误处理问题:

 

 

(1)忘记进行错误处理吗?

无 

 

(2)错误处理程序块一直没有机会被运行?

无 

 

(3)错误处理程序块本身就有毛病吗?如报告的错误与实际错误不一致,处理方式不正确等等。

无 

 

(4)错误处理程序块是马后炮吗?如在被它被调用之前软件已经出错。

无 

重要

文件I/O问题:

 

 

(1)对不存在的或者错误的文件进行操作吗?

否 

 

(2)文件以不正确的方式打开吗?

否 

 

(3)文件结束判断不正确吗?

否 

 

(4)没有正确地关闭文件吗?

否 

四、单元测试
   运用条件覆盖对该代码进行测试,共8种:
   1.算式具有小数,括号,文件输出;
   2.算式不包含小数,括号,文件输出;
   3.算式包含小数,括号,不输出文件;
   4.算式包含小数,输出文件,无括号;
   5.算式包含小数,无括号和文件输出;
   6.算式包含括号,文件输出,无小数;
   7.算式包含括号,无文件和小数;
   8.算式包含文件输出,无括号和小数;
   单元测试结果 
    测试结果正确,符合题目要求,达到讨论的预期;
 

 

五、工作记录

项目

预计

实际

设计时间

2h

1.5h

编码时间

24h

5h

测试时间

1h

2h

代码行数

200

182

测试代码行数

100

189

错误数量

4

2

错误修复时间

3h

1h

开发总时间

24h

10

六、评价及总结
      本次我作为领航员,首先在与驾驶员刘博文沟通与讨论题目时就可以看出刘博文是一位认真、负责、严谨的队友;在他在进行小学四则运算自动生成程序时,可以看出他思维敏捷清晰,代码思路明确,并且我能够很好的与他沟通进行协助;在测试代码时,他的代码基本没有错误,代码清晰明了,并且配有很多注释减轻了我的测试负担;总之,刘博文是一位能力强、认真负责、善于沟通的优秀队友,我很高兴能够与刘博文同学进行结对编程。
        对于此次的结队编程练习,我觉得我受益匪浅。首先我能够从我的结对队友身上学到很多,比如代码的编写、沟通的方法等。通过这次的结对编程不仅提高了我的分析问题能力和测试代码能力,也提高了我的合作编程能力。但是也暴露出我的一些不足,让我充分认识到了自己的不足并且及时进行了改正。我相信在以后的结对编程和团队编程中我将会做的更好,一步一步的提升自己的能力。
 
posted @ 2020-04-29 17:22  计科1701-王宏宇  阅读(167)  评论(0编辑  收藏  举报