zoukankan      html  css  js  c++  java
  • 20180320作业2:进行代码复审训练

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

    http://www.cnblogs.com/YuzL/p/8566841.html

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

    1、概要部分

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

    符合需求和规格说明

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

    考虑的很周全。

    (3)代码可读性如何?

    代码完整,附有部分注解。

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

    存在部分冗余的或重复的代码需要修改。

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

     执行并逐行检查过。

    2、设计规范部分

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

    遵循。

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

    不存在硬编码或者字符串。

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

    没有。

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

     我觉得没有什么可以清除的了,很清晰的思路。

    3、代码规范部分

    基本符合代码标准和风格,代码可读性高。

    4、具体代码部分

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

    没有。

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

    全部检查过。

    5、效能

    (1)代码的效能如何?

    达到了想要的效果,实现了想实现的功能。

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

    有。

    6、可读性

    可读性强。

    7、可测试性

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

    不需要。

    8、代码复审感想

    在代码复审时,首先要关注到代码规范,代码的规范是很重要的,体现着整个代码的架构,

    其中代码的规范体现在了代码的风格规范和设计规范。规范做的好,复审起来也简单。

    其次再加上一些注解也是很重要的,给审查提供了便利,如有错误也能清楚的查出。

     

  • 相关阅读:
    metasploit 常用命令汇总
    MSF命令 收集
    【转载】虫师『性能测试』文章大汇总
    渗透测试、取证、安全和黑客的热门链接
    Hackers top in China
    国外整理的一套在线渗透测试资源合集[转载]
    Filezilla中文字符文件看不到或显示乱码的解决办法
    Filezilla 多目录的访问设置
    ISAPI在IIS7上的配置
    数据库主体在该数据库中拥有 架构,无法删除解决方法(转)
  • 原文地址:https://www.cnblogs.com/1t-c/p/8613731.html
Copyright © 2011-2022 走看看