看似簡單!解讀C#程序員最易犯的7大錯誤

簡體傳統

摘要：編程時犯錯是必然的，即使是一個很小的錯誤也可能會導致昂貴的代價，聰明的人善于從錯誤中汲取教訓，盡量不再重復犯錯，在這篇文章中，我將重點介紹C#開發人員最容易犯的7個錯誤。

　　編程時犯錯是必然的，即使是一個很小的錯誤也可能會導致昂貴的代價，聰明的人善于從錯誤中汲取教訓，盡量不再重復犯錯，在這篇文章中，我將重點介紹C#開發人員最容易犯的7個錯誤。

　　格式化字符串

　　在C#編程中，字符串類型是最容易處理出錯的地方，其代價往往也很昂貴，在.NET Framework中，字符串是一個不可變的類型，當一個字符串被修改后，總是創建一個新的副本，不會改變源字符串，大多數開發人員總是喜歡使用下面這樣的方法格式化字符串：

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

　　上面的代碼太亂了，由于字符串是不可變的，這里它又使用了多重串聯，因此會在內存中創建三個不必要的字符串垃圾副本。

　　最好的辦法是使用string.Format，因為它內部使用的是可變的StringBuilder，也為凈化代碼鋪平了道路。

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

　　• 嵌套異常處理

　　開發人員喜歡在方法的末尾加上異常處理的嵌套方法，如：

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

　　如果相同的異常被處理多次，上面的代碼會發生什么?毫無疑問，性能開銷將會劇增。

　　解決辦法是讓異常處理方法獨立出來，如：

　　• 在大型數據集上使用foreach

　　大部分開發人員更喜歡使用foreach循環，而無視for循環，因為foreach更容易使用，但操作大型數據集時，使用foreach已經被證明是代價高昂的，在下面的代碼中，我同時使用for和foreach遍歷相同的數據庫，在圖1中顯示了兩種循環方法消耗的時間。

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

　　圖 1 for和foreach循環遍歷相同數據庫消耗的時間對比　　從上圖可以看出，foreach循環明顯要慢一些，它消耗的時間幾乎是for循環的兩倍，這是因為foreach循環中的dt.Rows要訪問數據庫中的所有行。因此需要遍歷大型數據集時最好使用for循環。

　　驗證簡單的原始數據類型

　　大多數開發人員都不知道內置的驗證原始數據類型的方法，如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);

　　根據我的經驗，int.TryParse是更快，更簡潔的方法。　　

　　處理對象實現IDisposable接口

　　在.NET Framework中，對象的處理和使用一樣重要，理想的辦法是在類中實現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接口的類。

　　聲明公共變量

　　聽起來可能有點簡單，但我們經常看到濫用公共變量聲明的情況，先來看一個例子：

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公共變量實施了很好的控制，它變成只讀，不能由調用者類修改。

　　利用System.Data.DataTable訪問數據

　　我常常看到開發人員使用列索引從數據庫訪問數據，如：

public class MyClass { 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查詢匹配數據時發生了變化，你的應用程序將會受到影響，正確的做法應該是使用列名訪問數據。

public class MyClass { 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]); } } }

　　這樣的代碼更加穩固，列順序發生變化不會給應用程序造成任何影響，如果在一個地方使用局部變量保存列名更好，即使將來你的列名發生了變化，也不用修改應用程序代碼。

　　小結

　　我希望你能從自身和其他程序員所犯的錯誤中汲取教訓，避免犯同樣的錯誤，如果你對本文闡述的C#程序員常犯的7類錯誤持有不同的看法，歡迎發表你的意見和想法。

IT168 2014-07-25 18:37:45

[新一篇] 解決T4模板的程序集引用的五種方案

[舊一篇] .NET程序員不要錯過如此強大的T4模板