一、结对，找到一个伙伴进行结对；
伙伴的博客链接：http://www.cnblogs.com/zwj121/p/8581014.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审，形成“代码复审检查表”。
代码复审检查表-1
|
概要部分 |
代码符合需求和规格说明么？ |
基本符合 |
|
代码设计是否考虑周全？ |
较为周全 |
|
|
代码可读性如何？ |
规范，容易理解 |
|
|
有冗余的或重复的代码吗? |
有 |
|
|
代码的每一行都执行并检查过了吗？ |
是 |
|
|
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式？ |
否 |
|
有没有硬编码或字符串/数字等存在？ |
有 |
|
|
代码有没有依赖于某一平台? |
没有 |
|
|
有没有无用的代码可以清除？ |
没有 |
|
|
代码规范部分
|
修改的部分符合代码标准么? |
符合 |
|
修改的部分的设计是否规范？ |
基本符合规范 |
|
|
具体代码部分 |
数据结构中有没有用不到的元素？ |
没有 |
|
对于调用的外部函数，是否检查了返回值？ |
是 |
|
|
效能 |
代码的效能（Performance）如何？ |
良好 |
|
代码中，特别是循环中是否有明显可优化的部分 |
是 |
|
|
可读性
|
有没有足够的注释？ |
有 |
|
逻辑是否容易理解？ |
是 |
|
|
段落间和符号旁有没有空白？ |
没有 |
|
|
可测试性 |
是否需要更新或创建新的单元测试？ |
是 |
代码复审感想：在本次对伙伴的单元测试代码复审中，我发现一个小问题就是他的代码中，覆盖用输入来做了。在单元测试中，我认为使用输入方式进行逻辑覆盖，这样不便于自动化测试， 应该放在代码里去做测试。 还有，就是我觉得代码还可以再做进一步简化，不过，整体代码设计逻辑清晰明了；每个功能都有注释，这一点很好。
所以，通过此次代码复审，我感觉进行复审的目的不仅是为了找出代码的错误，更重要的是它还能起到“教育”和“传播知识”的作用。