任务说明:
一、结对,找到一个伙伴进行结对;(在作业中标注自己的伙伴博客链接)
王小梅: http://www.cnblogs.com/CURRY-30/p/8570290.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
代码复审检查表 |
||
概要部分 |
代码符合需求和规格说明么? |
基本符合。 |
代码设计是否考虑周全? |
不是很周全。 |
|
代码可读性如何? |
较简单。 |
|
有冗余的或重复的代码吗? |
没有。 |
|
代码的每一行都执行并检查过了吗? |
是的。 |
|
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式? |
是的, |
有没有硬编码或字符串存在? |
没有。 |
|
代码有没有依赖于某一平台? |
没有。 |
|
有没有无用的代码可以清除? |
没有。 |
|
代码规范部分 |
修改的部分符合代码标准和风格么? |
代码基本符合规范标准和风格。 |
具体代码部分 |
数据结构中有没有用不到的元素? |
没有。 |
对于调用的外部函数,是否检查了返回值? |
是的。 |
|
效能 |
代码的效能如何? |
基本达到了任务要求。 |
循环中是否有明显可优化的部分? |
没有明显可优化部分。 |
|
可读性 |
代码可读性如何?有没有足够的注释? |
可读性中等。 |
可测试性 |
是否需要更新或创建新的单元测试? |
需要,代码存在部分问题,测试时会报错但仍能显示结果。 |
8、代码复审感想
个人认为,代码复审这一步在这个流程中是非常重要的,通过复审可以找出很多的错误,也可以完善之前的漏洞。而且复审自己也可以学到很多知识,通过对别人代码的检查,从总学到了别人对代码的理解,取其精华去其糟粕。而且检查出错误的同时,也可以给自己一个教训,以免下次不会犯同样的错误。