一、结对,找到一个伙伴进行结对;(在作业中标注自己的伙伴博客链接)
http://www.cnblogs.com/chenyali/p/8591900.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
1、概要部分
(1)代码符合需求和规格说明么?
大部分符合。
(2)代码设计是否考虑周全?
不够周全,代码中判断出了问题,却不输出错误原因,而是统一输出“邮箱格式错误”。
(3)代码可读性如何?
代码简单明了,可读性良好。
(4)有冗余的或重复的代码吗?
if判断语句冗余。
(5)代码的每一行都执行并检查过了吗?
都检查过了,没什么问题。
2、设计规范部分
(1)设计是否遵从已知的设计模式或项目中常用的模式?
代码遵循了已知的设计模式和在项目中的常用模式。
(2)有没有硬编码或字符串存在?
有数字和字符串的存在。
(3)代码有没有依赖于某一平台?
java没有依赖于某一平台,可移植性高。
(4)有没有无用的代码可以清除?
没有无用的代码可以清除。
3、代码规范部分
大部分还是符合代码标准的。
4、具体代码部分
(1)数据结构中有没有用不到的元素?
数据结构中没有用不到的元素。
(2)对于调用的外部函数,是否检查了返回值?
检查了返回值。
5、效能
(1)代码的效能如何?
达到了具体任务要求的程度。
(2)循环中是否有明显可优化的部分?
有的,if判断语句过多,可以合并一起。
6、可读性
可读性良好。
7、可测试性
是否需要更新或创建新的单元测试?
目前不需要更新或创建新的单元测试。
8、代码复审感想
代码在可复用的基础上,尽量简短一点。因为对于邮箱的判断,一般都用在用户的注册上,所以我觉得只需要用最简单又快捷的代码判断是否正确就可以了。