旧ASP.NET项目中DataTable资源释放问题的技术问询

之前我的问题被Stack标记为观点类，现重新发起技术问询。此前得到的建议是“永远不要返回DataReader，返回DataTable”，但参考微软示例中，GetCustomers()返回DataTable给调用者，调用者使用using语句处理DataReader以清理资源，却未对DataTable进行释放操作。

此外，针对以下代码：

private void DoSomething()
{
    string Txt, sql;
    sql = "Select * from table;";
    DataTableReader dr = GetReader(sql);
    while (dr.Read())
    {
        Txt = dr["Txt"] + "";
    }
}

private DataTableReader GetReader(string sql)
{
    SqlDataAdapter da = new SqlDataAdapter(sql, ConfigurationManager.AppSettings["DB"]);
    DataTable dt = new DataTable();
    da.Fill(dt);
    return dt.CreateDataReader();
}

请问：DoSomething()执行完毕后，GetReader()中创建的DataTable如何被清理？核心问题为：如何正确处理DataTable的资源释放？

关于DataTable的资源清理逻辑 #

1. DataTable的托管特性
   DataTable是纯托管对象，不直接持有数据库连接这类非托管资源，因此不需要手动调用Dispose()方法。.NET垃圾回收器（GC）会自动扫描并回收所有不再被引用的DataTable实例。

2. 你的代码中DataTable的生命周期
   GetReader()里创建的dt，会被返回的DataTableReader持有引用。当DoSomething()执行完毕后，dr变量超出作用域、不再被引用，此时dt也失去了唯一的引用链路，GC会在合适的时机回收它占用的内存。

3. 微软示例不释放DataTable的原因
   因为DataTable没有需要手动释放的非托管资源，GC会自动处理其内存回收。而像SqlDataReader这类DataReader实现，持有数据库连接这类非托管资源，必须用using或手动调用Close()/Dispose()来及时释放连接资源。

正确处理DataTable的建议 #

• 无需手动调用Dispose：只要确保DataTable不再被任何变量或对象引用，GC会自动完成回收。
• 避免长期持有引用：如果DataTable存储了大量数据，不要将其长期保存在静态变量、全局集合等地方，及时切断引用让GC尽早回收内存。
• 关联对象的清理：如果DataTable关联了DataView或其他持有其引用的对象，要确保这些关联对象也被及时释放，否则DataTable会被持续引用无法回收。

针对你的代码的优化建议 #

当前代码中的DataTableReader未被释放，虽然它本身不持有非托管资源，但遵循最佳实践的话，建议用using包裹：

private void DoSomething()
{
    string Txt, sql;
    sql = "Select * from table;";
    using (DataTableReader dr = GetReader(sql))
    {
        while (dr.Read())
        {
            Txt = dr["Txt"] + "";
        }
    }
}

using块结束时会自动调用dr.Dispose()，切断它对内部DataTable的引用，让GC能更快地回收DataTable占用的内存。

内容的提问来源于stack exchange，提问作者Austin