c# - 如何通过配置静态方法来防止故障(内存泄漏)？

Question

我正在使用 ASP.NET 4.0。

从过去几周开始，很少有用户提示应用程序开始出现故障。 GridView 突然开始显示同一用户或其他并发用户可能在当天的任何时间点访问过的 DropDown 控件的内容。同样，DropDown 控件可能会由任何旧结果集的 RowID 而不是实际项目填充。

我看到一篇文章:Users seeing other users data in ASP.NET作者在其中讨论了导致内存泄漏行为的静态对象。

这让我想起了我项目中的一个类，它是 Static 并且包含 public static 方法。此类包含用于填充 DropDown、为查询输入返回 DataSet 或基于查询输入返回标量对象的方法。

这个类的摘录如下:

public static class reuse
{
    public static void FillDropDownList(string Query, DropDownList DropDownName, string ConnectionStringParameter)
    {
        SqlDataReader dr;

        try
        {
            dbConnection.OpenConnection(ConnectionStringParameter);

            //Check whether the Drop Down has existing items. If YES, empty it.
            if (DropDownName.Items.Count > 0)
                DropDownName.Items.Clear();

            SqlCommand cmd = new SqlCommand(Query,dbConnection.cn);
            dr = cmd.ExecuteReader();

            DropDownName.Items.Add("-- Select --");
            DropDownName.Items.Add("All");
            while (dr.Read())
                DropDownName.Items.Add(dr[0].ToString());

            dr.Close();
        }
        catch (Exception ex)
        {
            rpkCustomErrorHandler.GetScript(HttpContext.Current.Response,ex.Message.ToString());
        }
        dbConnection.CloseConnection();
    }
}

我想知道这是否是我上面讨论的故障的原因。如果是，有没有办法在方法的任务完成后处理静态方法。我是否需要将类从 Static 更改为默认的普通类？

已编辑

我有另一个类，它也是静态的，并被上面的类使用:

public static class dbConnection
{
    public static SqlConnection cn = new SqlConnection();

    public static void OpenConnection()
    {
        try
        {
            cn.ConnectionString = ConfigurationManager.ConnectionStrings["cnWebTwDrill"].ToString();

            if (cn.State == ConnectionState.Closed)
                cn.Open();
        }
        catch (Exception)
        {
            throw;              
        }
    }
}

我是否应该从连接类中删除“静态”并每次使用唯一实例调用此类？

Answer 1

您向我们展示的代码是静态类/方法的错误使用示例。您放入一个静态方法或类中的东西，这些东西很容易适合对象并且不需要状态。

您拥有的功能应该在业务层(将检索数据)和 UI/ Controller (在本例中为页面)之间进行拆分，您可以在其中将数据分配给服务器控件。所有这些操作都是特定于请求的，没有理由为此使用静态方法。只是(糟糕的)过程编程的标志。在处理数据库访问(或一次性对象)时，您应该使用 using 语句。像这样

using(var conex=GetConnection())
{
   try
   {
     conex.Open(); 
     //do stuff

    var cmd= new SqlCommand();//cfg command
    using (var rd= cmd.ExecuteReader())   
      {
           //do read
      }
   }
   catch(Exception ex)
    {
       //handle exception
    }
}

using 语句会在 block 的末尾自动调用 Dispose。它基本上是

的快捷方式

try {}
finally{ //dispose }.