c# - 方法返回一个 IDisposable - 我是否应该处理结果，即使它没有分配给任何东西？

Question

这似乎是一个相当简单的问题，但经过一番搜索后我找不到这个特定的用例。

假设我有一个简单的方法，比如说，确定文件是否被某个进程打开。我可以这样做(不是 100% 正确，但相当好):

public bool IsOpen(string fileName)
{
  try
  {
    File.Open(fileName, FileMode.Open, FileAccess.Read, FileShare.None);
  }
  catch
  {
    // if an exception is thrown, the file must be opened by some other process
    return true;
  } 
}

(显然这不是确定这一点的最佳甚至正确方法 - File.Open 抛出许多不同的异常，所有异常都具有不同的含义，但它适用于此示例)

现在 File.Open 调用返回一个 FileStream，并且 FileStream 实现了 IDisposable。通常我们希望将任何 FileStream 实例的使用包装在一个 using block 中，以确保它们被正确处理。但是如果我们实际上没有将返回值分配给任何东西，会发生什么呢？是否仍然需要像这样处理 FileStream:

try
{
  using (File.Open(fileName, FileMode.Open, FileAccess.Read, FileShare.None));
  { /* nop */ }
}
catch 
{
  return true;
}

我应该创建一个 FileStream 实例并处理它吗？

try
{
  using (FileStream fs = File.Open(fileName, FileMode.Open, FileAccess.Read, FileShare.None));
}
...

或者这些完全没有必要？我们可以简单地调用 File.Open 并且不将其分配给任何东西(第一个代码示例)，然后让 GC 立即处理它吗？

Answer 1

是的，您绝对应该处理 FileStream。否则，流将保持打开状态，文件将无法使用，直到终结器恰好将其清理干净。

这里重要的是所有权:对于File.Open，假定调用者“拥有”返回给它的流——如果您拥有实现的东西IDisposable，您有责任处置它。

将此与 Image.FromStream 的情况进行比较:在那种情况下，您传入一个流，然后 Image 假定 它拥有该流。您不能自己关闭流，在那种情况下 - 您必须在完成后处理图像，它会处理流。

调用一个返回一次性东西的静态方法几乎总是假设调用者获得了资源的所有权。同上构造函数(实际上是静态方法。)

在这种情况下具有讽刺意味的是，如果您不处理 File.Open 返回的流，您会发现该文件是可用的 - 在创建它的同时在不确定的时间之前无法使用。