Code Review Checklist

General

· Does the code work? Does it       perform its intended function,     the logic is correct etc.

· 代码可以正常运行。实现了用户需求中的所有功能（至少测试过程中未发现异常）

· Is all the code easily                   understood?

· 代码结构划分较为清晰。实现了主要三个类分别用于表示表达式、分数以及IO处理。代码中有一处相对而言较晦涩难懂，即在生成表达式过程中如何确定这个项是否是Notation类型（详情请见http://www.cnblogs.com/kanelim/p/4830963.html）。需要建立两个List用于记录可用的Notation以及先前用过的Notation，以便于直接获取Notation以及返回先前状态。

· Does it conform to your               agreed coding conventions?       These will usually cover               location of braces, variable         and function names, line             length, indentations,                   formatting, and comments.

· 因为复审的是自己的代码，所以完全一致

· Is there any redundant or           duplicate code?

· 大部分子功能代码都已经模块化，暂未发现多余代码。

· Is the code as modular as           possible?

· 代码已经将输入输出、表达式、分数等不同功能做了划分。但复审过程中觉得Main函数为了处理输入情况导致条件判断比较臃肿，可以进一步模块化。

· Can any global variables be         replaced?

· 用C#编写，无全局变量一说，仅由一些类中的静态成员用于辅助功能实现，不可替代。

· Is there any commented out       code?

· 有，部分前期开发时的过时算法，以及用于调试的代码。

· Do loops have a set length         and correct termination               conditions?

· 有。且测试过程中未发现无限循环的情况。

· Can any of the code be               replaced with library                     functions?

· 所有能用库函数实现的均已使用库函数。

· Can any logging or debugging     code be removed?

· 调试用代码已经注释。

Security

· Are all data inputs checked        (for the correct type, length,         format, and range) and               encoded?

· 对第一个功能生成表达式而言，对输入格式做了较强的判断，但没有对内容做过多的语义判断，如可能输入“-n -100 -r 3”这样非法的内容。在这一方面还需要改进。在字符串长度也未限制。在判断指令后有对输入内容重新编码为程序需求的数据结构。

· 对第二个功能判分而言，仅默认每个表达式都是符合规范的（详见第一次作业需求的前提条件），未进行安全性检查。

· Where third-party utilities are      used, are returning errors            being caught?

· 未使用第三方工具；程序中的诸多函数虽然有返回异常值，但是未对异常情况进行处理，如：calcPriority()，因为用户需求默认输入是合法的，因此无需做处理。

· Are output values checked          and encoded?

· 输出的表达式进行了除数是否为0和减法是否为负数的审查，并且用ToString()函数对存储于Expression中的表达式编码后输出。

· Are invalid parameter values        handled?

· 未处理。同上，大部分函数默认用户输入是合法的。并且用户输入在main函数中已经判断了合法性，在传参时也认为是合法的。

Documentation

· Is any unusual behavior or          edge-case handling                      described?