20180320作业2:进行代码复审训练

一、结对,找到一个伙伴进行结对

小伙伴:谢建鑫,博客链接:http://www.cnblogs.com/x-j-x/p/8588584.html

二、代码复审核查表

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