一、结对,找到一个伙伴进行结对;(在作业中标注自己的伙伴博客链接)
伙伴博客链接:http://www.cnblogs.com/zyc1246/p/8558588.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
代码复审核查表 | ||
概要部分 | 代码符合需求和规格说明么? | 符合需求和规格说明 |
代码设计是否考虑周全? | 代码设计基本考虑周全,不过缺少一种情况:邮箱格式中同时出现字符和空格
| |
代码可读性如何? | 代码可读性良好 | |
有冗余的或重复的代码吗? | 没有冗余的或重复的代码 | |
代码的每一行都执行并检查过了吗? | 检查过,并不存在问题 | |
设计规范部分 | 设计是否遵从已知的设计模式或项目中常用的模式? | 设计遵从已知的设计模式或项目中常用的模式 |
有没有硬编码或字符串/数字等存在? | 无硬编码存在,存在字符串和数字 | |
代码有没有依赖于某一平台? | 不依赖于某一平台 | |
有没有无用的代码可以清除? | 没有无用的代码可以清除 | |
代码规范部分 | 修改的部分符合代码标准和风格么? | 代码基本符合标准和风格 |
具体代码部分 | 数据结构中有没有用不到的元素? | 没有用不到的元素 |
对于调用的外部函数,是否检查了返回值 | 无调用的外部函数 | |
效能 | 代码的效能如何? | 代码的效能良好 |
循环中是否有明显可优化的部分? | 没有可优化的部分 | |
可读性 | 代码可读性如何?有没有足够的注释? | 代码可读性良好,有基本的注释 |
可测试性 | 是否需要更新或创建新的单元测试 | 不需要更新或创建新的单元测试 |
代码复审感想:
这个小伙伴的代码简洁易懂,代码风格良好,没有故意打乱代码使别人看不懂,考虑读者的感受。代码不多,能基本达到需求,不过还是有存在有些情况没有考虑到。小伙伴与本人都是做判断邮箱格式是否正确,但是小伙伴的运行环境与本人不同,本人用JAVA运行,小伙伴用WEB运行,通过此次的代码复审,让我多学习了一种判断邮箱格式是否正确的方式。也通过复审小伙伴的代码,意识到自己存在的不足,例如没有做好注释,没有为读代码的人员考虑。代码复审的目的我认为除了检查代码存在的问题,也是一种提升自己在代码规范上的机会。