ストリームオブジェクトに対してClose（）またはDispose（）を呼び出す必要がありますか？

以下のようなクラスStream、StreamReader、StreamWriterなどの実装IDisposableインタフェース。つまりDispose()、これらのクラスのオブジェクトでメソッドを呼び出すことができます。彼らはpublicと呼ばれるメソッドも定義しましたClose()。オブジェクトの処理が完了したら、何を呼び出せばよいのでしょうか。両方とも呼ぶとどうなりますか？

私の現在のコードはこれです：

using (Stream responseStream = response.GetResponseStream())
{
   using (StreamReader reader = new StreamReader(responseStream))
   {
      using (StreamWriter writer = new StreamWriter(filename))
      {
         int chunkSize = 1024;
         while (!reader.EndOfStream)
         {
            char[] buffer = new char[chunkSize];
            int count = reader.Read(buffer, 0, chunkSize);
            if (count != 0)
            {
               writer.Write(buffer, 0, count);
            }
         }
         writer.Close();
      }
      reader.Close();
   }
}

ご覧のとおり、各オブジェクトのメソッドusing()を自動的に呼び出す構造を作成しましたDispose()。しかし、Close()メソッドも呼び出します。正しいですか？

ストリームオブジェクトを使用する際のベストプラクティスを教えてください。:-)

MSDNの例ではusing()構成を使用せず、Close()メソッドを呼び出します。

• 方法：FTPでファイルをダウンロードする

いいですか？

Reflector.NETにすばやくジャンプすると、Close()メソッドStreamWriterは次のとおりです。

public override void Close()
{
    this.Dispose(true);
    GC.SuppressFinalize(this);
}

そしてStreamReader：

public override void Close()
{
    this.Dispose(true);
}

のDispose(bool disposing)オーバーライドStreamReaderは次のとおりです。

protected override void Dispose(bool disposing)
{
    try
    {
        if ((this.Closable && disposing) && (this.stream != null))
        {
            this.stream.Close();
        }
    }
    finally
    {
        if (this.Closable && (this.stream != null))
        {
            this.stream = null;
            /* deleted for brevity */
            base.Dispose(disposing);
        }
    }
}

StreamWriterこの方法は同様です。

したがって、コードを読むと、ストリームに対してClose()＆Dispose()を好きなだけ何度でも任意の順序で呼び出すことができることは明らかです。動作に変化はありません。

したがって、使用するのがより読みやすいかどうかDispose()、Close()および/またはusing ( ... ) { ... }。

私の個人的な好みは、using ( ... ) { ... }「はさみで走らない」ことを助けるので、可能な限り常に使用することです。

しかし、これは正確さを助けますが、読みやすさを低下させます。C＃には既に中かっこがたくさんあるので、実際に中かっこでストリームを閉じているかどうかを確認するにはどうすればよいでしょうか。

だから私はこれを行うのが最善だと思います：

using (var stream = ...)
{
    /* code */

    stream.Close();
}

コードの動作には影響しませんが、読みやすさを向上させます。

いいえ、これらのメソッドを手動で呼び出すことはできません。usingブロックの最後で、Dispose()メソッドが自動的に呼び出され、アンマネージリソースを解放します（少なくとも、ストリーム、リーダー/ライターなどの標準.NET BCLクラスの場合）。したがって、次のようなコードを書くこともできます。

using (Stream responseStream = response.GetResponseStream())
    using (StreamReader reader = new StreamReader(responseStream))
        using (StreamWriter writer = new StreamWriter(filename))
        {
            int chunkSize = 1024;
            while (!reader.EndOfStream)
            {
                 char[] buffer = new char[chunkSize];
                 int count = reader.Read(buffer, 0, chunkSize);
                 if (count != 0)
                 {
                     writer.Write(buffer, 0, count);
                 }
            }
         }

Close()メソッド呼び出しDispose()。

ドキュメントには、これら2つの方法は同等であると記載されています。

StreamReader.Close：このCloseの実装は、true値を渡してDisposeメソッドを呼び出します。

StreamWriter.Close：このCloseの実装は、true値を渡してDisposeメソッドを呼び出します。

Stream.Close：このメソッドはDisposeを呼び出し、trueを指定してすべてのリソースを解放します。

したがって、これらはどちらも等しく有効です。

/* Option 1, implicitly calling Dispose */
using (StreamWriter writer = new StreamWriter(filename)) { 
   // do something
} 

/* Option 2, explicitly calling Close */
StreamWriter writer = new StreamWriter(filename)
try {
    // do something
}
finally {
    writer.Close();
}

個人的には、「ノイズ」が少ないので、最初のオプションを使用します。

Close()とDispose()メソッドの両方をサポートする多くのクラスでは、2つの呼び出しは同等です。ただし、一部のクラスでは、閉じられたオブジェクトを再度開くことができます。そのようなクラスの中には、再オープンを許可するために、クローズ後もいくつかのリソースを存続させるものがあります。他のユーザーはでリソースを有効に保つことはできませんClose()が、Dispose()再オープンを明示的に禁止するフラグをオンに設定することがあります。

の規約IDisposable.Disposeでは、二度と使用されないオブジェクトで呼び出すことは最悪の無害であることを明示的に要求しているため、を呼び出すかどうかにかかわらず、すべてのオブジェクトでかIDisposable.Disposeメソッドを呼び出すことをお勧めします。Dispose()IDisposableClose()

これは古い質問ですが、今では、それぞれをブロックする必要なしにusingステートメントを書くことができます。収容ブロックが完成すると、逆の順序で廃棄されます。

using var responseStream = response.GetResponseStream();
using var reader = new StreamReader(responseStream);
using var writer = new StreamWriter(filename);

int chunkSize = 1024;
while (!reader.EndOfStream)
{
    char[] buffer = new char[chunkSize];
    int count = reader.Read(buffer, 0, chunkSize);
    if (count != 0)
    {
        writer.Write(buffer, 0, count);
    }
}

https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/proposals/csharp-8.0/using