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

代码互审

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

标签:http   io   os   ar   使用   数据   on   问题   cti   

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

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

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

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

给出建议:

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

代码互审

标签:http   io   os   ar   使用   数据   on   问题   cti   

原文地址:http://www.cnblogs.com/dahuang/p/4047181.html

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