看似简单!解读C#程序员最易犯的7大错误

来源：百度文库编辑：神马文学网时间：2024/04/30 05:33:27

编程时犯错是必然的，即使是一个很小的错误也可能会导致昂贵的代价，聪明的人善于从错误中汲取教训，尽量不再重复犯错，在这篇文章中，我将重点介绍C#开发人员最容易犯的7个错误。

　　格式化字符串

　　在C#编程中，字符串类型是最容易处理出错的地方，其代价往往也很昂贵，在.NET Framework中，字符串是一个不可变的类型，当一个字符串被修改后，总是创建一个新的副本，不会改变源字符串，大多数开发人员总是喜欢使用下面这样的方法格式化字符串：

C# code

string updateQueryText = "UPDATE EmployeeTable SET Name='" + name+ "' WHERE EmpId=" + id;

　　上面的代码太乱了，由于字符串是不可变的，这里它又使用了多重串联，因此会在内存中创建三个不必要的字符串垃圾副本。

　　最好的办法是使用string.Format，因为它内部使用的是可变的StringBuilder，也为净化代码铺平了道路。

C# code

string updateQueryText = string.Format("UPDATE EmployeeTable SET Name='{0}'WHERE EmpId={1}", name, id);

　　• 嵌套异常处理

　　开发人员喜欢在方法的末尾加上异常处理的嵌套方法，如：

C# code

public class NestedExceptionHandling{public void MainMethod(){try{//some implementationChildMethod1();}catch (Exception exception){//Handle exception}}private void ChildMethod1(){try{//some implementationChildMethod2();}catch (Exception exception){//Handle exceptionthrow;}}private void ChildMethod2(){try{//some implementation}catch (Exception exception){//Handle exceptionthrow;}}}

　　如果相同的异常被处理多次，上面的代码会发生什么?毫无疑问，性能开销将会剧增。

　　解决办法是让异常处理方法独立出来，如：

C# code

public class NestedExceptionHandling{public void MainMethod(){try{//some implementationChildMethod1();}catch(Exception exception){//Handle exception}}private void ChildMethod1(){//some implementationChildMethod2();}private void ChildMethod2(){//some implementation}}

　　• 在大型数据集上使用foreach

　　大部分开发人员更喜欢使用foreach循环，而无视for循环，因为foreach更容易使用，但操作大型数据集时，使用foreach已经被证明是代价高昂的，在下面的代码中，我同时使用for和foreach遍历相同的数据库，在图1中显示了两种循环方法消耗的时间。

C# code

static void Main(string[] args){DataTable dt = PopulateData();Stopwatch watch = new Stopwatch();//For loopwatch.Start();for (int count = 0; count < dt.Rows.Count; count++){dt.Rows[count]["Name"] = "Modified in For";}watch.Stop();Console.WriteLine("Time taken in For loop: {0}",watch.Elapsed.TotalSeconds);//Foreach loopwatch.Start();foreach (DataRow row in dt.Rows){row["Name"] = "Modified in ForEach";}watch.Stop();Console.WriteLine("Time taken in For Each loop: {0}",watch.Elapsed.TotalSeconds);Console.ReadKey();}

1

　　图 1 for和foreach循环遍历相同数据库消耗的时间对比
　　从上图可以看出，foreach循环明显要慢一些，它消耗的时间几乎是for循环的两倍，这是因为foreach循环中的dt.Rows要访问数据库中的所有行。因此需要遍历大型数据集时最好使用for循环。

　　验证简单的原始数据类型

　　大多数开发人员都不知道内置的验证原始数据类型的方法，如System.Int32，因此很多人都是自己实现的，下面就是一个自己实现的验证一个字符串是否是数值的代码：

C# code

public bool CheckIfNumeric(string value){bool isNumeric = true;try{int i = Convert.ToInt32(value);}catch(FormatException exception){isNumeric = false;}return isNumeric;}

　　它使用了try catch语句，因此不是最佳的做法，更好的办法是象下面这样使用int.TryParse：

C# code

int output = 0;bool isNumeric = int.TryParse(value, out output);　　根据我的经验，int.TryParse是更快，更简洁的方法。　　　　处理对象实现IDisposable接口　　在.NET Framework中，对象的处理和使用一样重要，理想的办法是在类中实现IDisposable接口的dispose方法，在使用这个类的对象后，可以通过调用dispose方法进行处理。　　下面的代码显示了一个SqlConnection对象的创建，使用和处理：[code=C#]public void DALMethod(){SqlConnection connection = null;try{connection = new SqlConnection("XXXXXXXXXX");connection.Open();//implement the data access}catch (Exception exception){//handle exception}finally{connection.Close();connection.Dispose();}}

　　在上面的方法中，连接处理在最后一个代码块中被明确调用，如果发生一个异常，catch代码块就会执行，然后再执行最后一个代码块处理连接，因此在最后一个代码块执行之前，连接将一直留在内存中，.NET Framework的一个基本原则就是当对象不被使用时就应该释放资源。

　　下面是调用dispose更好的办法：

C# code

public void DALMethod(){using (SqlConnection connection = new SqlConnection("XXXXXXXXXX")){connection.Open();//implement the data access}}

　　当你使用using代码块时，对象上的dispose方法将在执行退出代码块时调用，这样可以保证SqlConnection的资源被处理和尽早释放，你也应该注意到这个办法也适用于实现IDisposable接口的类。

　　声明公共变量

　　听起来可能有点简单，但我们经常看到滥用公共变量声明的情况，先来看一个例子：

C# code

static void Main(string[] args){MyAccount account = new MyAccount();//The caller is able to set the value which is unexpectedaccount.AccountNumber = "YYYYYYYYYYYYYY";Console.ReadKey();}public class MyAccount{public string AccountNumber;public MyAccount(){AccountNumber = "XXXXXXXXXXXXX";}}

　　在上面的MyAccount类中声明了一个AccountNumber公共变量，理想情况下，AccountNumber应该是只读的，但MyAccount类却没有对它实施任何控制。

　　声明公共变量正确的做法应该是使用属性，如：

C# code

public class MyAccount{private string _accountNumber;public string AccountNumber{get { return _accountNumber; }}public MyAccount(){_accountNumber = "XXXXXXXXXXXXX";}}

　　这里MyAccount类对AccountNumber公共变量实施了很好的控制，它变成只读，不能由调用者类修改。

　　利用System.Data.DataTable访问数据

　　我常常看到开发人员使用列索引从数据库访问数据，如：

C# code

public class MyClass{public void MyMethod(){//GetData fetches data from the database using a SQL queryDataTable dt = DataAccess.GetData();foreach (DataRow row in dt.Rows){//Accessing data through column indexint empId = Convert.ToInt32(row[0]);}}}

　　按照这种写法，如果列顺序在SQL查询匹配数据时发生了变化，你的应用程序将会受到影响，正确的做法应该是使用列名访问数据。

C# code

public class MyClass{private const string COL_EMP_ID = "EmpId";public void MyMethod(){//GetData fetches data from the database using a SQL queryDataTable dt = DataAccess.GetData();foreach (DataRow row in dt.Rows){//Accessing data through column nameint empId = Convert.ToInt32(row[COL_EMP_ID]);}}}

　　这样的代码更加稳固，列顺序发生变化不会给应用程序造成任何影响，如果在一个地方使用局部变量保存列名更好，即使将来你的列名发生了变化，也不用修改应用程序代码。

　　小结

　　我希望你能从自身和其他程序员所犯的错误中汲取教训，避免犯同样的错误，如果你对本文阐述的C#程序员常犯的7类错误持有不同的看法，欢迎发表你的意见和想法。

看似简单!解读C#程序员最易犯的7大错误看似简单的英语学习看似简单的英语提高语句 C#的调试和错误处理现代程序员的几个错误观点现代程序员的几个错误观点 C# Remoting的一个简单例子 C# 中 String.Format() 的简单用法看似恐怖道理简单：奇异的日本游泳池幻象(组图) 看似简单实则复杂成功方法的复制与创造面试中看似简单的十道难题最易犯的十大错误最易犯的十大错误程序员犯的非技术错误(Top 5) | 酷壳 C#代码规范 .NET程序员需要提升的修养 - 51CTO.COM 简单C# MessageBox的实现解析 - 51CTO.COM 用C#创建一个简单的文本编辑器吃素最易犯的9大错误聪明老婆最易犯的两大错误解读 C# 中的正则表达式程序员修身养性的十大原则程序员修身养性的十大原则程序员开发应用的五大兵器趣味盘点十种被人解读错误的职业