一、我的结对成员的博客链接地址: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 |
|
|
|
(3)边界条件,switch分支,循环死循环 |
|
|
|
(4)有无断言(Assert)来保证我们认为得不变条件得到满足 |
|
|
|
(5)对于资源的申请释放,有无泄漏,有无优化空间 |
|
|
|
(6)数据结构中有无用不到的元素 |
|
|
|
5、效能
|
(1)代码效能如何最坏情况怎样 |
|
|
(2)代码(尤其循环)有无可优化 |
|
|
|
(3)系统和网络调用是否超时,如何处理 |
|
|
|
6、可读性
|
代码可读性如何,有无足够注释 |
|
|
7、可测试性
|
代码是否需要更新或创建新的单元测试 |
|
三、结对编程(经过比较选取王祥月成员的项目为基础开始改进)
1、代码编写基本规范
(1)注释规范
1.标注功能块
2.解释简单命名的变量作用
(2)变量命名规范
多采用英文单词
2、描述结对编程的感受
1.张莹感受:
2.王祥月感受:
3、结对场景照片
四、结对项目编程
1、github的连接地址
2、设计说明(改进处)
1.代码规范和设计规范的修改
(a)改进了部分变量名定义
(b)删除了多余数组
2.成项目增加的需求
(a)增加了数据异常处理问题,如在输入题目生成范围的数据时,输入了“abc“等字符数据,程序将退出/默认忽视当前功能。
(b)增大算式生成数的范围,改为long型。超出将退出。