任务说明:
一、结对,找到一个伙伴进行结对;(在作业中标注自己的伙伴博客链接)
吴丽红: http://www.cnblogs.com/ftfive/p/8591920.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
代码复审检查表 |
||
概要部分 |
代码符合需求和规格说明么? |
基本符合需求和规格说明。 |
代码设计是否考虑周全? |
考虑的不是很周全。 |
|
代码可读性如何? |
代码可读性中等。 |
|
有冗余的或重复的代码吗? |
没有。 |
|
代码的每一行都执行并检查过了吗? |
是的,都执行并检查过。 |
|
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式? |
是的,设计遵从已知模式或常用模式。 |
有没有硬编码或字符串存在? |
没有。 |
|
代码有没有依赖于某一平台? |
没有依赖。 |
|
有没有无用的代码可以清除? |
没有可清除的无用代码。 |
|
代码规范部分 |
修改的部分符合代码标准和风格么? |
代码大部分符合规范标准和风格。 |
具体代码部分 |
数据结构中有没有用不到的元素? |
没有。 |
对于调用的外部函数,是否检查了返回值? |
是的,检查了返回值。 |
|
效能 |
代码的效能如何? |
基本达到了任务要求。 |
循环中是否有明显可优化的部分? |
没有明显可优化部分。 |
|
可读性 |
代码可读性如何?有没有足够的注释? |
可读性中等。 |
可测试性 |
是否需要更新或创建新的单元测试? |
需要,代码存在部分问题,测试时会报错但仍能显示结果。 |
8、代码复审感想
这次代码复审训练时遇到最大的问题就是没有注释,所以审查起来很吃力,关键时刻只能靠请教学霸才能搞懂,这个时候真正感受到了注释的重要性。其次,审查时我发现小伙伴的代码虽长,但满足了判断传入的身份证号码的正确性的基本要求;代码运行时报错,仍需再做修改。总的来讲,通过代码复审这个过程我还是学到了很多:别人的代码就像一面镜子,在给别人指出错误缺陷的同时,我们也能感受到自己的程序所存在的问题和需改进的地方。所以,代码复审很重要,是不可不做的一个关键步骤。