一.结对成员
2班李国相---2班李佳琪
二.结对成员博客链接地址
https://www.cnblogs.com/lietian12345/p/11734865.html
三.代码审查结果表
类别 |
审查项 |
李国相被审查结果 |
李佳琪被审查结果 |
概要部分 |
代码符合需求和规格说明么? |
符合 |
符合 |
代码设计是否考虑周全? |
是 |
是 |
|
代码可读性如何? |
良好 |
易读 |
|
代码是否容易维护? |
容易 |
是 |
|
代码的每一行都执行并检查过了吗? |
是 |
已检查 | |
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式? |
是 |
遵从 |
有没有硬编码或字符串/数字等存在? |
有 |
有 |
|
代码有没有依赖于某一平台? |
没有 |
否 |
|
有没有无用的代码可以清除? |
否 |
否 |
|
代码规范部分 |
修改的部分符合代码标准和风格么? |
比较符合 |
符合 |
具体代码部分 |
错误处理结果 |
良好 |
良好 |
对于调用的外部函数,是否检查了返回值 |
是 |
是 |
|
效能 |
代码的效能如何? |
良好 |
一般 |
循环中是否有明显可优化的部分? |
否 |
有 |
|
可读性 |
代码可读性如何? |
可读性不错 |
易读 |
有没有足够的注释? |
有 |
有 | |
可测试性 |
是否需要更新或创建新的单元测试 |
不需要 |
不需要 |
四.选取李佳琪的项目为基础,进行结对编程。结对项目撰写的博客要求:
1.变量名全部大写,单词间用下划线隔开!
2.包名统一是用小写,点分隔符之间有且有一个自然语义的英语单词!
3.抽象类命名使用Abstract或Base开头;异常类命名使用Exception结尾;测试类命名以它要测试的类的名称开始,以Test结尾!
4.统一使用 4 个空格进行缩进
5.统一使用单行注释 注释标明变量用途以及类所实现的功能等;
五.对于结对编程的感受
结对编程大大提升了编程的效率,两个人之间互补互助,效果远大于1+1,综合两个人的想法代码更加简洁、高效。
六、 增加的需求:(a)考虑数据异常处理问题,如在输入题目生成范围的数据时,输入了“abc“等字符数据,程序如何处理。
(b)增大算式生成数的范围(如整数存不下的数),程序如何处理。
https://github.com/guoxiang-li/text1/blob/master/Test.java