DataSet和DataTable都实现了IDisposable，因此，根据传统的最佳实践，我应该调用它们的Dispose()方法。
然而，从我目前所读到的内容来看，DataSet和DataTable实际上没有任何非托管资源，因此Dispose()实际上并没有做太多工作。
另外，我不能只使用using(DataSet myDataSet…)，因为DataSet有一个数据表的集合。
因此，为了安全起见，我需要遍历myDataSet。表，释放每个数据表，然后释放数据集。
那么，对我所有的数据集和数据表调用Dispose()是否值得呢?
附录:
对于那些认为数据集应该被处理的人: 一般来说，处理的模式是use using或try..最后，因为您希望保证Dispose()将被调用。
然而，对于集合来说，这很快就会变得很糟糕。例如，如果对Dispose()的调用之一抛出异常，该怎么办?是否吞下它(这是“坏的”)以便继续处理下一个元素?
或者，您是否建议我只调用myDataSet.Dispose()，而忘记在myDataSet.Tables中处理数据表?
当前回答
每当一个对象实现IDisposeable时，我就调用dispose。它的存在是有原因的。
数据集可能占用大量内存。它们越早被标记为清理越好。
更新
我已经5年没有回答这个问题了。我仍然同意我的答案。如果存在dispose方法，则应该在处理完对象时调用该方法。实现IDispose接口是有原因的。
其他回答
如果您的意图或这个问题的上下文实际上是垃圾收集，那么您可以显式地将数据集和数据表设置为null，或者使用关键字using并让它们超出范围。Dispose并不像Tetraneutron之前说的那样。GC将收集不再引用的数据集对象以及超出范围的对象。
我真的希望SO能强迫人们在投票之前写一条评论。
即使对象没有非托管资源，通过打破对象图，处理也可以帮助GC。一般来说，如果一个对象实现了IDisposable，则应该调用Dispose()。
Dispose()是否实际执行某些操作取决于给定的类。对于DataSet, Dispose()实现继承自MarshalByValueComponent。它从容器中移除自身并调用dispose事件。源代码如下(用。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);
}
}
}
}
}
这是正确处置数据表的方法。
private DataTable CreateSchema_Table()
{
DataTable td = null;
try
{
td = new DataTable();
//use table DataTable here
return td.Copy();
}
catch { }
finally
{
if (td != null)
{
td.Constraints.Clear();
td.Clear();
td.Dispose();
td = null;
}
}
}
这里有几个讨论，解释了为什么Dispose对于DataSet不是必要的。
处理还是不处理?
DataSet中的Dispose方法存在只是因为继承的副作用——换句话说，它在终结中实际上没有做任何有用的事情。
应该在数据表和数据集对象上调用Dispose吗?其中包括一位最有价值球员的解释:
这个系统。数据命名空间(ADONET)不包含 非托管资源。因此，没有必要处理任何这些作为 只要你没有给自己加上什么特别的东西。
了解Dispose方法和数据集?有来自权威斯科特·艾伦的评论:
在实践中，我们很少处理数据集，因为它没有什么好处。”
所以，共识是，目前没有好的理由调用Dispose在一个数据集上。
这可能是处置和释放DataSet所消耗的内存的最佳/适当的方式。
try
{
DataSet ds = new DataSet("DS");
//use table DataTable here
}
catch { }
finally
{
if (ds != null)
{
ds.EnforceConstraints = false;
ds.Relations.Clear();
int totalCount = ds.Tables.Count;
for (int i = totalCount - 1; i >= 0; i--)
{
DataTable td1 = ds.Tables[i];
if (td1 != null)
{
td1.Constraints.Clear();
td1.Clear();
td1.Dispose();
td1 = null;
}
}
ds.Tables.Clear();
ds.Dispose();
ds = null;
}
}
