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

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

伙伴博客链接：http://www.cnblogs.com/zyc1246/p/8558588.html

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

代码复审核查表
概要部分	代码符合需求和规格说明么？	符合需求和规格说明
	代码设计是否考虑周全？	代码设计基本考虑周全，不过缺少一种情况：邮箱格式中同时出现字符和空格
	代码可读性如何？	代码可读性良好
	有冗余的或重复的代码吗?	没有冗余的或重复的代码
	代码的每一行都执行并检查过了吗？	检查过，并不存在问题
设计规范部分	设计是否遵从已知的设计模式或项目中常用的模式？	设计遵从已知的设计模式或项目中常用的模式
	有没有硬编码或字符串/数字等存在？	无硬编码存在，存在字符串和数字
	代码有没有依赖于某一平台？	不依赖于某一平台
	有没有无用的代码可以清除？	没有无用的代码可以清除
代码规范部分	修改的部分符合代码标准和风格么？	代码基本符合标准和风格
具体代码部分	数据结构中有没有用不到的元素？	没有用不到的元素
	对于调用的外部函数，是否检查了返回值	无调用的外部函数
效能	代码的效能如何？	代码的效能良好
	循环中是否有明显可优化的部分？	没有可优化的部分
可读性	代码可读性如何？有没有足够的注释？	代码可读性良好，有基本的注释
可测试性	是否需要更新或创建新的单元测试	不需要更新或创建新的单元测试

 

代码复审感想：

  这个小伙伴的代码简洁易懂，代码风格良好，没有故意打乱代码使别人看不懂，考虑读者的感受。代码不多，能基本达到需求，不过还是有存在有些情况没有考虑到。小伙伴与本人都是做判断邮箱格式是否正确，但是小伙伴的运行环境与本人不同，本人用JAVA运行，小伙伴用WEB运行，通过此次的代码复审，让我多学习了一种判断邮箱格式是否正确的方式。也通过复审小伙伴的代码，意识到自己存在的不足，例如没有做好注释，没有为读代码的人员考虑。代码复审的目的我认为除了检查代码存在的问题，也是一种提升自己在代码规范上的机会。