黑马程序员技术交流社区

标题: 看似简单!解读C#程序员最易犯的7大错误 [打印本页]

---

作者: 熊丽    时间: 2013-8-2 10:50
标题: 看似简单!解读C#程序员最易犯的7大错误
编程时犯错是必然的，即使是一个很小的错误也可能会导致昂贵的代价，聪明的人善于从错误中汲取教训，尽量不再重复犯错，在这篇文章中，我将重点介绍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 implementation
ChildMethod1();
}
catch (Exception exception)
{
//Handle exception
}
}

private void ChildMethod1()
{
try
{
//some implementation
ChildMethod2();
}
catch (Exception exception)
{
//Handle exception
throw;

}
}

private void ChildMethod2()
{
try
{
//some implementation
}
catch (Exception exception)
{
//Handle exception
throw;
}
}
}



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

　　解决办法是让异常处理方法独立出来，如：
C# code
public class NestedExceptionHandling
{
public void MainMethod()
{
try
{
//some implementation
ChildMethod1();
}
catch(Exception exception)
{
//Handle exception
}
}

private void ChildMethod1()
{
//some implementation
ChildMethod2();
}

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 loop
watch.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 loop
watch.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();
}

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();
}
}



　

---

作者: 许庭洲    时间: 2013-8-2 20:14
值得学习ing!

---

欢迎光临 黑马程序员技术交流社区 (http://bbs.itheima.com/)

黑马程序员IT技术论坛 X3.2