正确处理 responseStream 和 StreamReader 关闭

Question

我有一个用 C# 编写的独立处理程序 (.ASHX)，它处理带有 XML 有效负载的传入推送通知，然后解析这些 XML 并提取和处理从中获取的任何必要信息。问题是我无法在本地对此进行测试，因为我们的开发环境防火墙防火墙太严密，而且我不允许传入通知出现异常。

我已经在我们的测试环境中对此进行了测试，并且通知正在被正确接收和处理，但是在随机代码审查之后，看起来我处理responseStreams 和streamReaders 的方式可能存在问题。

我的问题是，这是处理这些资源关闭的有效方法还是可能导致NullReferenceException？

处理这些资源的标准做法是什么，我应该使用using 语句来保持它们仅用于该范围还是这段代码就足够了？

Stream responseStream = null;
StreamReader reader = null;
string serverResponse = null;

try
{
    responseStream = response.GetResponseStream();
    reader = new StreamReader(responseStream);
    serverResponse = reader.ReadToEnd();
}

finally
{
    if (reader == null)
    {
        reader.Close();
    }

    if (responseStream == null)
    {
        responseStream.Close();
    }

    response.Close();
 }

Answer 1

最好使用using 块：

string serverResponse = null;
using (Stream responseStream = response.GetResponseStream())
{
   using (StreamReader reader = new StreamReader(responseStream))
   {
       serverResponse = reader.ReadToEnd();
   }
}

这样，即使using 语句中出现异常，所有流也会正确关闭。

Answer 2

除了您的代码之外，我还想使用以下代码：

responseStream.Seek(0, SeekOrigin.Begin);

using (StreamReader sr = new StreamReader(responseStream))
{
    message = sr.ReadToEnd();
}

我会添加一个“catch”，如果您的响应流以某种方式失败怎么办，例如它在另一端提前关闭，协议等不匹配？

看看the CommunicationException Class

附：我个人不喜欢嵌套 using 语句，但如果必须使用它们，请查看 AggregateExceptions。