代码复审

以下是我看了彭林江的代码以后的复审报告：

程序结构还是比较清楚的，比较规整，对程序的实现思路还是比较清晰。c#是我们刚接触不久的语言，在代码中能看到彭林江同学是下了一些功夫的，用了不少不同于c或者java的写法来简化自己的程序，效果也很好。

 

也有一些可以改进的地方：

1、首先比较大的一个问题：代码结构问题，主要Program类变成了一个God类，几乎所有的功能都在这个类里面实现了，无论是单词的匹配功能，还是排序功能，都被Program类全部包揽。相反的File类、WordAndValue类则成了idle类，甚至没必要存在。

从这一点来讲，我的改进建议是：重新审视这次任务，梳理一下每个类的工作，把应该由某个类自己完成的功能让它自己完成，当然这可能要对代码结构做很大调整。

2、 第二个问题就是对变量的定义不很好导致代码可读性不高。比如ExtenWriteFile方法中的top变量，当初读者看到时很难很快判定这个变量的意义。

改进建议：

a、使用完整描述符来准确地描述变量、字段或类

b、使用适用于领域内的术语

c、巧妙节俭地使用缩写

3、 最后一点就是代码不够精简，（当然不是什么大问题，但好的程序风格总是让人觉得舒服的），好几个地方偏繁琐，

改进建议:多看一下好程序的程序风格，多读读练练。这个问题感觉同第一个问题有点类似，需要解决好代码的复用以及模块分割的问题。