Individual Project 复审 叶露婷
我复审的是王卓的程序,程序的思路比较缜密,考虑到了很多方面的情况,下面给出一点建议。
1.代码的可阅读性不强,不过具有一定的面向对象的特性,不同的类有自己独立的功能,但是在每个类的实现过程中,首先,变量的命名有些随意,不能从变量的名字直观看出这个变量的大概用途,其次,功能没有细化,有些函数在实现功能时有很多的for语句或者if语句的嵌套,导致代码的可读性不高。
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 }
浙公网安备 33010602011771号