代码复审

我们小组阅读的是Magiccode组的代码，总体上代码书写符合规范，结构也很合理。每个类每个方法都在干自己的事情。使用的变量都在很近的地方定义的，没有出现调来调去很混乱的代码。但是其中还是有些美中不足，以下是我的一些个人意见：

1.没有注释。给的代码里，通篇没有一个注释，对于我的理解造成了很大的障碍，同时相信对于他们之后的维护也会造成一定的麻烦。

2.代码书写基本符合规范，但是风格却不甚统一，刚开始怀疑是个人习惯的问题。但是后来发现，同样的句式前后也不一样，就不知道用意何在了。请看下面第一图和第二图的对比，以及第三图和4中的图。

3.有些地方，写的不够严谨，比如下面没有考虑到打不开的情况。

4.有个别地方，纯属个人意见，在这里提出，听听大家的意见。比如下面的例子，个人觉得“可以先判断有没有List，有点滥用抛出异常的感觉”，但是也有人觉得下面的写法更好。

5.还有些问题，不甚理解，比如下面的例子，为什么会出现word==null的情况呢，出现的话，岂不是在浪费空间。

当然，看别人的代码也是有收获的，比如：

下面的例子中，使用单键模式，这样全局只会有一个SearchManager实例，将构造函数封闭，只提供一个GetInstance方法供外部取得唯一的实例变量。以前在其他地方看到过，用在这里很不错。

转载于:https://www.cnblogs.com/buaashine/archive/2012/12/13/2815630.html