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 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類錯誤持有不同的看法，歡迎發表你的意見和想法。

 

轉自：http://tech.it168.com/a2010/0927/1108/000001108666.shtml