一、结对,找到一个伙伴进行结对
小伙伴:谢建鑫,博客链接:http://www.cnblogs.com/x-j-x/p/8588584.html
二、代码复审核查表
代码复审核查表 | ||
概要部分 | 代码符合需求和规格说明么? | 符合 |
代码设计是否考虑周全? | 有待完善 | |
代码可读性如何? | 一般 | |
有冗余的或重复的代码吗? | 无 | |
代码的每一行都执行并检查过了吗 | 是 | |
设计规范部分 | 设计是否遵从已知的设计模式或项目中常用的模式? | 是 |
有没有硬编码或字符串/数字等存在? | 有 | |
代码有没有依赖于某一平台? | 没有 | |
有没有无用的代码可以清除? | 没有 | |
代码规范部分 | 修改的部分符合代码标准和风格么? | 符合 |
具体代码部分 | 数据结构中有没有用不到的元素? | 无 |
对于调用的外部函数,是否检查了返回值 | 已检查返回值 | |
效能 | 代码的效能如何? | 一般 |
循环中是否有明显可优化的部分? | 是 | |
可读性 | 代码可读性如何?有没有足够的注释? | 可读性不高,没有注释 |
可测试性 | 是否需要更新或创建新的单元测试 | 需要 |
代码复审感想:我觉的小伙伴的代码在方法上缺少注释,对于邮箱的判断方法过于单一,用于其他类型的邮箱则判断不起作用,在这方面上小伙伴可以多加完善。同时这次的代码复审也让我意识到了自己的代码的不足之处,以后一定要加以完善。 |