码迷,mamicode.com
首页 > 其他好文 > 详细

进行代码复审训练

时间:2018-03-22 12:40:44      阅读:128      评论:0      收藏:0      [点我收藏+]

标签:情况   处理异常   系统   str   数据   创建   设计规范   span   form   

任务说明:

一、结对,找到一个伙伴进行结对;林坤煌http://www.cnblogs.com/LKH254974541/p/8604791.html

二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。

1.概要部分

1)代码能符合需求和规格说明么?

答:可以符合需求与规格

2)代码设计是否有周全的考虑?

答:不是很周全,修改了几次

3)代码可读性如何?

答:可以简单读懂

4)代码容易维护么?

答:代码容易维护

5)代码的每一行都执行并检查过了吗?

答:检查过了

2.设计规范部分

1)设计是否遵从已知的设计模式或项目中常用的模式?

答:是

2)有没有硬编码或字符串/数字等存在?

答:有一部分

3)代码有没有依赖于某一平台,是否会影响将来的移植(如Win32到Win64)

答:没有依赖,不会影响

4)开发者新写的代码能否用已有的Library/SDK/Framework中的功能实现?在本项目中是否存在类似的功能可以调用而不用全部重新实现?

答:不能,不存在

5)有没有无用的代码可以清除?(很多人想保留尽可能多的代码,因为以后可能会用上,这样导致程序文件中有很多注释掉的代码,这些代码都可以删除,因为源代码控制已经保存了原来的老代码。)

答:有一点无用的代码

3.代码规范部分

1)修改的部分符合代码标准和风格么(详细条文略)?

答:符合代码标准

4.具体代码部分

1)有没有对错误进行处理?对于调用的外部函数,是否检查了返回值或处理了异常?

答:没有对错误进行处理,没有检查返回值和处理异常

2)参数传递有无错误,字符串的长度是字节的长度还是字符(可能是单/双字节)的长度,是以0开始计数还是以1开始计数?

答:有错误,字符串的长度是字节的长度,以1开始计数

3)边界条件是如何处理的?Switch语句的Default是如何处理的?循环有没有可能出现死循环?

答:通过百度查询和书上的知识,或者问同学解决的,循环不会出现死循环

4)有没有使用断言(Assert)来保证我们认为不变的条件真的满足?

答:没有使用

5)对资源的利用,是在哪里申请,在哪里释放的?有没有可能导致资源泄露(内存、文件、各种GUI资源、数据库访问的连接,等等)?有没有可能优化?

答:资源从内存中申请资源,不会导致资源泄露,有可能。

6)数据结构中是否有无用的元素?

答:检查过后没有

5.效能

1)代码的效能(Performance)如何?最坏的情况是怎样的?

答:代码正确,程序运行正常。

2)代码中,特别是循环中是否有明显可优化的部分(C++中反复创建类,C#中string的操作是否能用StringBuilder来优化)?

答:有可优化部分,但是代码正确就没有修改

3)对于系统和网络调用是否会超时?如何处理?

答:不会超时,通过在网上查找处理

6.可读性

代码可读性如何?有没有足够的注释?

答:代码不是很复杂,有一点注释

7.可测试性

代码是否需要更新或创建新的单元测试?

还可以有针对特定领域开发(如数据库、网页、多线程等)的核查表。

答:需要

8、代码复审感想

看别人的代码,最大的感受就是很难看懂,因为注释量很少而且整个代码当中只有一个主函数没有其他的函数,对于阅读代码并且理解代码就会十分的缓慢了,

进行代码复审训练

标签:情况   处理异常   系统   str   数据   创建   设计规范   span   form   

原文地址:https://www.cnblogs.com/362xu/p/8622604.html

(0)
(0)
   
举报
评论 一句话评论(0
登录后才能评论!
© 2014 mamicode.com 版权所有  联系我们:gaon5@hotmail.com
迷上了代码!