任务说明:

一、结对,找到一个伙伴进行结对;(在作业中标注自己的伙伴博客链接)

吴丽红: http://www.cnblogs.com/ftfive/p/8591920.html

二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。

代码复审检查表

概要部分

代码符合需求和规格说明么?

基本符合需求和规格说明。

代码设计是否考虑周全?

考虑的不是很周全。

代码可读性如何?

代码可读性中等。

有冗余的或重复的代码吗?

没有。

代码的每一行都执行并检查过了吗?

是的,都执行并检查过。

设计规范部分

设计是否遵从已知的设计模式或项目中常用的模式?

是的,设计遵从已知模式或常用模式。

有没有硬编码或字符串存在?

没有。

代码有没有依赖于某一平台?

没有依赖。

有没有无用的代码可以清除?

没有可清除的无用代码。

代码规范部分

修改的部分符合代码标准和风格么?

代码大部分符合规范标准和风格。

具体代码部分

数据结构中有没有用不到的元素?

没有。

对于调用的外部函数,是否检查了返回值?

是的,检查了返回值。

效能

代码的效能如何?

基本达到了任务要求。

循环中是否有明显可优化的部分?

没有明显可优化部分。

可读性

代码可读性如何?有没有足够的注释?

可读性中等。

可测试性

是否需要更新或创建新的单元测试?

需要,代码存在部分问题,测试时会报错但仍能显示结果。

 

8、代码复审感想

这次代码复审训练时遇到最大的问题就是没有注释,所以审查起来很吃力,关键时刻只能靠请教学霸才能搞懂,这个时候真正感受到了注释的重要性。其次,审查时我发现小伙伴的代码虽长,但满足了判断传入的身份证号码的正确性的基本要求;代码运行时报错,仍需再做修改。总的来讲,通过代码复审这个过程我还是学到了很多:别人的代码就像一面镜子,在给别人指出错误缺陷的同时,我们也能感受到自己的程序所存在的问题和需改进的地方。所以,代码复审很重要,是不可不做的一个关键步骤。

posted on 2018-03-21 20:19  库宝  阅读(106)  评论(0编辑  收藏  举报