c# - 我是否必须关闭我的代码从未访问过的 HttpRequest 输入流？

Question

我有一些遗留代码在 ASP.NET 中运行，其工作方式如下:

void HandleRequest(HttpContext context)
{
    try {
       if( some condition ) {
          try {
             doSomething( context.Request.InputStream );
          } finally {
             context.Request.InputStream.Close();
          }
       }
       // moar code follows which doesn't access InputStream
    } finally {
        context.Request.InputStream.Close();
    }
}

请注意，无论 InputStream 是否被访问，它都会被检索并关闭。我想这是在浪费时间，第二个 .Close() 是多余的。

我是否必须关闭我的代码从未访问过的 HttpRequest 输入流？

Answer 1

因为您没有创建 InputStream，所以您不应该负责关闭它。

诸如此类的规则可以让我们保持清醒 - 如果您负责关闭它，那么您将负责哪些其他对象？ HttpContext 有很多属性需要检查。如果他们向 HttpContext 添加新属性，现有代码会发生什么情况？

如果他们希望您对关闭它负责，则应该改为 public Stream CreateInputStream()，绝对不是属性。