看似简单!解读C#程序员最易犯的7大错误-技术开发专区

看似简单!解读C#程序员最易犯的7大错误

作者：IT168黄永兵编辑：胡铭娅 2010-09-28 00:05 IT168网站原创

　　• 处理对象实现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类错误持有不同的看法，欢迎发表你的意见和想法。

第1页：格式化字符串第2页：在大型数据集上使用foreach 第3页：处理对象实现IDisposable接口

关注我们