一、结对，找到一个伙伴进行结对；伙伴的博客链接：http://www.cnblogs.com/Chenjx-666/p/8558209.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审，形成“代码复审检查表”。
|
概要部分 |
代码符合需求和规格说明么？ |
大致符合 |
|
代码设计是否考虑周全？ |
不周全 |
|
|
代码可读性如何？ |
简单 |
|
|
有冗余的或重复的代码吗? |
无 |
|
|
代码的每一行都执行并检查过了吗？ |
是 |
|
|
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式？ |
否 |
|
有没有硬编码或字符串/数字等存在？ |
有 |
|
|
代码有没有依赖于某一平台? |
有 |
|
|
有没有无用的代码可以清除？ |
无 |
|
|
代码规范部分
|
修改的部分符合代码标准么? |
符合 |
|
修改的部分的设计是否规范？ |
大致符合规范 |
|
|
具体代码部分 |
数据结构中有没有用不到的元素？ |
无 |
|
对于调用的外部函数，是否检查了返回值？ |
是 |
|
|
效能 |
代码的效能（Performance）如何？ |
良好 |
|
代码中，特别是循环中是否有明显可优化的部分 |
无 |
|
|
可读性
|
有没有足够的注释？ |
无注释 |
|
逻辑是否容易理解？ |
是 |
|
|
段落间和符号旁有没有空白？ |
无 |
|
|
可测试性 |
是否需要更新或创建新的单元测试？ |
否 |
代码复审感想：
通过分析该代码相对简单易懂，代码相对较少，出现问题更易查找。尽管部分特殊情况没有考虑周全，相对而言在某一方面上来说代码审查本身提高开发者的能力，让其从自身犯过的错误中学习，从他人的思路中学习。在代码审查中也意识到自己存在的不足，例如没有做好注释，没有为读代码的人员考虑。代码复审的目的我认为除了检查代码存在的问题，也是一种提升自己在代码规范上的机会。尤其在软件开发中bug在所难免，过度苛求本身有悖常理。发现问题要立刻指出问题，不然代码审查就没有任何的价值和意义。