一、结对,找到一个伙伴进行结对;(在作业中标注自己的伙伴博客链接)
队友:杨艳如
博客地址:http://www.cnblogs.com/Anno-Domini/
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
代码复审检查表 | ||
概要部分 |
代码符合需求和规格说明么? |
基本符合 |
代码设计是否考虑周全? |
否 | |
代码可读性如何? |
一般 | |
有冗余的或重复的代码吗? |
基本没有 | |
代码的每一行都执行并检查过了吗? |
是 | |
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式? |
否 |
有没有硬编码或字符串存在? |
有 | |
代码有没有依赖于某一平台? |
没有 | |
有没有无用的代码可以清除? |
没有 | |
代码规范部分
|
修改的部分符合代码标准么? |
是 |
修改的部分符合代码风格么? |
是 | |
具体代码部分 |
数据结构中有没有用不到的元素? |
无 |
对于调用的外部函数,是否检查了返回值? |
是 | |
效能 |
代码的效能如何? |
一般 |
循环中是否有明显可优化的部分? |
没有 | |
可读性 |
代码可读性如何?有没有足够的注释?
|
代码摆放较乱,但有注释 |
可测试性 |
是否需要更新或创建新的单元测试? |
否 |
代码复审感想
编写代码最重要的是书写规范,能使人看得懂。我的小伙伴,编写代码是规范的,但是她摆放代码的时候,并没有注意到代码全都左对齐了,没有缩进代码。这样看起来代码就很混乱无章,以及她将所有的类名全都没有放进文章中,不过,我的小伙伴在写代码的时候,将代码都注释了,这就使得看代码时更简单了。还有一点不足的是,她是需要自己键盘键入邮箱。若有时间的话,最好还是给自己的文章中的代码重写摆放一遍:缩进用4个空格,行宽限制100字符,多行语句要分行;还有将类名都放进文章中,这样方便自己以后阅读和检查。