Individual Project 复审 叶露婷

 我复审的是王卓的程序,程序的思路比较缜密,考虑到了很多方面的情况,下面给出一点建议。

1.代码的可阅读性不强,不过具有一定的面向对象的特性,不同的类有自己独立的功能,但是在每个类的实现过程中,首先,变量的命名有些随意,不能从变量的名字直观看出这个变量的大概用途,其次,功能没有细化,有些函数在实现功能时有很多的for语句或者if语句的嵌套,导致代码的可读性不高。
 

 1 int a,k=0; 
 

2.对于C#语言掌握不够,一些数据可以用更好的存储方式,比如单词的存储可以用List。一些可以直接用到C#的方法却用了复杂的方法自己实现。

 

3.对于封装性没有体现,一些内部的变量和方法,只是为内部的功能服务的,但还是写成对外部可见的,这样在以后复杂的代码中不是一个很好的风格。

 1 public void quick(Word[] many)
 2         {
 3             int i, j,n, flag = 1;
 4             Word temp;
 5             for (int k = 0; ; k++)
 6             {
 7                 if (many[k] == null)
 8                 {
 9                     n = k;
10                     break;
11                 }
12             }
13             i = n - 2;
14             while(i>=0&&flag==1)
15             {
16                 flag = 0;
17                 for (j = 0; j <=i; j++18                     if(many[j].num<many[j+1].num||(many[j].num==many[j+1].num&&String.Compare(many[j+1].key,many[j].key)==1))
19                     {
20                         temp=many[j];
21                         many[j]=many[j+1];
22                         many[j+1]=temp;
23                         flag=1;
24                     }
25                 i--;
26             }
27         }

 

posted on 2014-10-23 23:29  叶小婷儿  阅读(148)  评论(0)    收藏  举报

导航