首页 > 代码库 > 代码互审

代码互审

对于同伴的代码,首先我想指出一下我从同伴代码中发现的优点:

  • 代码格式比较规整,基本做到了变量名让人一看就知道其基本作用。比如‘TotalCount‘, ‘Dictionary‘, ‘path‘等变量。
  • 用了Dictionary,Regex等相关的比较高级的方法,从代码中可看出对c#掌握得比较熟练。
  • 整个程序高内聚、低耦合,比较符合面向对象思想。

其次,也指出我认为有所不足的地方:

  • 还是有一些地方为了方便,使用了面向过程的变成思想,我也经常犯这样的错误,为了将某个值传入某个方法中,便将该该值得属性从相应类中公开,这样做非常欠妥,说明我们编写代码之前对模型的抽象还不够细致完善,才会导致了这样的现象。希望以后的编程中更加着重练习面向对象的编程思想。
  • 对输入方面考虑的不够周全,以后可以将输入可能出现exception的地方都使用try,catch语句块进行输入处理。
  • 对于大规模的输入数据,通过运行可以看出排序的时间还是比较长,对于算法还有改进的余地。
  • 在使用了正则表达式之后,不必要再使用自己写的if else语句拆分字符串了,这样使代码变得繁琐。

给出建议:

  • 当遇到不会解决的问题时,可以使用MSDN库(http://msdn.microsoft.com/library/)查找相关信息,上面都有非常详细的解决方法及示例,个人觉得非常实用。
  • 其次,面向对象编程思想的养成是非常不容易的,在平常没事儿的时候就可以训练。在上课下课的路上看到某个东西,试着总结他的属性、方法也不失为一种好的方法。

代码互审