zoukankan      html  css  js  c++  java
  • 进行代码复审训练

    任务说明:

    一、结对,找到一个伙伴进行结对;林燕云http://www.cnblogs.com/lyyhhh/p/8574039.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开始计数?

    答:有错误已改正,字符串的长度是字节的长度,以0开始计数

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

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

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

    答:没有使用

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

    答:在网上找到的,不会导致资源泄漏

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

    答:检查过后没有

    5.效能

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

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

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

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

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

    答:不会超时,已通过百度解决

    6.可读性

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

    答:代码不是很复杂,没有任何注释

    7.可测试性

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

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

    答:不需要

    8、代码复审感想

    在看别人的代码中,遇到的第一个问题就是:没有注释。没有注释给我们阅读带来了很多的麻烦。定义了一堆的变量,完全不知道是做什么用的。对于这样一个小的工程来讲,仔细看一看,还是可以看明白的。但是对于更大的工程呢?这是,突然想起来,我自己写的代码注释也基本为零。。可以想象阅读我们组的代码同学也非常的痛苦。。况且我写的东西很多都是参考网上别人写的,如果没有注释可话,可能我自己过一段时间都搞不明白了。所以,再一次的感受到注释的作用。总的来讲,通过这个过程,还是学到了很多,别人的代码就像一面镜子,在给别人指出错误缺陷的时候,我们也能感受到自己的程序存在的问题。

  • 相关阅读:
    Android学习笔记安卓基础知识
    V8编程1
    dfdf
    NodeJS沙箱
    CGI编程cgihtml库简析
    dsfdsf
    Android学习笔记获取屏幕大小
    Android学习笔记AndroidManifest.xml配置文件详解
    HTTP状态码解析
    简易c++版本日志库
  • 原文地址:https://www.cnblogs.com/yangshuya/p/8610020.html
Copyright © 2011-2022 走看看