代码复审

　　看到小伙伴的代码，我表示深深的佩服。感觉自己离别人的差距怎么就这么大，心里还真的很羡慕。按照要求，对小伙伴的代码审查结果如下：

 

Code Review Checklist
General
· Does the code work? Does it       perform its intended function,     the logic is correct etc.	· 代码大多数情况能够正常运行，但是中间有一次出现了乱码（烫烫烫烫烫），由于有随机数，所以没能再次找到这个bug，除此之外，计算上有问题，发现有些式子的计算结果不对
· Is all the code easily                   understood?	· 代码结构十分清晰，这点让我十分佩服。因此，我打算根据小伙伴的代码结构，修改自己的程序
· 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?	· 代码结构很好，模块化明显，有主函数处理输入输出，有fraction类处理计算，有express类处理式子的相关结果
· Can any global variables be         replaced?	· 全局变量不多，就几个，没必要替换，各自实现自己的功能
· 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?	· 有一个求最大公约数的函数可以替换，但是也是别人写好的库，并不是C++自带的库
· Can any logging or debugging     code be removed?	· 没有发现调试用代码，估计是小伙伴在提交前就已经删除了吧
Security
· Are all data inputs checked        (for the correct type, length,         format, and range) and               encoded?	· 输入的格式没有限制，调用了C的库函数atoi()，但是当输入参数是一个非数字字符时就会返回0，这样得到的结果就是不正确的
· Where third-party utilities are      used, are returning errors            being caught?	· 未使用第三方工具；没有返回异常值，返回值都是0，不管格式正确与否，但是小伙伴打印输出了相应的字符串说明相关异常
· Are output values checked          and encoded?	· 输出的表达式进行了除数是否为0和减法是否为负数的审查
· Are invalid parameter values        handled?	· 未处理
Documentation
· Do comments exist and                describe the intent of the            code?	·基本上在我觉得需要说明的地方都出现了注释，有少部分需要联系其他代码才能看明白的代码没有comment
· Are all functions commented?	· 不是每一个函数都有注释，而且注释并没有说明函数的功能，这点我觉得需要改进一下
· Is any unusual behavior or          edge-case handling                      described?	· 没有描述
· Is the use and function of            third-party libraries                      documented?	· 使用的第三方接口（函数）都是C库中有的，因此未进行说明。
· Are data structures and units      of measurement explained?	· 有作解释，但是解释并不多，只是简单的说明了一下
· Is there any incomplete              code? If so, should it be              removed or flagged with a          suitable marker like ‘TODO’?	· 无未完成的代码。
Testing
· Is the code testable? i.e.            don’t add too many or hide          dependencies, unable to              initialize objects, test                  frameworks can use methods      etc.	· 整个程序是容易测试的
· Do tests exist and are they          comprehensive? i.e. has at          least your agreed on code          coverage.	· 没有测试代码。
· Do unit tests actually test            that the code is performing          the intended functionality?	· 没有测试代码。
· Are arrays checked for ‘out-        of-bound’ errors?	· 数组都是定长的，而且按照理论是不可能越界的
· Could any test code be                replaced with the use of an        existing API?	· 不可

　　看了小伙伴的代码，发现有不少可以改进的地方，但是更多的是值得自己学习的地方。我们都是用C++实现的，但是她的代码看着很舒服，自己的代码却是连回头看的想法都没有，这一点，感觉自己特别失败。希望在以后的学习中，慢慢改进自己不好的代码风格，学习好的代码结构，向小伙伴学习。