何时检查此对象是否已被处理并抛出ObjectDisposedException是否合理?

时间:2011-05-27 09:38:07

标签: c# dispose idisposable objectdisposedexception

在实现IDisposable的类中,何时检查对象是否已被处理是合理的,如果有,则抛出ObjectDisposedException?在所有公共方法和属性中(Dispose除外)?有时?从未?

5 个答案:

答案 0 :(得分:4)

您应该仅在不适用于已处置对象的方法中实现该检查。

例如:
如果您的类关闭Dispose中的数据库连接或文件句柄,则需要这些数据库连接或文件句柄的所有方法都需要检查实例是否已经公开。

答案 1 :(得分:2)

唯一真正指定的是public void Dispose()本身应该抛出任何东西。

任何需要(非)托管资源的方法都应抛出。

在我看来,这只留下了一些有争议的案件:

  • IsOpen,IsDisposed:我不会抛出
  • 其他IsSomeStatus:它取决于。
  • 长度,数量,位置:我认为封闭的Stream没有长度,所以抛出

当类合并(不相关)函数(如Stream 集合)时,会变得更加困难。我们不应该这样做。

答案 2 :(得分:2)

如果一个对象支持IsDisposed,那么该方法本身就不应该抛出;如果IsDisposed返回true,则抛出许多其他方法是正确的,但异常应该由这些方法而不是IsDisposed生成。可能有一个实用程序方法AssertNotDisposed,如果处理了一个对象,它会抛出,但是这个行为可以从具有该名称的方法中获得。

否则,我建议在很多情况下让对象保持IDisposable对象是有用的,并且能够在保持有用状态的同时Dispose内部对象。例如,一个对象的功能是显示和维护一个无模式对话框以从用户获取信息,即使在关闭该框之后也可以保留字段内容的副本。这样的对象应该提供一个“Close”方法,该方法将处理内部Disposable对象但保持有用状态。虽然它也可以有一个Dispose方法,它会调用Close但是也设置一个“NoLongerValid”标志会导致字段属性抛出,我认为这不会真正添加任何值。

我将批准许多案例,其中一个对象在处置后可以保持有用的状态,这表明一个类别可能应该被拆分。例如,Font类可能应该拆分为非一次性FontInfo类(包含字体描述,但不包括GDI句柄)和IDisposable ReadyFont类(继承FontInfo,并封装GDI字体对象)。使用字体的例程可以检查给出的对象是FontInfo还是ReadyFont;在前一种情况下,他们可以创建一个GDI字体,使用它并释放它;在后一种情况下,他们可以使用ReadyFont的GDI字体对象并释放它。然后,ReadyFont的创建者将负责确保其清理。

实际上,我不知道系统在渲染控件时是否会尝试使用与控件的Font属性关联的GDI对象,但是我知道如果Font是Disposed它不会发出尖叫声(即使在将其分配给Font属性之前将其处置!)。如果需要,控件肯定能够创建新的GDI字体;我不知道他们是否总是创建一个新的GDI字体,或者他们是否只在旧的GDI字体处理后才这样做。前一种行为似乎更具性能,但除非仔细编码,否则如果一个线程在另一个线程正在使用它时尝试Dispose a Font,则会导致问题。

答案 3 :(得分:1)

正如您所说,我将在除Dispose和IsDisposed之外的所有公共方法和属性中实现此检查。这是standard pattern,以防止开发人员在错误的印象中使用您的类型,因为它仍然有效。

答案 4 :(得分:1)

使用已处置对象是您希望尽快找到的编程错误。

检查的地方越多,您发现错误的速度就越快 你肯定应该检查一个被处置的对象会导致其他类型的异常(即空指针),以免混淆用户。

在其他地方,如果值得努力,则取决于应用程序。