一、提供你的结对成员的博客链接地址
孙毓惟的博客地址:
二、结对成员对四则运算项目进行代码互审,并给出代码审查结果表
1.王翯的代码审查表(由孙毓惟完成)
1.概要部分 |
1.1代码符合需求和规格说明吗 |
符合 |
1.2代码设计是否考虑周全 |
是 |
|
1.3代码可读性如何 |
好 |
|
1.4代码容易维护吗 |
容易 |
|
1.5代码每一行都执行并检查过了吗 |
是 |
|
2、代码设计规范
|
2.1设计是否遵从已知的设计模式或项目中常用的模式 |
是 |
2.2有无硬编码或字符串/数字等存在 |
无/否 |
|
2.3代码是否依赖某一平台,是否会影响将来的影响移植 |
否 |
|
2.4开发者新添功能/相似功能是否能用已有来调试 |
是 |
|
2.5有没有无用代码可删除 |
无 |
|
3、代码规范部分
|
修改的代码符合代码规范和风格吗 |
是 |
4、具体代码部分
|
4.1有无对错误进行处理,对于调用外部函数,是否检查了返回值或处理了异常 |
是 |
4.2参数传递是否有错误,字符串长度是字节长度还是字符,是0开始计数还是1开始计数 |
否 从0 |
|
4.3边界条件如何处理,switch语句的default分支如何处理,循环有没有可能出现死循环 |
无 |
|
4.4有无断言(Assert)来保证我们认为得不变条件得到满足 |
无 |
|
4.5对于资源的申请释放,有无泄漏,有无优化空间 |
有 |
|
4.6数据结构中有无用不到的元素 |
无 |
|
5、效能
|
5.1代码效能如何,最坏情况怎样 |
好 |
5.2代码中,特别是循环中有无明显的可优化 |
有 |
|
5.3对于系统和网络的调用是否超时,如何处理 |
否 |
|
6、可读性
|
代码可读性如何,有无足够注释 |
好 有 |
7、可测试性
|
代码是否需要更新或创建新的单元测试 |
是 |
2.孙毓惟的代码审查表(由王翯完成)
1.概要部分 |
1.1代码符合需求和规格说明吗 |
符合 |
1.2代码设计是否考虑周全 |
是 |
|
1.3代码可读性如何 |
好 |
|
1.4代码容易维护吗 |
容易 |
|
1.5代码每一行都执行并检查过了吗 |
是 |
|
2、代码设计规范
|
2.1设计是否遵从已知的设计模式或项目中常用的模式 |
是 |
2.2有无硬编码或字符串/数字等存在 |
无/否 |
|
2.3代码是否依赖某一平台,是否会影响将来的影响移植 |
否 |
|
2.4开发者新添功能/相似功能是否能用已有来调试 |
是 |
|
2.5有没有无用代码可删除 |
无 |
|
3、代码规范部分
|
修改的代码符合代码规范和风格吗 |
是 |
4、具体代码部分
|
4.1有无对错误进行处理,对于调用外部函数,是否检查了返回值或处理了异常 |
是 |
4.2参数传递是否有错误,字符串长度是字节长度还是字符,是0开始计数还是1开始计数 |
否 从0 |
|
4.3边界条件如何处理,switch语句的default分支如何处理,循环有没有可能出现死循环 |
无 |
|
4.4有无断言(Assert)来保证我们认为得不变条件得到满足 |
无 |
|
4.5对于资源的申请释放,有无泄漏,有无优化空间 |
有 |
|
4.6数据结构中有无用不到的元素 |
无 |
|
5、效能
|
5.1代码效能如何,最坏情况怎样 |
好 |
5.2代码中,特别是循环中有无明显的可优化 |
有 |
|
5.3对于系统和网络的调用是否超时,如何处理 |
否 |
|
6、可读性
|
代码可读性如何,有无足够注释 |
好 有 |
7、可测试性
|
代码是否需要更新或创建新的单元测试 |
是
|
三、选取其中一个成员的项目为基础,进行结对编程
1.代码编写基本规范
缩进:4个字符
行宽:100个字符
括号:恰当使用()
断行与空白的{}行:使用得当
分行:一行一条语句
命名:见名知意,适量
大小写:区分
注释:为代码添加合适的注释
2.描述结对编程的感受:
互相鼓励,不容易沮丧,因为在面对问题的时候,会有人一起分担,共同尝试新的策略。互相监督,不容易偷懒,两个人一起工作需要互相配合,如果想偷懒去干别的,就会拖延工作进度。互相学习编程技巧:在编程中,相互讨论,可以更快更有效地解决问题,互相请教对方,可以得到能力上的互补。多双眼睛,少点 bug:两人互相监督工作,可以增强代码和产品质量,并有效的减少 BUG。
3.结对场景照片
四、结对项目编程要求:
1.GitHub链接地址:https://github.com/SunYuwei19980504/jisuan
2.增加的需求:
(a)考虑数据异常处理问题:
如在输入题目生成范围的数据时,输入了“abc“等字符数据,程序会提示输入错误。
(b)增大算式生成数的范围:
如整数存不下的数,程序会提示输入错误。