zoukankan      html  css  js  c++  java
  • 代码复审核查表

    开发者:05郭蕾   复审者:03王瑞   

    1.概要部分

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

    (2)代码设计没有考虑周全,有很多代码使用的很不合适。

    (3)代码可读性还性吧,不过有的还存在很多问题。

    (4)代码比较容易维护。

    (5)代码的每一行都执行了,都检查了。

    2.设计规范部分

    (1)代码遵循了已知的设计模式和在项目中的常用模式, 学习了很多知识。

    (2)我代码设计中有字符串和数字的存在。

    (3)程序代码没有依赖于某一平台,从Win32移植到Win64上没有出现很大的问题。

    (4)在本程序中类似的功能差不多都可以调用而不用全部重新来实现。

    (5)有无用的代码可以删除,在我的抽签程序中,我对于按组抽签中的添加人员姓名进行了编码,实际上是不需要的,因为一个学号就做够啦,而且还有窗体控件的功能实现。

    3.代码规范部分

    (1)修改的部分有很多地方是符合代码标准和风格的,但是有也有代码是没有符合标准和风格的。

    4.具体代码部分

    (1)在抽签程序中对错误进行了处理,在实现号码滚动的代码上出现了错误,只能实现代码中的数字,而不能随机的进行输入数字。对于调用的外部函数,检查了返回值。

    (2)参数传递无错误,字符串的长度是字节的长度,是双字节,是以1开始计数

    (3)Switch语句的用的很好,没有出现死循环

    (4)没有使用断言(Assert)来保证我们认为不变的条件真的满足

    (5)对资源的利用,是在C#书上借鉴的,内存、文件、各种GUI资源、数据库访问的连接没有可能导致资源泄露,有可能优化

    (6)数据结构中有很的元素是没有用到的。

    5.效能

    (1)代码的效能(Performance)还可以,最坏的情况是程序打不开,不可运行。

    (2)代码中,特别是循环中没有明显可优化的部分。

    (3)对于系统和网络调用会超时,可以等待一会。

    6.可读性

    代码可读性很易懂,没有足够的注释,代码量很少。

    7.可测试性

    代码需要更新和创建新的单元测试。

    可以针对部分功能的实现对代码进行进一步改进或创建新的单元测试。

  • 相关阅读:
    gitlab备份及恢复
    Nginx一:常用命令和配置文件介绍
    SpringTask三:使用SpringBoot
    SpringTask二:注解配置方式
    SpringTask一:xml配置方式
    大坑!maven的web项目初始化错误:sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification ...
    Quartz(5):quartz监听器
    Quartz(4):quartz.properties配置文件介绍
    Quartz(3):simpleTrigger和cronTrigger触发器的介绍
    Quartz(2):JobDetail、JobExectionContext、JobDataMap的介绍
  • 原文地址:https://www.cnblogs.com/GL950225/p/5370222.html
Copyright © 2011-2022 走看看