c# - 読み取り専用のFileStreamを破棄する必要があります

Question

メソッドのローカル変数である読み取り専用のFileStreamがあります。

public void SomeMethod()
{
   var fileStream = File.Open(fileName, FileMode.Open, FileAccess.Read, FileShare.ReadWrite));
   ... //some stream operations
}

メソッドの最後でDisposeを（明示的にまたは「使用して」）呼び出す必要がありますか？何が変わりますか？

score 4 · Accepted Answer

それが何をするかは問題ではなく、IDisposableによって実装されるため、使用が終了したら暗黙的または明示的にFileStream呼び出す必要があります。Disposeこれは、コードでクラスを使用する契約の一部です。

score 3 · Accepted Answer

usingそれを文にまとめた方がいいと思います。そのコードを堅牢にしたい場合は、例外処理も必要です。それは今のように機能しますが、それはただの悪い習慣です。

score 3 · Accepted Answer

実装しているインスタンスを破棄することは必須であり、ステートメントIDisposableを通じて破棄することをお勧めします。using

特にすべてのケースを分析することが役立つとは思わないでください。

score 2 · Accepted Answer

破棄に失敗した場合FileStream、GC が起動するまで (非決定論的に) は閉じられません。

そして、これが起こるまで、開いているファイルハンドルを保持し、ファイルへのいくつかのタイプのアクセス (書き込み、削除など) を防ぎます。

FileShare.ReadWrite指定が必ずしも役立つとは限らないことに注意してください。別のライターがFileShare.None(たとえばを呼び出してFile.OpenWrite) を使用してファイルを開こうとした場合、ファイルを閉じるまでそのライターは開くことができません。

したがって、はい、usingステートメントでファイルを閉じてください。

score 0 · Accepted Answer

オブジェクトを破棄できる場合は、必要がなくなったらできるだけ早く破棄する必要があります。FileStream クラスのトピックから:

ファイルの一部がロックされた状態でプロセスが終了した場合、または未解決のロックがあるファイルを閉じた場合の動作は未定義です。

score 0 · Accepted Answer

誰もが示唆したように：IDisposable好ましくはと一緒に、すべてを処分してusingください。

ファイルの場合、他のすべての人がファイルにアクセス/変更するのを本当にブロックしたい場合があります。この場合、ある時点でまだファイルを破棄しますが、この「ある時点」は、コード/アプリケーションの有効期間のかなり後である可能性があります。

c# - 読み取り専用のFileStreamを破棄する必要があります

6 に答える 6

Related

Reference