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

    一、结对,找到一个伙伴进行结对;(在作业中标注自己的伙伴博客链接)

    我的伙伴:王振强 ,博客链接:http://www.cnblogs.com/wzqking/p/8579348.html

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

    代码复审核查表 

     

     

     

     

    概要部分

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

    基本符合

    代码设计是否考虑周全?

    代码可读性如何?

    可读性还行

    有冗余的或重复的代码吗?

    没有,代码简洁易懂

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

    检查过了

     

     

     

    设计规范部分

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

    遵从

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

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

    没有

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

    没有

    代码规范部分

    修改的部分符合代码标准和风格么?

    基本符合

     

    具体代码部分

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

    没有

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

    检查了

     

    效能

    代码的效能如何?

    可执行结果

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

    可读性

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

    可读性较好,有很多的注释

    可测试性

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

      代码复审感想:他的代码还是比较好理解的,对于部分代码有着注释还是比较简单易懂的,而且对于他的代码,我觉得功能齐全,把需要的不需要的都包含了,可以说是很完整的一份作业,毕竟我们都是新手,对于他的有些代码,我还是看不太懂,请教了一下同学,考虑的判断条件要详尽,因为代码格式规范的话,可以让人一目了然,复审时不会那么难审。

  • 相关阅读:
    Linux下端口被占用确认
    Debuggex – 超好用的正则表达式可视化调试工具
    URL最大长度
    无需重启 修改计算机名直接生效
    UDP穿越NAT原理(p2p)
    gdb中信号
    锁 (读写锁优先级 写饥饿) (锁竞争引发的高系统调用)
    Loopback接口用途---用作管理地址。
    C/C++调试:gdbserver的简单使用
    总结一下NDK crash排查步骤
  • 原文地址:https://www.cnblogs.com/940wxd/p/8647386.html
Copyright © 2011-2022 走看看