如果不首先删除侦听器,重新分配数据源是否会导致内存泄漏？

Question

我的表格上有两个网格.每当一行在grid1中获得焦点时,其关联的子行将通过ADO.NET从数据库中获取,grid2的DataSource将被重新分配,如下所示:

     //focused row changed handler
     DataTable Children = FetchChildren(parentid);
     grid2.DataSource = Children.DefaultView;
     Children.RowDeleted += (sndr, evt) =>
      {
        //
      };

ASIDE:grid1包含很多行,因此我不想在一个(耗时)查询中获取所有父行的子行,然后过滤客户端子行的大数据集.

当用户使用表单期间多次重新分配本地Children变量和grid2的数据源时,这样的匿名事件监视器会发生什么？不明确删除处理程序会导致内存泄漏吗？

Answer 1

不，在您显示的代码中，假设您使用该术语来指示对象不会被垃圾收集，则不会出现内存泄漏。DataTable匿名事件处理程序的委托是一个引用，一旦超出范围，它就会与创建的其余对象一起进行垃圾回收。

我创建了以下测试设备来模拟您的代码正在执行的操作：

static object DataSource;

static void Main(string[] args)
{
    Test1();
    // clean up
    GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced, true);
    GC.WaitForPendingFinalizers();
    GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced, true);
}

static void Test1()
{
    for (var i = 0; i < 1000; i++)
    {
        var dt = FetchChildren(i);
        DataSource = dt.DefaultView;
        dt.RowDeleted += (s, e) =>
        {
            var table = (DataTable)s;
            Trace.WriteLine(String.Format("{0}:{1}:{2}", e.Action, e.Row.RowState, table));
        };
        // do work
        var dv = (DataView)DataSource;
        dv.Delete(5);
    }
    DataSource = null;
}

// create a useful datatable
static DataTable FetchChildren(int parent)
{
    var dt = new DataTable();
    dt.Columns.Add("key", typeof(int));
    dt.Columns.Add("guid", typeof(string));

    for(var i=0; i<10; i++)
    {
        var row = dt.NewRow();
        row[0] = parent;
        row[1] = Guid.NewGuid().ToString("N");
        dt.Rows.Add(row);
    }

    return dt;
}

当我使用 Visual Studio 中的 Performance Explorer 在Instrumentaion 模式下运行此命令并启用收集 .Net 对象生命周期信息时，这是我的结果：

正如您所看到的，当分析结束时，所有 DataTable 实例都回到 0，并且这适用于该测试中实例化的所有类型。

如果您保留引用，那么您可能会在方法末尾得到 1000 个实例，如本测试用例所示：

static void Test2()
{
    for (var i = 0; i < 1000; i++)
    {
        var dt = FetchChildren(i);
        var local = DataSource; // our previous DataTable
        dt.RowDeleted += (s, e) =>
        {
            var table = (DataTable)s;
            Trace.WriteLine(String.Format("{0}:{1}:{2}", e.Action, e.Row.RowState, local)); // use it here
        };
        DataSource = dt.DefaultView;
        // do work
        var dv = (DataView)DataSource;
        dv.Delete(5);
    }
    //DataSource = null; // don't dereference
}

因此，只要您不保留对之前使用的实例的引用，就应该没问题。