结对同学代码复审

Code Review

General

  1. 代码可以正常运行。但是有些功能没有实现,比如,不能识别分数,不能识别括号。
  2. 代码风格不错,有简单的注释,简单易懂。
  3. 代码很简单,没有冗余的地方。

 

Security

  1. 没有对输入进行检查,输入不规范时会导致程序崩溃。当然,第一次作业中并没有对输入不符合规范的情况的处理。
  2. 没有对异常进行捕捉

 

Documentation

  1. 有解释代码的注释,可以帮助读者理解代码含义。

 

Testing

  1.代码没有测试程序。没有进行测试。

  2.数组没有越界检查。

  3.没有捕捉任何异常。

 

posted on 2015-09-30 12:55  pikali  阅读(160)  评论(1编辑  收藏  举报