码迷,mamicode.com
首页 > 其他好文 > 详细

代码复审

时间:2014-10-24 00:06:38      阅读:152      评论:0      收藏:0      [点我收藏+]

标签:style   使用   strong   sp   文件   on   问题   代码   bs   

代码中值得肯定的地方:

1.尝试使用了正则表达式,正确地解决了分词的问题

2.扫描文件的时候,使用c#的EnumerateFiles类将符合条件的文件首先存入一个数组,这样的做法避免了在统计过程中递归访问子目录

3.代码的逻辑清晰,有高类聚、低耦合的意识,把有关功能聚集到了一起,不同功能的模块也进行了一定的分离。

存在的一些问题:

1.缺少必要的注释,尽管代码比较简单,给自己的代码写注释仍然是一个需要坚持的习惯,最起码会有利于后期的维护。

2.整个程序只有一个program.cs,可见还没有明显的面向对象的意识,如果能够按功能写出一些类来就好了。

3.可能是在完成中不追求更好,排序部分并没有按照要求重写方法,使得程序的运行结果不能严格满足按ascii码排序的要求,希望以后能够注意

4.代码风格的问题,有一些变量的名字并不能清楚地表达清楚该变量的作用,降低了程序的可读性,同样也是不利于后期维护的。

5.对输入的处理上面,并不是特别完善,希望能增强程序的健壮性。

 

 

最后非常高兴能进行这次结对编程!

代码复审

标签:style   使用   strong   sp   文件   on   问题   代码   bs   

原文地址:http://www.cnblogs.com/penglinjiang/p/4047127.html

(0)
(0)
   
举报
评论 一句话评论(0
登录后才能评论!
© 2014 mamicode.com 版权所有  联系我们:gaon5@hotmail.com
迷上了代码!