关闭

C#几个经常犯错误汇总

发表于:2012-3-22 09:45

字体: | 上一篇 | 下一篇 | 我要投稿

 作者:指尖流淌    来源:51Testing软件测试网采编

  在我们平常编程中,时间久了有时候会形成一种习惯性的思维方式,形成固有的编程风格,但是有些地方是需要斟酌的,即使是一个很小的错误也可能会导致昂贵的代价,要学会善于总结,从错误中汲取教训,尽量不再犯同样错误,注重编程之美,代码的优雅,总结几个平常经常犯的错误。

  1、在C#编程中,字符型类型是最容易处理出错的地方,代价是非常昂贵,在.Net Framwork中,字符串是一个相当特别的引用类型,string本省就是一个不可继承的密封类,但是它具有了值类型所应用的特点,但是它在CLR中内存还是保存于托管堆之上,也就是说,当我们每次定义一个字符串类型的时候,就在堆内存中开辟一端内存,而当我们字符串被修改之后,它会创建一个新的内存,注意这里的内存是不连续的,而是通过修改栈内地址引用而拼凑字符串,不会改变源字符串在内存中的地址,所以有些程序员总是喜欢使用这样的方法格式化字符串:

string  SelectText="select * from "+TableName+" where UserName='"+Name+"'";

  上述代码,使用了字符串拼凑的方法,因为使用了多重串联,因此会在内存中创建两个不必要的字符串垃圾副本。

  其实在C#中,已经为我们提供了StringBuilder和String.Fromat来解决此问题,虽然他们可以实现同样的功能,但是他们有质的变化,StringBuilder在内存中开辟的是一段连续内存,当增加新字符串时候,它会在栈中指向的同一个堆内存中连续存放字符,这就形成了性能的提升。所以我们将上面代码改成:

string SelectText=string.Format("select  *  from {0} where UserName={1}",TableName,Name);

  2、大多数开发人员都不知道内置的验证数据类型的方法,如System.Int32,因此很多人都是自己实现的,其实这是不妥的,因为这些基本类型中都存在自己固有的类型验证方法,下面这个就是自己实现验证的一个字符串是否是数值的代码:

  1. public bool CheckIfNumeric(string value)  
  2. {  
  3.     bool IsNumeric=true;  
  4.     try 
  5.      {  
  6.            int i=Convert.ToInt32(value);  
  7.      }  
  8.      catch(FormatException excepiton)  
  9.      {  
  10.            IsNumeric=false;  
  11.      }  
  12.      return IsNumeric;  
  13. }

  虽然使用了try catch语句,这不是最佳的做法,更好的方法是下面使用Int.TryParse;

  1. int output=0;  
  2. bool IsNumeric=int.TryParse(value,out output);

  int.TryParse是更快、更简洁的方法。

  3、自己利用IDisposable接口手动释放内存

  在.NET Framework中,对象的处理和使用一样重要,理想的方法是在使用完对象的时候,在类中实现IDisposable接口中的dispose方法进行内存的释放,当然在.Net本身提供的垃圾回收机制(GC)中就提供了这样的功能,在我们实例化类对象时,在类本身的析构函数中会调用dispose方法,GC在各级内存堆满的情况下,自动检查对象使用情况,去相应的释放内存,但是运行在非托管平台上的方法,需要我们自己手动释放内存,比如我们常见的SqlConnection对象,也就有了下面的创建、使用和处理方法:

  1. public void  DALOneMethod()  
  2. {  
  3.     SqlConnection  connection=null;  
  4.     try 
  5.      {  
  6.           connection =new SqlConnection("。。。。。。。。。。。");  
  7.           connection.Open();  
  8.           //sqlcommand。。run  
  9.      }  
  10.      catch(Exception exception)  
  11.      {  
  12.              // manager exception  
  13.      }  
  14.      finally 
  15.      {  
  16.             connection.Close();  
  17.             connection.Disopse();  
  18.      }  
  19. }

  上述代码是大部分程序员会出现的代码,乍看没啥问题,连接处理在最后一个代码中被明确调用,但是如果发生了一个异常,catch代码块就被执行,然后再执行最后一个代码块处理连接,因此在最后一个代码块执行之前,连接将一直留在内存中,大部分我们会在此处记录错误,一般涉及到IO操作,如果延时时间比较长的话,这个连接将在内存时间长时间停留。我们一个原则就是当对象不再使用的时候我们里面释放资源。

31/3123>
《2023软件测试行业现状调查报告》独家发布~

关注51Testing

联系我们

快捷面板 站点地图 联系我们 广告服务 关于我们 站长统计 发展历程

法律顾问:上海兰迪律师事务所 项棋律师
版权所有 上海博为峰软件技术股份有限公司 Copyright©51testing.com 2003-2024
投诉及意见反馈:webmaster@51testing.com; 业务联系:service@51testing.com 021-64471599-8017

沪ICP备05003035号

沪公网安备 31010102002173号