在 C# 中使用空 using 语句关闭一次性对象是一种好习惯吗？

Question

我正在开发一个可以发送 ftp 请求的类，它有一个实用方法可以执行不同类型的 ftp 方法：

private FtpWebResponse DoFttpRequest(Uri uri, NetworkCredential credentials, string method, string file = null)
{
    var request = (FtpWebRequest)WebRequest.Create(uri);
    request.Credentials = credentials;
    request.Method = method;

    if (!string.IsNullOrEmpty(file))
    {
        using (var stream = request.GetRequestStream())
        using (var writer = new StreamWriter(stream))
        {
            writer.Write(file);
        }
    }

    return (FtpWebResponse)request.GetResponse();
}

如您所见，此方法执行 ftp 方法并将响应流返回给调用者。下面是使用该方法通过ftp将字符串内容写入文件的客户端方法：

public void WriteToFile(string path, string contents)
{
    var uri = new Uri(path);
    using (var ftpResponse = DoFttpRequest(uri, _credentials, Ftp.UploadFile, contents)) { }
}

如您所见，这里我使用空 using 语句 using (var ftpResponse = DoFttpRequest(uri, _credentials, Ftp.UploadFile, contents)) { } 来处理接收到的流。 这是处理这样的对象的好方法吗？是否有必要处置这个流，因为它可能会被垃圾收集器处置？

Answer 1

是否有必要处置此流，因为它可能会 反正由垃圾收集器处理

您可以使用这个简单的代码来查看不处理响应流可能会如何完全破坏应用程序。我使用 http 请求而不是 ftp 来简化测试，但这同样适用于 ftp 请求。

public class Program {
    static void Main(string[] args) {
        // this value is *already* 2 by default, set for visibility
        ServicePointManager.DefaultConnectionLimit = 2;
        // replace example.com with real site
        DoFttpRequest("http://example.com");
        DoFttpRequest("http://example.com");
        DoFttpRequest("http://example.com");
        Console.ReadLine();
    }

    private static HttpWebResponse DoFttpRequest(string uri) {
        var request = (HttpWebRequest) WebRequest.Create(uri);
        var response = (HttpWebResponse) request.GetResponse();
        Console.WriteLine("got response");
        return response;
    }
}

请注意，您不会处理 HttpWebResponse。将会发生的情况是您将在控制台中看到 2 条“得到响应”消息，然后应用程序将挂起尝试第三次获得响应。这是因为每个端点（每台主机）的并发连接限制为 2，因此虽然与主机（此处为 example.com）的 2 个连接“正在进行” - 与同一主机的下一个连接必须等待它们完成。因为您不处理响应 - 在 GC 收集它们之前，这些连接不会“完成”。在那之前 - 您的应用程序挂起，然后因超时而失败（如果 request.Timeout 设置为某个合理的时间）。所有后续请求也会挂起，然后因超时而失败。如果您处理响应 - 应用程序将按预期工作。

因此，请务必丢弃一次性物品。不需要使用块，你可以做DoFtpRequest(..).Dispose()。但是，如果您更喜欢空使用 - 至少不要声明不必要的变量，只需执行 using (DoFttpRequest(..)) {}。在空使用和Dispose 之间进行选择时要注意的一件事是DoFtpRequest 返回null 的可能性，因为如果它返回null - 显式Dispose 将抛出NullReferenceException 而空使用只会忽略它（你可以这样做DoFttpRequest(...)?.Dispose(); 如果您期望 null 但不想使用 using)。

Answer 2

using 语句实际上是执行某种代码，然后简单地调用 Dispose 方法。 这就是为什么你只能使用它的类型继承自 IDisposible 接口（在大多数情况下）

因此您实际上不必使用 using 语句。只需调用

DoFttpRequest(uri, _credentials, Ftp.UploadFile, contents)).Dispose()

如果您不自行处置和反对，垃圾收集器会在范围完成后自动处置它。 当您使用 c#、java 等高级语言时，您不必过多考虑内存......它们被称为 内存管理语言。他们会为您处理这类员工。