任务说明:
一、结对,找到一个伙伴进行结对;(在作业中标注自己的伙伴博客链接)
伙伴涂招仁博客:http://www.cnblogs.com/tusi/p/8584248.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
代码复审核查表 | ||
概要部分 | 代码符合需求和规格说明么? | 基本符合 |
代码设计是否考虑周全? | 还算周全 | |
代码可读性如何? | 相对简单易读 | |
有冗余的或重复的代码吗? | 无 | |
代码的每一行都执行并检查过了吗? | 检查过,没有发现问题 | |
设计规范部分 | 设计是否遵从已知的设计模式或项目中常用的模式? | 否 |
有没有硬编码或字符串/数字等存在? | 有硬编码或字符串/数字等存在 | |
代码有没有依赖于某一平台? | 没有 | |
有没有无用的代码可以清除? | 无 | |
代码规范部分 | 修改的部分符合代码标准和风格么? | 基本符合 |
具体代码部分 | 数据结构中有没有用不到的元素? | 没有 |
对于调用的外部函数,是否检查了返回值 | 是 | |
效能 | 代码的效能如何? | 可正常运行实现功能 |
循环中是否有明显可优化的部分? | 有 | |
可读性 | 代码可读性如何?有没有足够的注释? | 没有详细的注释,但代码简单易读 |
可测试性 | 是否需要更新或创建新的单元测试 | 需要 |
代码复审感想:伙伴的代码问题基本不大,也可以通过NetBeansIDE实现,但是他是通过从键盘上一个一个输入邮箱来判断其是否正确,这不仅效率低,也达不到老师的本意。check_mail应该是直接在JAVA中一次性写完要判断的邮箱,然后通过代码实现判断;还有一点就是他在写代码的时候并没有详细的注释,这会让大部分人看的吃力,这两点是我伙伴需要改进的地方。