zoukankan      html  css  js  c++  java
  • 4月8日作业 代码审核复查

    开发者:33郭强

    复审者:04许奥博

    1.概要部分

    (1)所编写的代码基本符合需求和规格说明。

    (2)所编写的代码设计考虑并不周全,开发流程略显混乱。

    (3)所编写的代码简洁明了,能让人看懂,也能看懂设计思路。

    (4)所编写的代码比较容易维护,接口设计比较完美。

    (5)所编写的代码的每一行没有单独执行并检查过,但未出现明显错误。

    2.设计规范部分

    (1)遵从已知的设计模式或项目中常用的模式,并从中学习和借鉴了一些知识点。

    (2)有硬编码或字符串/数字等存在。

    (3)代码不依赖于某一平台,在XP编写完成,使用win8系统依然可以打开,所以不会有移植问题。

    (4)在本项目中类似的功能可以调用而不用全部重新实现。

    (5)没有无用的代码可以清除。每段代码都有它可以实现的功能,且不可删除。

    3.代码规范部分

    (1)修改的部分符合代码标准和风格,按照需求和规格修改,使其达到标准,简单明了,通俗易懂。

    4.具体代码部分

    (1)在编程过程中遇到了许多错误,我们对错误进行了详细的处理,并检查了返回值。

    (2)长度是以1开始计数的,在参数传递上没有问题。

    (3)循环处理的很好,没有出现死循环。

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

    (5)没有可能导致资源泄露,有继续优化的可能性。

    (6)数据结构中的元素只是大部分用到了。

    5.效能

    (1)代码的效能较好,最坏的情况是是否可重复部分的功能不能很好地实现。

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

    (3)系统和网络调用时不会产生超时。

    6.可读性

    代码可读性比较好,有少量关键注释。

    7.可测试性

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

  • 相关阅读:
    【数据库】-数据库显示不允许保存更改
    C#编程:从控制台读取数字的两种方式
    【终结版】C#常用函数和方法集汇总
    运行supervisord -c /etc/supervisor/supervisord.conf 出错,解决办法
    supervisord监控服务必备命令
    【持续更新中···】Linux下的小技巧
    运行supervisorctl reload报错解决方法
    ASP.NET Core 发布 centos7 配置守护进程
    【C++】std::是什么?
    DNS查询相关
  • 原文地址:https://www.cnblogs.com/A4L0/p/5367339.html
Copyright © 2011-2022 走看看