我应该Dispose()DataSet和DataTable吗？

Question

DataSet和DataTable都实现了IDisposable,因此,通过传统的最佳实践,我应该调用它们的Dispose()方法.

但是,从我到目前为止所读到的,DataSet和DataTable实际上并没有任何非托管资源,因此Dispose()实际上并没有做太多.

另外,我不能只使用,using(DataSet myDataSet...)因为DataSet有一组DataTables.

所以,为了安全起见,我需要遍历myDataSet.Tables,处理每个DataTable,然后处理DataSet.

那么,在我的所有DataSet和DataTables上调用Dispose()是否值得麻烦？

附录:

对于那些认为应该处理DataSet的人:通常,处理的模式是使用usingor try..finally,因为你想保证将调用Dispose().

然而,这对于一个集合来说真的很快.例如,如果对Dispose()的一个调用抛出异常,你会怎么做？你吞下它(这是"坏"),以便你可以继续处理下一个元素？

或者,你是否建议我只调用myDataSet.Dispose(),而忘记在myDataSet.Tables中处理DataTables？

Answer 1

以下是一些讨论,解释了为什么Dispose不是DataSet所必需的.

处置或不处理？:

DataSet中的Dispose方法仅存在因为继承的副作用 - 换句话说,它实际上并没有在最终化中做任何有用的事情.

应该在DataTable和DataSet对象上调用Dispose吗？包括MVP的一些解释:

system.data命名空间(ADONET)不包含非托管资源.因此,只要你没有添加一些特殊的东西,就没有必要处置它们.

了解Dispose方法和数据集？有权威Scott Allen的评论:

在实践中,我们很少处理DataSet,因为它提供的好处很少"

因此,一致认为目前没有充分的理由在DataSet上调用Dispose.

Answer 2

更新(2009年12月1日):

我想修改这个答案,并承认原来的答案是有缺陷的.

原始分析确实适用于需要最终确定的对象 - 并且如果没有准确,深入的理解,表面上不应接受的做法仍然存在.

但是,事实证明,DataSets,DataViews,DataTables 在其构造函数中抑制了最终化 - 这就是为什么在它们上面调用Dispose()显然什么也不做.

据推测,这是因为他们没有非托管资源; 因此,尽管MarshalByValueComponent允许非托管资源,但这些特定实现并不需要,因此可以放弃最终确定.

(.NET作者会注意抑制通常占用最多内存的类型的最终化,这说明了这种实践对于最终类型的重要性.)

尽管如此,自从.NET Framework(大约8年前)开始以来,这些细节仍然没有得到充分记录(这是非常令人惊讶的)(您基本上可以通过自己的设备筛选出相互冲突的,模糊不清的材料来将这些碎片放在一起有时令人沮丧,但确实提供了对我们每天依赖的框架的更完整的理解.

经过大量阅读,这是我的理解:

如果一个对象需要最终化,它可能占用内存的时间超过它需要的时间 - 这就是为什么:a)任何定义析构函数的类型(或从定义析构函数的类型继承)都被认为是可终结的; b)在分配时(在构造函数运行之前),指针放在Finalization队列上; c)最终目标通常需要回收2个集合(而不是标准1); d)抑制终结不会从终结队列中删除对象(由SOS中的!FinalizeQueue报告)此命令具有误导性; 知道最终化队列中的对象(在其中本身)是没有用的; 知道什么对象在终结队列中并且仍然需要最终化将是有帮助的(是否有一个命令？)

抑制终结在对象的头中稍微关闭,向运行时指示它不需要调用其Finalizer(不需要移动FReachable队列); 它保留在Finalization队列中(并继续由SOS中的!FinalizeQueue报告)

DataTable,DataSet,DataView类都以MarshalByValueComponent为根,这是一个可以(可能)处理非托管资源的可终结对象

• 由于DataTable,DataSet,DataView不会引入非托管资源,因此它们会抑制其构造函数中的最终化
• 虽然这是一种不寻常的模式,但它使调用者不必担心在使用后调用Dispose
• 这可能是DataTables可能在不同的DataSet之间共享的事实,这可能是DataSets不关心处理子DataTables的原因.
• 这也意味着这些对象将出现在SOS中的!FinalizeQueue下
• 但是,在单个集合之后,这些对象仍然可以回收,就像它们的非最终对应物一样

4(新参考):

• http://www.devnewsgroups.net/dotnetframework/t19821-finalize-queue-windbg-sos.aspx
• http://blogs.msdn.com/tom/archive/2008/04/28/asp-net-tips-looking-at-the-finalization-queue.aspx
• http://issuu.com/arifaat/docs/asp_net_3.5unleashed
• http://msdn.microsoft.com/en-us/magazine/bb985013.aspx
• http://blogs.msdn.com/tess/archive/2006/03/27/561715.aspx

原答案:

对此有很多误导性和一般非常差的答案 - 任何落在这里的人都应该忽略噪音并仔细阅读下面的参考文献.

毫无疑问,应该在任何Finalizable对象上调用Dispose .

DataTables 是 Finalizable.

调用Dispose可以显着加快内存的回收速度.

MarshalByValueComponent在其Dispose()中调用GC.SuppressFinalize(this) - 跳过这意味着在回收内存之前必须等待数十个(如果不是数百个)Gen0集合:

通过对最终化的基本理解,我们已经可以推导出一些非常重要的事情:

首先,需要完成的对象比没有完成的对象长寿.事实上,他们可以活得更久.例如,假设gen2中的对象需要最终确定.最终确定将被安排,但对象仍然在gen2中,因此在下一代gen2集合发生之前不会重新收集.这确实可能是一段很长的时间,事实上,如果事情进展顺利,那将是很长一段时间,因为gen2集合成本很高,因此我们希望它们很少发生.需要最终确定的旧对象可能必须等待几十个(如果不是数百个)gen0集合才能回收它们的空间.

其次,需要定型的物体会造成附带损害.由于内部对象指针必须保持有效,因此不仅直接需要最终化的对象会在内存中停留,而且对象所引用的所有内容也将直接或间接地保留在内存中.如果一个巨大的对象树被一个需要最终化的对象锚定,那么整个树都会徘徊,可能会像我们刚才讨论的那样长时间停留.因此,谨慎使用终结器并将它们放在具有尽可能少的内部对象指针的对象上是很重要的.在我刚给出的树例子中,您可以通过将需要完成的资源移动到单独的对象并在树的根中保持对该对象的引用来轻松避免该问题.通过适度的更改,只有一个对象(希望是一个漂亮的小对象)会停留,最终化成本最小化.

最后,需要完成的对象为终结器线程创建工作.如果你的终结过程是一个复杂的过程,那么唯一的终结者线程将花费大量时间来执行这些步骤,这可能导致工作积压,从而导致更多对象等待最终确定.因此,终结者尽可能少地工作是至关重要的.还要记住,虽然所有对象指针在最终确定期间仍然有效,但可能是这些指针导致已经完成的对象,因此可能不太有用.即使指针有效,最好避免在终结代码中遵循对象指针.安全,简短的终结代码路径是最好的.

从Gen2中看到100多个未引用的DataTables MB的人那里得到它:这非常重要,完全错过了这个帖子的答案.

参考文献:

1 - http://msdn.microsoft.com/en-us/library/ms973837.aspx

2 - http://vineetgupta.spaces.live.com/blog/cns!8DE4BDC896BEE1AD!1104.entry http://www.dotnetfunda.com/articles/article524-net-best-practice-no-2-improve-garbage使用集电极-性能- finalizedispose-pattern.aspx

3 - http://codeidol.com/csharp/net-framework/Inside-the-CLR/Automatic-Memory-Management/

Answer 3

您应该假设它执行了一些有用的操作并调用Dispose,即使它当前没有任何操作.NET Framework的版本,不能保证它将在未来版本中保持这种方式导致资源使用效率低下.

Answer 4

即使对象没有非托管资源,处置也可能通过破坏对象图来帮助GC.通常,如果object实现了IDisposable,则应调用Dispose().

Dispose()实际上是否做某事取决于给定的类.在DataSet的情况下,Dispose()实现继承自MarshalByValueComponent.它从容器中删除自己并调用Disposed事件.源代码如下(用.NET Reflector反汇编):

protected virtual void Dispose(bool disposing)
{
    if (disposing)
    {
        lock (this)
        {
            if ((this.site != null) && (this.site.Container != null))
            {
                this.site.Container.Remove(this);
            }
            if (this.events != null)
            {
                EventHandler handler = (EventHandler) this.events[EventDisposed];
                if (handler != null)
                {
                    handler(this, EventArgs.Empty);
                }
            }
        }
    }
}

Answer 5

你自己创建DataTables吗？因为通常不需要遍历任何Object的子节点(如在DataSet.Tables中),因为它是Parent的工作来处置它的所有子成员.

通常,规则是:如果您创建了它并且它实现了IDisposable,请将其公开.如果你没有创建它,那么不要处置它,这是父对象的工作.但每个对象可能都有特殊规则,请查看文档.

对于.net 3.5,它明确地说"不再使用时处理它",这就是我要做的.

Answer 6

我可以在对象实现IDisposeable的任何时候调用dispose.这是有原因的.

DataSet可能是巨大的内存耗尽.他们越早被标记为清理就越好.

更新

我回答这个问题已经5年了.我仍然同意我的回答.如果存在dispose方法,则在完成对象时应调用它.IDispose接口的实现有一个原因.

Answer 7

如果您的意图或此问题的上下文确实是垃圾收集，那么您可以将数据集和数据表显式设置为 null 或使用关键字 using 并让它们超出范围。Dispose 并没有像 Tetraeutron 之前所说的那样做太多。GC 将收集不再引用的数据集对象以及超出范围的数据集对象。

我真的希望 SO 强迫人们在拒绝投票之前实际写评论。