一、结对,找到一个伙伴进行结对:陈学敏,伙伴博客链接:http://www.cnblogs.com/chenxuemin/p/8594316.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
1、概要部分
(1)代码符合需求和规格说明么?
答:符合用户需求和规格
(2)代码设计是否考虑周全?
答:基本考虑周全
(3)代码可读性如何?
答:可读性较差
(4)有冗余的或重复的代码吗?
答:有重复的代码
(5)代码的每一行都执行并检查过了吗?
答:检查过了
2、设计规范部分
(1)设计是否遵从已知的设计模式或项目中常用的模式?
答:设计遵循已知的设计模式
(2)有没有硬编码或字符串存在?
答:有存在硬编码和字符串
(3)代码有没有依赖于某一平台?
答:没有依赖某一平台
(4)有没有无用的代码可以清除?
答:有可以清除的代码
3、代码规范部分
答:代码整体比较规范,但有几行代码还需要规范整理
4、具体代码部分
(1)数据结构中有没有用不到的元素?
答:数据结构中没有用不到的元素
(2)对于调用的外部函数,是否检查了返回值?
答:没有调用外部函数
5、效能
(1)代码的效能如何?
答:代码效能良好
(2)循环中是否有明显可优化的部分?
答:代码中有明显可以优化的部分
6、可读性
答:代码较为复杂,可读性较差
7、可测试性
是否需要更新或创建新的单元测试?
答:不需要创建新的单元测试
8、代码复审感想
答:通过复审伙伴的代码,我发现了几处可以优化的地方,其一,代码行数将近两百,过于冗长,而且代码中注释不多,也不够详细,无法直观理解,很多地方不知道如何分析入手;其二,部分代码冗余重复,可以精简一些代码;其三,所有代码都在主函数中,如果改成外部函数,主函数需要时随时调用,不会显得头重脚轻,也会增加可读性。通过复审伙伴的代码,我不但发现伙伴的一些代码问题,而且我也发现了一些自己的代码问题,别人犯的错误自己也会犯,通过这次复审代码,我审视了自己的代码风格,反省了自己的错误,通过这样的方式,我对代码又有了新的理解。