zoukankan      html  css  js  c++  java
  • #个人作业Week2——结对编程对象代码复审

    General

    1. 代码能够正确运行,能够正确生成指定数量的题目和答案,并且能够对给出的题目和答案文件进行比对,输出结果。
    2. 代码没有非常复杂的逻辑,比较容易理解,但是在缺少注释的情况下有部分代码需要较长时间的分析才能理解。
    3. 建议将Program类中的方法改为非static类型的,有利于提高程序的封装性。

    Security

    1. 对于控制台输入的格式检查不到位,例如:如果参数-n中出现字母,或者答案文件中出现字母,在调用int.Parse()时会出现程序崩溃的错误。
    2. 很多函数都没有对输入的参数进行格式检查,比如说Number类中的四则运算函数,如果调用方法时输入的参数other不合法时,容易产生除数为零等问题。

    Documentation

    1. 缺少一些对整个类或者整个程序的整体说明,在第一次阅读代码的时候有点儿没有头绪。
    2. 简单的函数有注释,例如四则运算,求最大公约数等,但是最需要注释的几个表达式处理以及计算的具体过程缺少注释。
    3. 基本上都有注释,但是在Expression类中的expressions和operators两个属性的具体用途最好用注释具体说一下。

    Testing

    1. 缺少unit test对所有方法的正确性进行测试。
  • 相关阅读:
    HDOJ1267 下沙的沙子2[DP或卡特兰数]
    HDOJ1711 Number Sequence[KMP模版]
    HDOJ2546 饭卡[DP01背包问题]
    寻找必败态——一类博弈问题的快速解法
    kmp 模版
    网络流题目
    HDOJ1261 字串数[组合+大数]
    传说中效率最高的最大流算法(Dinic) [转]
    ACM博弈论
    HDOJ1061 Rightmost Digit[简单数学题]
  • 原文地址:https://www.cnblogs.com/summerMTY/p/4843741.html
Copyright © 2011-2022 走看看