一、结对,找到一个伙伴进行结对;
伙伴的博客链接:http://www.cnblogs.com/mainguo/p/8569579.html 【学号15100418】
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
概要部分 |
代码符合需求和规格说明么? |
基本符合 |
代码设计是否考虑周全? |
否 |
|
代码可读性如何? |
一般 |
|
有冗余的或重复的代码吗? |
没有 |
|
代码的每一行都执行并检查过了吗? |
执行、检查过 |
|
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式? |
否 |
有没有硬编码或字符串/数字等存在? |
有 |
|
代码有没有依赖于某一平台? |
有 |
|
有没有无用的代码可以清除? |
无 |
|
代码规范 |
修改的部分符合代码标准么? |
符合 |
修改的部分的设计是否规范? |
基本符合 |
|
具体代码部分 |
数据结构中有没有用不到的元素? |
没有 |
对于调用的外部函数,是否检查了返回值? |
是 |
|
效能 |
代码的效能(Performance)如何? |
一般 |
代码中,特别是循环中是否有明显可优化的部分? |
有 |
|
可读性 |
有没有足够的注释? |
有 |
逻辑是否容易理解? |
否 |
|
段落间和符号旁有没有空白? |
有 |
|
可测试性 |
是否需要更新或创建新的单元测试? |
是 |
总结: 通过本次软件复审,才体会到软件复审的重要性。自己的代码,要自己找错是比较困难的,
因为开发者是对自己的思路费是有信心,觉得是完美的,但是多少都会有些差错,通过复审者的查看,
可以帮你找出不足及有欠考虑的地方,从而提高软件质量。在查看别人代码的同时,我们还可以学到
他人的思路与方法,以及一些细节的问题(注释、缩进等)。对方的代码规范、注释和思路都是不错的,
我觉得比较不完善的地方就是我们可以自己设立空、非空、正确性、不正确行的情况,这样测试的时
候就可以一目了然,还有就是可以把代码的可读性、可复用性提高。