zoukankan      html  css  js  c++  java
  • 代码复审训练——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、代码复审感想

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

  • 相关阅读:
    逃避是解决不了问题
    div包含table
    change is possible
    POI修改Excel
    要做的事情太多,把手边的事情做好。
    不自信,不努力
    添加省略号
    C#基类库
    如何做到在页面POSTBACK刷新后,使LISTBOX的滚动条仍然保持上次的位置
    安装dedecms后台登录空白的解决方法
  • 原文地址:https://www.cnblogs.com/kaixuannihao/p/8617553.html
Copyright © 2011-2022 走看看