进行代码复审训练
任务说明:
一、结对,找到一个伙伴进行结对;
杨舒雅:http://www.cnblogs.com/yangshuya/p/8573819.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
1、概要部分
(1)代码符合需求和规格说明么?
答:能符合需求与规格。
(2)代码设计是否考虑周全?
答:考虑相对周全。
(3)代码可读性如何?
答:可读性比较高,相对较简单。
(4)有冗余的或重复的代码吗?
答:有。
(5)代码的每一行都执行并检查过了吗?
答:检查过了。
2、设计规范部分
(1)设计是否遵从已知的设计模式或项目中常用的模式?
答:符合。
(2)有没有硬编码或字符串存在?
答:存在。
(3)代码有没有依赖于某一平台?
答:有。
(4)有没有无用的代码可以清除?
答:有
3、代码规范部分
修改的部分符合代码标准和风格么(详细条文略)?
答:符合。
4、具体代码部分
(1)数据结构中有没有用不到的元素?
答:有啊。
(2)对于调用的外部函数,是否检查了返回值?
答:检查了。
5、效能
(1)代码的效能如何?
答:一般吧。
(2)循环中是否有明显可优化的部分?
答:有。
6、可读性
答:可读性高。
7、可测试性
答:低。
是否需要更新或创建新的单元测试?
答:需要。
8、代码复审感想
答:在代码审查中如果发现问题,对于同伴来说这是好事,虽然软件开发中bug在所难免,过度苛求本身有悖常理,早点发现bug对同伴来说可以早点修改代码,如果我不愿意在审查中指出问题,代码审查就没有任何的价值和意义了。同伴如果能帮我复审代码,能持之以恒,则对我有很大好处。