任务说明:
一、结对,找到一个伙伴进行结对;(在作业中标注自己的伙伴博客链接)
伙伴吴林萍博客地址:http://www.cnblogs.com/wulinping/p/8594025.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
代码复审核查表 |
||
概要部分 |
代码符合需求和规格说明么? |
基本符合 |
代码设计是否考虑周全? |
否,身份证的前六位号码未进行检验 |
|
代码可读性如何? |
较好,代码简洁明了,有备注,可读性还不错 |
|
有冗余的或重复的代码吗? |
无 |
|
代码的每一行都执行并检查过了吗? |
是的,检查过了 |
|
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式? |
遵从 |
有没有硬编码或字符串/数字等存在? |
有 |
|
代码有没有依赖于某一平台? |
不依赖 |
|
有没有无用的代码可以清除? |
没有 |
|
代码规范部分 |
修改的部分符合代码标准和风格么? |
基本符合 |
具体代码部分 |
数据结构中有没有用不到的元素? |
没有 |
对于调用的外部函数,是否检查了返回值 |
检查了 |
|
效能 |
代码的效能如何? |
良好 |
循环中是否有明显可优化的部分? |
否 |
|
可读性 |
代码可读性如何?有没有足够的注释? |
可读性较好,有进行注释 |
可测试性 |
是否需要更新或创建新的单元测试 |
否 |
代码复审感想:我的小伙伴他的代码简单易懂,同时也有适当的注释,我觉得在复审代码的过程中,也能提升自己的代码阅读理解能力,通过他的代码来发现自身代码的不足。此次代码复审印象深刻的就是编写代码一定要格式规范。代码要简单明了,模块的代码要分清,这样有利于他人查看代码,同时我觉得这样子也方便自己以后修改代码时的方便,方便查看与修改。