一、我的结对成员【张莹】的博客链接地址:
https://www.cnblogs.com/delephine/p/11716046.html
二、结对成员对四则运算项目代码审查结果表:
部分 |
内容 |
张莹审查结果 |
王祥月审查结果 |
1、概要部分 |
(1)代码符合需求和规格说明吗 |
符合 |
符合 |
(2)代码设计是否考虑周全 |
是 |
是 |
|
(3)代码可读性如何 |
好 |
好 |
|
(4)代码容易维护吗 |
容易 |
容易 |
|
(5)代码每一行都执行并检查过了吗 |
是 |
是 |
|
2、代码设计规范
|
(1)设计是否遵从设计模式 |
是 |
是 |
(2)有无硬编码或字符串/数字等存在 |
无 |
无 |
|
(3)是否依赖某平台影响移植 |
否 |
否 |
|
(4)开发者新添功能/相似功能是否能用已有来调试 |
是 |
是 |
|
(5)有无无用代码可删除 |
无 |
无 |
|
3、代码规范部分 |
符合代码规范和风格吗 |
是 |
符合 |
4、具体代码部分
|
(1)有无对错误进行处理,对于调用外部函数,是否检查了返回值或处理了异常 |
是 |
是 |
(2)参数传递是否有错误,字符串长度是字节长度还是字符,计数是0开始还是1 |
否,从0 |
无,0开始 |
|
(3)边界条件,switch分支/循环死循环 |
无 |
无 |
|
(4)有无断言(Assert)来保证我们认为得不变条件得到满足 |
无 |
有 |
|
(5)对于资源的申请释放,有无泄漏,有无优化空间 |
有 |
有 |
|
(6)数据结构中有无用不到的元素 |
无 |
无 | |
5、效能
|
(1)代码效能如何,最坏情况怎样 |
好 |
好 |
(2)代码(尤其循环)有无可优化 |
有 |
有 |
|
(3)系统和网络调用是否超时如何处理 |
否 |
否 |
|
6、可读性 |
代码可读性如何,有无足够注释好 |
好,有 |
好,有注释 |
7、可测试性 |
代码是否需要更新或创建新的单元测试 |
是 | 否 |
三、结对编程(经过比较选取王祥月成员的项目为基础开始改进)
1、代码编写基本规范
(1)注释规范
1.标注功能块
2.解释简单命名的变量作用
(2)变量命名规范
多采用英文单词
2、描述结对编程的感受
1.张莹感受:作为一名Java菜鸟,在王祥月大佬的帮助和鼓励下完成了基本的程序编写、修改和审核不禁激动地内牛满面,
同时意识到由于自身知识储备的不足,导致整个团队进度速度变慢,因此急需完备自己。
但是两人合作确实在某些方面很大程度上提高了效率,汲取双方意见出炉的代码质量也得到了提升。
结对编程是一个相互学习、相互磨合的渐进过程,在度过了学习阶段后,结对编程小组的开发质量、开发时间通常比两人单独开发有明显的改善。
2.王祥月感受:合作非常愉快,和队友交流顺畅,队友作一名c语言大哥个人水平高。
在经过愉快的合作我们使程序更加完善,学习了解到了结对编程复审流程和效果。收获颇丰。
3、结对场景照片
四、结对项目编程
1、github的连接地址:
https://github.com/wangxiangyue4/count?files=1
2、设计说明(改进处)
1.代码规范和设计规范的修改
(a)改进了部分变量名定义
(b)删除了多余数组
2.成项目增加的需求
(a)增加了数据异常处理问题,如在输入题目生成范围的数据时,输入了“abc“等字符数据,程序将退出/默认忽视当前功能。
(b)增大算式生成数的范围,改为long型。超出将退出。