c# - 流的通用 IDisposable 包装器 - 捕获所有异常的替代方法

Question

所以我刚刚为一个类型编写了这个实现，该类型抽象文件并在它包含的流上实现IDisposable。它旨在用于任何项目:

public void Dispose()
{
  if (_contentStream == null)
    return;
  lock (_contentStreamLocker)
  {
    if (_contentStream == null)
      return;

    try
    {
      _contentStream.Dispose();
    }
    catch (Exception)
    {
      //legitimate? - don't let any exception prevent 
      //us from successfully disposing?
    }  
    _contentStream = null;
  }
}

所以我显然感觉很脏，因为:

• 我正在捕获异常
• 我没有对异常做任何事情

但是由于我无法保证底层流的行为(也考虑了自定义实现) - 即如果它已经被处置或者它处于实现决定处置不可能的状态 - 我真的看不出什么还有其他事情要做。

显然我不应该隐藏 Stream 的 Dispose() 的错误实现；但我认为这里有一个中途之家。

特别是，该对象不一定“拥有”流，因此不必始终负责 Dispose()ing。因此，它应该容忍流已被其他人处理。

显然，在大多数情况下这不会成为问题；也许在 using block 中获取一个文件，读取它，用它做一些事情，然后处理它。但也有一些边缘情况；他们打扰我!

最初我将其写为 catch(ObjectDisposeException){ } - 但这只是一种非正式模式，也不能依赖。

我该怎么办？保持这样，尽管吞噬异常是恶业？ Gulp only ObjectDisposeException，即使这也好不了多少？或者我根本不应该有任何异常吞咽或处理？

我可以看到所有情况的优点；只是无法弄清楚他们中的哪一个获胜。

Answer 1

一般来说，如果您无法确定哪一个获胜，那是因为存在无法“一般”处理的有用信息。在这种情况下，我认为最好让调用者来处理它，因为他们将处于一个具体的情况下，可以让他们更好地判断。

流是如何引入这个类的？如果您可以设计使其被传入，那么为什么不让调用者处理它的处理而不用关心它。 StreamReader 承担这个责任一直困扰着我。如果没有的话我也不会太在意。

请勿吞下异常，尤其是在供外部使用的代码中。