进行代码复审训练

一、结对，找到一个伙伴进行结对；

  伙伴的博客链接：http://www.cnblogs.com/zwj121/p/8581014.html

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

                                                代码复审检查表-1

概要部分	代码符合需求和规格说明么？	基本符合
	代码设计是否考虑周全？	较为周全
	代码可读性如何？	规范，容易理解
	有冗余的或重复的代码吗?	有
	代码的每一行都执行并检查过了吗？	是
设计规范部分	设计是否遵从已知的设计模式或项目中常用的模式？	否
	有没有硬编码或字符串/数字等存在？	有
	代码有没有依赖于某一平台?	没有
	有没有无用的代码可以清除？	没有
代码规范部分	修改的部分符合代码标准么?	符合
	修改的部分的设计是否规范？	基本符合规范
具体代码部分	数据结构中有没有用不到的元素？	没有
	对于调用的外部函数，是否检查了返回值？	是
效能	代码的效能（Performance）如何？	良好
	代码中，特别是循环中是否有明显可优化的部分	是
可读性	有没有足够的注释？	有
	逻辑是否容易理解？	是
	段落间和符号旁有没有空白？	没有
可测试性	是否需要更新或创建新的单元测试？	是

 

代码复审感想：在本次对伙伴的单元测试代码复审中，我发现一个小问题就是他的代码中，覆盖用输入来做了。在单元测试中，我认为使用输入方式进行逻辑覆盖，这样不便于自动化测试， 应该放在代码里去做测试。 还有，就是我觉得代码还可以再做进一步简化，不过，整体代码设计逻辑清晰明了；每个功能都有注释，这一点很好。

 所以，通过此次代码复审，我感觉进行复审的目的不仅是为了找出代码的错误，更重要的是它还能起到“教育”和“传播知识”的作用。