代码复审训练——15100216

 

一、结对，找到一个伙伴进行结对：陈学敏，伙伴博客链接：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、代码复审感想

答：通过复审伙伴的代码，我发现了几处可以优化的地方，其一，代码行数将近两百，过于冗长，而且代码中注释不多，也不够详细，无法直观理解，很多地方不知道如何分析入手；其二，部分代码冗余重复，可以精简一些代码；其三，所有代码都在主函数中，如果改成外部函数，主函数需要时随时调用，不会显得头重脚轻，也会增加可读性。通过复审伙伴的代码，我不但发现伙伴的一些代码问题，而且我也发现了一些自己的代码问题，别人犯的错误自己也会犯，通过这次复审代码，我审视了自己的代码风格，反省了自己的错误，通过这样的方式，我对代码又有了新的理解。