代码复审

给结对伙伴叶露婷的代码复审：

1.变量命名存在些许问题，尽量不要使用拼音，如果需要按照中文意思命名可以采用英语。

1 public void dictionarystore()

2 {

3 String temp;

4 count jishu;

5 

6 foreach (Object o in str)

7 {

8 temp = o.ToString().ToLower();

9 if (table.ContainsKey(temp))

10 {

11 jishu = table[temp];

12 jishu.setnum();

13 if (jishu.getdanci().CompareTo(o.ToString()) < 0)

14 {

15 jishu.setdanci(o.ToString());

16 }

17 }

18 else

19 {

20 jishu = new count(o.ToString(), 1);

21 table.Add(temp, jishu);

22 }

23 }

24 

25 

26 }

27 }

 

2.使用了过多的类，但是这些类的存在没有很大的实际意义，本来只有两百行的代码分成这么多个类有些繁琐，可以将一些功能比较薄弱的类归并到其他的类中实现其功能。比如下面这代码的功能是获取文件中的内容，完全可以将内容的获取和分割放在一个类中，这应该是属于同一种处理，也是同一个对象应该提供的功能。不要将一些杂乱的功能归为一个类。面向对象不是简单的分割，而是有原因的。合理确定对象，这样在以后写大型程序的时候能够更易看懂更好管理。

1 public void dirscanner(String dir)

2 {

3 String[] allfile = null;

4 String[] alldirectory = null;

5 if (!Directory.Exists(dir) && !File.Exists(dir))

6 {

7 Console.WriteLine("该路径无效");

8 Environment.Exit(0);

9 }

10 else if (Directory.Exists(dir))

11 {

12 allfile = Directory.GetFiles(dir);

13 alldirectory = Directory.GetDirectories(dir);

14 for (int i = 0; i < allfile.Length; i++)

15 {

16 dirscanner(allfile[i]);

17 }

18 for (int i = 0; i < alldirectory.Length; i++)

19 {

20 dirscanner(alldirectory[i]);

21 }

22 }

23 

24 String houzhui = dir.Substring(dir.LastIndexOf(".") + 1);

25 if (houzhui == "txt" || houzhui == "cpp" || houzhui == "h" || houzhui == "cs")

26 {

27 String text = File.ReadAllText(dir);

28 str.Append('-' + text);

29 }

30 }

31 }

 

3.这里的dictionary的值对的值得类型为一个自己定义的类，有些浪费空间。可以考虑一下还有没有其他的方式，能够同时兼顾时间和空间。

1 public void dictionarystore()

2 {

3 String temp;

4 count jishu;

5 

6 foreach (Object o in str)

7 {

8 temp = o.ToString().ToLower();

9 if (table.ContainsKey(temp))

10 {

11 jishu = table[temp];

12 jishu.setnum();

13 if (jishu.getdanci().CompareTo(o.ToString()) < 0)

14 {

15 jishu.setdanci(o.ToString());

16 }

17 }

18 else

19 {

20 jishu = new count(o.ToString(), 1);

21 table.Add(temp, jishu);

22 }

23 }

 

4.时间效率方面还行，使用了dictionary很好的改善了时间效率，对文件内容的处理使用了正则表达式和spilt分割，都很得当。功能的实现也很独立，对变量的命名总体上还是合适的，代码比较容易读懂。