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

    任务说明:

    一、结对,找到一个伙伴进行结对;

    杨舒雅:http://www.cnblogs.com/yangshuya/p/8573819.html

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

    1、概要部分

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

    答:能符合需求与规格。

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

    答:考虑相对周全。

    (3)代码可读性如何?

    答:可读性比较高,相对较简单。

    (4)有冗余的或重复的代码吗?

    答:有。

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

     答:检查过了。

    2、设计规范部分

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

    答:符合。

    (2)有没有硬编码或字符串存在?

    答:存在。

    (3)代码有没有依赖于某一平台?

    答:有。

    (4)有没有无用的代码可以清除?

     答:有

    3、代码规范部分

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

    答:符合。

    4、具体代码部分

    (1)数据结构中有没有用不到的元素?

    答:有啊。

    (2)对于调用的外部函数,是否检查了返回值?

    答:检查了。

    5、效能

    (1)代码的效能如何?

    答:一般吧。

    (2)循环中是否有明显可优化的部分?

    答:有。

    6、可读性

    答:可读性高。

    7、可测试性

    答:低。

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

    答:需要。

    8、代码复审感想

    答:在代码审查中如果发现问题,对于同伴来说这是好事,虽然软件开发中bug在所难免,过度苛求本身有悖常理,早点发现bug对同伴来说可以早点修改代码,如果我不愿意在审查中指出问题,代码审查就没有任何的价值和意义了。同伴如果能帮我复审代码,能持之以恒,则对我有很大好处。

  • 相关阅读:
    Python开发之路
    openstack系列阅读目录
    linux基础
    PEP8规范
    我真的还是18岁的那个我
    为什么很多IT公司不喜欢进过培训机构的人呢?
    GRE与VXLAN
    VLAN模式
    网络虚拟化基础一:linux名称空间Namespaces
    四:OVS+GRE之网络节点
  • 原文地址:https://www.cnblogs.com/lyyhhh/p/8618615.html
Copyright © 2011-2022 走看看