编程时犯错是必然的,我们来解读一下编程中最容出现的错误
1、拼接字符串
在C#编程中,字符串类型的处理是比较容易出错的地方,在.NET Framework中,字符串是一个不可变的类型,当一个字符串被修改后,总是创建一个新的副本,不会改变源字符串,大多数开发人员总是喜欢使用下面这样的方法格式化字符串:
string updateSqlText = "UPDATE Table SET Name='" + name+ "' WHERE Id=" + id;
这里它使用了多重串联拼接,因此会在内存中创建三个不必要的字符串垃圾副本,这种方式是最容易忽略的,最好的办法是使用string.Format,因为它内部使用的是可变的StringBuilder,也为净化代码铺平了道路,如下:
string updateSqlText = string.Format("UPDATE Table SET Name='{0}' WHERE Id={1}", name, id);
2、嵌套异常处理
在方法中添加异常处理模块try-cathc是必然的,但是没有必要在一个方法里面多次加上异常处理的嵌套方法,如下:
public class Class1 { 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; } } }
如果相同的异常被处理多次,整个项目都这样写?毫无疑问,性能开销将会剧增。
解决办法是让异常处理方法独立出来(一个大的方法只需要一个异常处理即可,特殊复杂场景可酌情多次使用),如:
public void MainMethod() { try { //some implementation ChildMethod1(); } catch (Exception exception) { //Handle exception } }private void ChildMethod1() { //some implementation ChildMethod2(); }private void ChildMethod2() { //some implementation}
3、for和foreach的选择
大部分开发人员更喜欢使用for循环,而无视foreach循环,因为for更容易使用,但操作大型数据集时,使用foreach无疑是最快的,
根据广大网友实验证明(分别对记录数为10000,100000,1000000条记录的时候进行采样分析),
foreach的平均花费时间只有for20%-30%左右。所以,我也要根据实际请求选择使用而不是一直使用某一种。
C#中foreach在处理集合和数组相对于for存在以下几个优势和劣势:
foreach语句简洁
效率比for要高(C#是强类型检查,for循环对于数组访问的时候,要对索引的有效值进行检查)
不用关心数组的起始索引是几(因为有很多开发者是从其他语言转到C#的,有些语言的起始索引可能是1或者是0)
处理多维数组(不包括锯齿数组)更加的方便
在类型转换方面foreach不用显示地进行类型转换
当集合元素如List<T>等在使用foreach进行循环时,每循环完一个元素,就会释放对应的资源
上面说了foreach循环的时候会释放使用完的资源,所以会造成额外的gc开销,所以使用的时候,请酌情考虑
foreach也称为只读循环,所以再循环数组/集合的时候,无法对数组/集合进行修改
数组中的每一项必须与其他的项类型相等
4、验证简单的原始数据类型
很多人员都忽略内置的验证原始数据类型的方法,如System.Int32(其他类型亦然),因此都是自己实现的方法,下面就是一个自己实现的验证一个字符串是否是数值的代码:
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:
int output = 0;bool isNumeric = int.TryParse(value, out output);
5、处理对象实现IDisposable接口
对象的处理和使用一样重要,理想的办法是在类中实现IDisposable接口的dispose方法,在使用这个类的对象后,可以通过调用dispose方法进行处理。
下面的代码显示了一个SqlConnection对象的创建,使用和处理:
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更好的办法:
public void DALMethod() { using (SqlConnection connection = new SqlConnection("XXXXXXXXXX")) { connection.Open(); //implement the data access } }
当你使用using代码块时,对象上的dispose方法将在执行退出代码块时调用,这样可以保证SqlConnection的资源被处理和尽早释放,
你也应该注意到这个办法也适用于实现IDisposable接口的类。
6、声明公共变量
听起来可能有点简单,但我们经常看到滥用公共变量声明的情况,先来看一个例子:
static void Main(string[] args) { MyAccount account = new MyAccount(); //The caller is able to set the value which is unexpected account.AccountNumber = "YYYYYYYYYYYYYY"; Console.ReadKey(); }public class MyAccount { public string AccountNumber; public MyAccount() { AccountNumber = "XXXXXXXXXXXXX"; } }
在上面的MyAccount类中声明了一个AccountNumber公共变量,理想情况下,AccountNumber应该是只读的,但MyAccount类却没有对它实施任何控制。
声明公共变量正确的做法应该是使用属性,如:
public class MyAccount { private string _accountNumber; public string AccountNumber { get { return _accountNumber; } } public MyAccount() { _accountNumber = "XXXXXXXXXXXXX"; } }
这里MyAccount类对AccountNumber公共变量实施了很好的控制,它变成只读,不能由调用者类修改。
7、利用System.Data.DataTable访问数据
人多人经常使用列索引从数据库访问数据,如:
public void MyMethod() { //GetData fetches data from the database using a SQL query DataTable dt = DataAccess.GetData(); foreach (DataRow row in dt.Rows) { //Accessing data through column index int empId = Convert.ToInt32(row[0]); } }
按照这种写法,如果列顺序在SQL查询匹配数据时发生了变化,你的应用程序将会受到影响,正确的做法应该是使用列名访问数据。
private const string COL_EMP_ID = "EmpId";public void MyMethod() { //GetData fetches data from the database using a SQL query DataTable dt = DataAccess.GetData(); foreach (DataRow row in dt.Rows) { //Accessing data through column name int empId = Convert.ToInt32(row[COL_EMP_ID]); } }
这样的代码更加稳固,列顺序发生变化不会给应用程序造成任何影响,
如果在一个地方使用局部变量保存列名更好,即使将来你的列名发生了变化,也不用修改应用程序代码。