看似簡單!解讀C#程序員最易犯的7大錯誤
編程時犯錯是必然的，即使是一個很小的錯誤也可能會導致昂貴的代價，聰明的人善于從錯誤中汲取教訓，盡量不再重復犯錯，在這篇文章中，我將重點介紹C#開發人員最容易犯的7個錯誤。
格式化字符串
在C#編程中，字符串類型是最容易處理出錯的地方，其代價往往也很昂貴，在.NET Framework中，字符串是一個不可變的類型，當一個字符串被修改后，總是創建一個新的副本，不會改變源字符串，大多數開發人員總是喜歡使用下面這樣的方法格式化字符串：
+ "' WHERE EmpId=" + id;
上面的代碼太亂了，由于字符串是不可變的，這里它又使用了多重串聯，因此會在內存中創建三個不必要的字符串垃圾副本。
最好的辦法是使用string.Format，因為它內部使用的是可變的StringBuilder，也為凈化代碼鋪平了道路。
WHERE EmpId={1}", name, id);
• 嵌套異常處理
開發人員喜歡在方法的末尾加上異常處理的嵌套方法，如：
{
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
}
}
• 在大型數據集上使用foreach
大部分開發人員更喜歡使用foreach循環，而無視for循環，因為foreach更容易使用，但操作大型數據集時，使用foreach已經被證明是代價高昂的，在下面的代碼中，我同時使用for和foreach遍歷相同的數據庫，在圖1中顯示了兩種循環方法消耗的時間。
{
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，因此很多人都是自己實現的，下面就是一個自己實現的驗證一個字符串是否是數值的代碼：
{
bool isNumeric = true;
try
{
int i = Convert.ToInt32(value);
}
catch(FormatException exception)
{
isNumeric = false;
}
return isNumeric;
}
它使用了try catch語句，因此不是最佳的做法，更好的辦法是象下面這樣使用int.TryParse：
bool isNumeric = int.TryParse(value, out output);
根據我的經驗，int.TryParse是更快，更簡潔的方法。
處理對象實現IDisposable接口
在.NET Framework中，對象的處理和使用一樣重要，理想的辦法是在類中實現IDisposable接口的dispose方法，在使用這個類的對象后，可以通過調用dispose方法進行處理。
下面的代碼顯示了一個SqlConnection對象的創建，使用和處理：
{
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更好的辦法：
{
using (SqlConnection connection = new SqlConnection("XXXXXXXXXX"))
{
connection.Open();
//implement the data access
}
}
當你使用using代碼塊時，對象上的dispose方法將在執行退出代碼塊時調用，這樣可以保證SqlConnection的資源被處理和盡早釋放，你也應該注意到這個辦法也適用于實現IDisposable接口的類。
聲明公共變量
聽起來可能有點簡單，但我們經常看到濫用公共變量聲明的情況，先來看一個例子：
{
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類卻沒有對它實施任何控制。
聲明公共變量正確的做法應該是使用屬性，如：
{
private string _accountNumber;
public string AccountNumber
{
get { return _accountNumber; }
}
public MyAccount()
{
_accountNumber = "XXXXXXXXXXXXX";
}
}
這里MyAccount類對AccountNumber公共變量實施了很好的控制，它變成只讀，不能由調用者類修改。
利用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]);
}
}
}
這樣的代碼更加穩固，列順序發生變化不會給應用程序造成任何影響，如果在一個地方使用局部變量保存列名更好，即使將來你的列名發生了變化，也不用修改應用程序代碼。
小結
我希望你能從自身和其他程序員所犯的錯誤中汲取教訓，避免犯同樣的錯誤，如果你對本文闡述的C#程序員常犯的7類錯誤持有不同的看法，歡迎發表你的意見和想法。
留言列表
