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