20180320作业2：进行代码复审训练

 任务说明：

一、结对，找到一个伙伴进行结对；（在作业中标注自己的伙伴博客链接）

http://www.cnblogs.com/yang1com/p/8594704.html

 

二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审，形成“代码复审检查表”。

1、概要部分

(1)代码符合需求和规格说明么？

基本符合

(2)代码设计是否考虑周全？

不周全，当没有输入的时候和邮箱的长度未确定的时候都没有考虑

(3)代码可读性如何？

代码简单明了，可读性较好

(4)有冗余的或重复的代码吗?

在条件判断的地方，可能合起来并其中有冗余。

(5)代码的每一行都执行并检查过了吗？

 检查，如果if判断语句中用if(){...}else if(){...}会更加简洁明了，更规范。

 

2、设计规范部分

(1)设计是否遵从已知的设计模式或项目中常用的模式？

设计过程遵循从已知的设计模式和项目中常用的模式

(2)有没有硬编码或字符串存在？

有字符串和数字的存在

(3)代码有没有依赖于某一平台？

没有，html文件，直接用浏览器打开即可，可移植性高，可以跨平台运行。

(4)有没有无用的代码可以清除？

 无

 

3、代码规范部分

大体符合规范

 

4、具体代码部分

(1)数据结构中有没有用不到的元素？

无

(2)对于调用的外部函数，是否检查了返回值？

没有调用外部函数

 

5、效能

(1)代码的效能如何？

达到项目的要求

(2)循环中是否有明显可优化的部分？

有，可以将判断语句合并

 

6、可读性

基本读懂

7、可测试性

测试失败

8、代码复审感想

代码需要添加注释以提高可读性，考虑的判断条件要详尽。