看似簡單!解讀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; } } }

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

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


			
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中顯示了兩種循環方法消耗的時間。


			
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

  圖 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模板
回頂部
寫評論


評論集


暫無評論。

稱謂:

内容:

驗證:


返回列表