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

    一、结对,找到一个伙伴进行结对;伙伴的博客链接:http://www.cnblogs.com/frx15100213/

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

    概要部分

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

    大致符合

    代码设计是否考虑周全?

    不太周全

    代码可读性如何?

    简单明了

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

    没有

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

    设计规范部分

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

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

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

    没有

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

    没有

    代码规范部分

     

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

    符合

    修改的部分的设计是否规范?

    大致符合规范

    具体代码部分

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

    没有

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

    效能

    代码的效能(Performance)如何?

    良好 

    代码中,特别是循环中是否有明显可优化的部分

    没有

    可读性

     

    有没有足够的注释?

    没有注释

    逻辑是否容易理解?

    段落间和符号旁有没有空白?

    没有

    可测试性

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

    代码复审感想:

      提高代码质量在项目的早期发现缺陷,将损失降至最低评审的过程也是重新梳理思路的过程,双方都加深了对系统的理解促进团队沟通、促进知识共享、共同提高。代码审查本身提高开发者的能力,让其从自身犯过的错误中学习,从他人的思路中学习。在代码审查中如果发现问题,对于问题的发现者来说这是好事,应该予以鼓励。但对于被发现者,我们不主张使用这个方式予以惩罚。软件开发中bug在所难免,过度苛求本身有悖常理。更糟的是,如果造成参与者怕承担责任,不愿意在审查中指出问题,代码审查就没有任何的价值和意义。

  • 相关阅读:
    Flex 学习笔记------组件和视图
    Flex 学习笔记------基于LZMA的文件压缩与上传
    Flex 学习笔记------FLACC & Crossbridge
    Flex 学习笔记------全局事件
    Flex 学习笔记------对象的深层拷贝
    Flex 学习笔记------as 与 js 的通信
    Flex 学习笔记------Local Shared Object 和 Custom Class
    Flex 学习笔记------读取Jpeg图片的width,height和colorSpace
    翻译:eval() 不是魔鬼,只是易被误解
    翻译:javascript 内存管理
  • 原文地址:https://www.cnblogs.com/zwj121/p/8620205.html
Copyright © 2011-2022 走看看