|
General |
|
|
Does the code work? Does it perform its intended function, the logic is correct etc. |
代码并没有完全完善，生成功能和计算功能实现但是仍有缺陷，检验功能还未开发 |
|
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? |
采用的C++编写，除了random其他函数都放在Main函数里面有些过于紧凑，可以将里面的函数模块化 |
|
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? |
能用到库函数的都用到了 |
|
Can any logging or debugging code be removed? |
有的被注释到了 |
|
Security |
|
|
Are all data inputs checked (for the correct type, length, format, and range) and encoded? |
并没有 |
|
Where third-party utilities are used, are returning errors being caught? |
没有 |
|
Are output values checked and encoded? |
没有，存在生成的真分数分母为0的情况没有处理 |
|
Are invalid parameter values handled? |
没有 |
|
Documentation |
|
|
Do comments exist and describe the intent of the code? |
注释明确的指出了代码的内容 |
|
Are all functions commented? |
所有的function都有注释 |
|
Is any unusual behavior or edge-case handling described? |
没有 |
|
Is the use and function of third-party libraries documented? |
是的 |
|
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? |
程序中没有包含unit tests，需要人工测试 |
|
Are arrays checked for ‘out-of-bound’ errors? |
数据结构多使用数组，全部设定在数组范围内并没有出现out-of-bound |
|
Could any test code be replaced with the use of an existing API? |
程序中没有包含test code |
我的结队伙伴并没有完成自己的全部功能，所以以上很多审查内容不能完全完成。就他目前所完成的功能进行审查和复核。
个人程序分析：
对照以上表格的内容，我觉得自己并不完善的地方有测试部分，并没有包含test code。生成计算式的时候总会出现随机数生成不随机的情况，所以表达式的生成很有限制。