结对成员链接地址
付正印的博客地址
代码审核表
付正印的代码审核表
-
代码符合需求和规格说明吗
需求 结果 删除重复 没有完成 结果存储到本地 完成 运算数为负数 完成 控制参数 完成 乘除法 完成 -
代码设计是否考虑周全
情况 结果 代码设计 设计性差,全部代码都在 main
中完成,有很多的重复代码 -
代码可读性
问题 结果 代码可读性 良好,包含注释,部分缩进有问题 -
代码容易维护吗
情况 结果 维护性 维护性差,在一个函数 main
中完成,在一个函数中有太多的if
和switch
语句了 -
代码的执行
情況 结果 代码覆盖率 100% 代码错误情况 无 -
面对错误输入的处理
情况 结果 如何执行 输入错误数据类型 会抛出异常 需要输入一个数字时,输入一个字母 输入错误字符 无 无 -
资源是否释放
涉及到的资源 结果 Scanner 未释放 PrintStream 未释放
李禹澄的代码审核表
-
代码符合需求和规格说明吗
需求 结果 删除重复 完成 结果存储到本地 完成 运算数为负数 完成 控制参数 完成 乘除法 完成 特殊功能 完成 -
代码设计是否考虑周全
情况 结果 代码设计 设计性好,便于维护 -
代码可读性
问题 结果 代码可读性 良好,包含注释,命名有一定规范 -
代码容易维护吗
情况 结果 维护性 维护性好,主要功能都使用了类和设计模式。但是主客户端的if else-if 语句过多导致主客户端不便于维护 -
代码的执行
情況 结果 代码覆盖率 100% 代码错误情况 无 -
面对错误输入的处理
情况 结果 如何执行 输入错误数据类型 会抛出异常 需要输入一个数字时,输入一个字母 输入错误字符 会整体退出程序,不便于使用 在需要y或者n 的判断是输入其他字符 -
资源是否释放
涉及到的资源 结果 Scanner 完成释放 FileWriter 完成释放
结对编程
选择付正印的项目
-
代码编写规范
类型 规范 变量 第一个单词小写,后面的单词首字母大写 常量 单词全部小写,中间使用下划线链接 注释 字段以及语句的注释在当前行右面;函数的注释使用javadoc 注释;流程的注释在if 等的右边 -
结对编程的感受
通过结对编程可以互相讨论,可能想法不一样,但是只要能够良好沟通,最后达成同意,会很便于后面的合作的。
-
场景图片
结对编程的结果
修改的内容
-
减少主客户端的if 与while 的嵌套,减少else-if 的使用,将生成结果的while 移出来,减少复杂度
-
在需要输入数字时,用户输入错误时,通过
try{}catch(){}
捕获异常,然后重新开始。 -
在需要输入数字过大,导致结果超出范围,通过
try{}catch(){}
捕获异常,然后抛弃此次结果,重新计算(属于更新的需求)。 -
部分内容(比如文件输出的部分)放到独立的函数中(属于更新的需求)。