第四次博客作业-结对项目
1.结对成员的博客链接地址
王海楠同学链接:https://www.cnblogs.com/wanghainan/p/11710193.html
2.结对成员对四则运算项目代码互审的代码审查结果表
|
核查内容 |
王海楠 |
王佳琦 |
1.概要部分 |
1)代码符合需求和规格说明么? |
符合 |
符合 |
2)代码设计是否考虑周全? |
是 |
是 |
|
3)代码可读性如何? |
可读性高 |
可读性高 |
|
4)代码容易维护么? |
容易 |
容易 |
|
5)代码的每一行都执行并检查过了吗? |
是 |
是 |
|
2.设计规范部分 |
1)设计是否遵从已知的设计模式或项目中常用的模式? |
是 |
是 |
2)有没有硬编码或字符串/数字的存在? |
有 |
有 |
|
3)代码有没有依赖于某一平台,是否会影响将来的移植 |
没依赖平台 否 |
没依赖平台 否 |
|
4)开发者新写的代码能否用已有的Library/SDK/Framework中的功能实现?在本项目中是否存在类似的功能可以调用而不用全部重新实现? |
否 否 |
否 否 |
|
5)有没有无用的代码可以清除? |
没有 |
没有 |
|
3.代码规范部分 |
修改的部分符合代码标准和风格吗? |
符合 |
符合 |
4.具体代码部分 |
1)有没有对错误进行处理?对于调用的外部函数,是否检查了返回值或处理了异常? |
有 是 |
有 是 |
2)参数传递有无错误,字符串长度是字节的长度还是字符的长度,是以0开始计数还是以1开始计数? |
无 0 |
无 0 |
|
3)边界条件是如何处理的?switch语句的default分支是如何处理的?循环有没有可能出现死循环? |
从一开始就确定边界条件 遇到default结束switch循环 若没有default则当没有合适的case时结束 没可能 |
先确定边界条件
没有default时,则当没有合适的case时结束 没可能 |
|
4)有没有使用断言来保证我们认为不变的条件真的得到满足? |
没有 |
没有 |
|
5)对资源的利用,是在哪里申请,在哪里释放的?有无可能存在资源泄露?有没有优化空间? |
|
|
|
6)数据结构中有没有用不到的元素? |
没有 |
有,但后期整合时删除了 |
|
5.效能 |
1)代码的效能如何?最坏的情况是怎样的? |
效能较好 |
效能较好 |
2)代码中,特别是循环中是否有明显可优化的部分? |
否 |
否 |
|
3)对于系统和网络的调用是否会超时?如何处理? |
不会 |
没有进行网络调用 |
|
6.可读性 |
代码可读性如何?有没有足够的注释? |
可读性高 注释完整详细 |
可读性较好 |
7.可测试性 |
代码是否需要更新或创建的单元测试?针对特定领域的开发,可以整理专门的核查表? |
不需要 |
否 |
3.代码编写基本规范
1)命名规范:
Package的命名:
Package的名字每个单词必须由大写字母开头而其他字母都以小写单词组成;
Class的命名:
Class的名字每个单词必须由大写字母开头而其他字母都小写的单词组成;
Class 成员的命名:
变量、方法、属性:大小写混排的单词组成,首字母小写;
Static Final 变量的命名:
Static Final常量:大写单词组成,单词之间使用“_”连接;
参数的命名:
参数的名字必须和变量的命名规范一致;
2)Java 文件样式
所有的 Java(*.java) 文件都必须遵守如下的样式规则;
3)代码样式
代码应该用 unix 的格式,而不是 windows 的(比如:回车变成回车+换行);
4)注释
源程序有效注释量必须在20%以上。
说明:注释的原则是有助于对程序的阅读理解,在该加的地方都加了,注释不宜太多也不能太少,注释语言必须准确、易懂、简洁;
常规注释标记说明
注释起始为“/**…*/”,注释文档的第一条为总结性语句,可在注释文档中使用HTML的标签语句,但要杜绝使用“HL”“HR”标签;
4.描述结对编程的感受
结对编程就是两个人一起编程,而两个人完美的配合就可以达到1+1=2甚至>2的效果,在这个过程中,两个人需要互相了解、相互配合、积极参与;
两个人想法的不同可以互补然后得出好的结论,再运用到代码的设计中去,完善它、优化它,得到更好的结果;
两个人相互配合然后可以查缺补漏,进行一定的互补,相互督促,共同进步;
在效率上有一定的提高,完好的分工合作可以减轻对方的负担;
5.结对场景照片
6.增加的需求:
(a)当在输入题目生成范围的数据时输入“abc”等字符时,异常处理,报错;
(b)增大算式生成数的范围;
7.Github链接地址
posted on 2019-10-22 18:46 Jeacky-wang 阅读(153) 评论(0) 编辑 收藏 举报