Question

我有这个方法：

public Stream Load(string term)
{
    var url = CreateSearchUrl(term);

    var webRequest = (HttpWebRequest)WebRequest.Create(url);
    var webResponse = webRequest.GetResponse();

    return new GZipStream(webResponse.GetResponseStream(), CompressionMode.Decompress);
}

正如您所看到的，我将流返回给调用者，但我不确定这是否安全，因为WebRequest被运行时处理，从而使我返回的流无效。

我可以将它转换为字节数组并返回一个MemoryStream甚至使用WebClient但我只是不喜欢这个想法=）。

谢谢！

Answer 1

在没有资源泄漏的情况下，没有一种简单的方法可以安全地返回Stream。主要问题是处理WebResponse：

public Stream Load(string term)
{
    var url = CreateSearchUrl(term);

    var webRequest = (HttpWebRequest)WebRequest.Create(url);
    var webResponse = webRequest.GetResponse(); // whoops this doesn't get disposed!

    return new GZipStream(webResponse.GetResponseStream(), CompressionMode.Decompress);
}

关闭WebResponse实际上比关闭响应流更重要，因为关闭WebResponse会隐式关闭响应流。

我知道让WebResponse与Stream一起处理的唯一方法是在处理WebResponse（以及GZipStream）时围绕GZipStream实现一个装饰器。虽然这可行，但代码很多：

class WebResponseDisposingStream : Stream
{
    private readonly WebResponse response;
    private readonly Stream stream;

    public WebResponseDisposingStream(WebResponse response, Stream stream)
    {
        if (response == null)
            throw new ArgumentNullException("response");
        if (stream == null)
            throw new ArgumentNullException("stream");

        this.response = response;
        this.stream = stream;
    }

    public override void Close()
    {
        this.response.Close();
        this.stream.Close();
    }

    // override all the methods on stream and delegate the call to this.stream

    public override void Flush() { this.stream.Flush(); } // example delegation for Flush()
    // ... on and on for all the other members of Stream
}

也许更好的方法是继续传递样式，其中使用Stream的代码作为委托传递：

public void Load(string term, Action<Stream> action)
{
    var url = CreateSearchUrl(term);

    var webRequest = (HttpWebRequest)WebRequest.Create(url);

    using (var webResponse = webRequest.GetResponse())
    using (var responseStream = webResponse.GetResponseStream())
    using (var gzipStream = new GZipStream(responseStream, CompressionMode.Decompress))
    {
        action(gzipStream);
    }
}

现在调用者只需传入Stream应该做的事情。在下面，长度打印到控制台：

Load("test", stream => Console.WriteLine("Length=={0}", stream.Length));

最后一点：如果您不了解，HTTP内置了对压缩的支持。有关详细信息，请参阅Wikipedia。 HttpWebRequest通过AutomaticDecompression属性内置了对HTTP压缩的支持。使用HTTP压缩基本上使压缩对代码透明，并且使用HTTP工具（浏览器，小提琴等）也能更好地工作。

来自HttpWebRequest生命周期范围的响应流

1 个答案: