结对同学代码复审
Code Review
General
- 代码可以正常运行。但是有些功能没有实现,比如,不能识别分数,不能识别括号。
- 代码风格不错,有简单的注释,简单易懂。
- 代码很简单,没有冗余的地方。
Security
- 没有对输入进行检查,输入不规范时会导致程序崩溃。当然,第一次作业中并没有对输入不符合规范的情况的处理。
- 没有对异常进行捕捉
Documentation
- 有解释代码的注释,可以帮助读者理解代码含义。
Testing
1.代码没有测试程序。没有进行测试。
2.数组没有越界检查。
3.没有捕捉任何异常。