任务说明：
一、结对，找到一个伙伴进行结对；（在作业中标注自己的伙伴博客链接）
伙伴博客链接：http://www.cnblogs.com/Sir00437/p/8593937.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审，形成“代码复审检查表”。
|
代码复审核查表 |
||
|
概要部分 |
代码符合需求和规格说明么？ |
基本符合 |
|
代码设计是否考虑周全？ |
否 |
|
|
代码可读性如何？ |
一般 |
|
|
有冗余的或重复的代码吗? |
没有 |
|
|
代码的每一行都执行并检查过了吗？ |
检查了 |
|
|
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式？ |
遵从 |
|
有没有硬编码或字符串/数字等存在？ |
无硬编码的存在，但是存在字符串和数字 |
|
|
代码有没有依赖于某一平台？ |
不依赖 |
|
|
有没有无用的代码可以清除？ |
没有 |
|
|
代码规范部分 |
修改的部分符合代码标准和风格么？ |
基本符合 |
|
具体代码部分 |
数据结构中有没有用不到的元素？ |
没有 |
|
对于调用的外部函数，是否检查了返回值 |
是 |
|
|
效能 |
代码的效能如何？ |
良好 |
|
循环中是否有明显可优化的部分？ |
没有 |
|
|
可读性 |
代码可读性如何？有没有足够的注释？ |
代码可读性不错，没有注释 |
|
可测试性 |
是否需要更新或创建新的单元测试 |
需要 |
8、代码复审感想
这次代码复审我觉得小伙伴的代码格式很规范，代码也是简单易读。但是因为我有些地方不理解，刚好他也没有注释，我就不懂了，不过大致整体还是可以看懂的。我觉得小伙伴在写代码的时候注重代码格式这点是需要我学习的，因为代码格式规范的话，可以让人一目了然，复审时不会那么难审。其次还有一点就是代码有些地方是需要注释的。注释也方便以后复审的同学观看，也便于自己以后的修改。我觉得复审代码这一步是必不可少的，因为往往写代码的人找不到看不出自己代码的问题，可以让别人来帮自己检查不足之处。而且审代码的人也能从中学到一些技巧。