明示的にIDisposableに変換

Question

私はいくつかのtry...catchブロック内の文字列のいくつかの必要な作業を行うためにいくつかのXmlReaderとXmlWriterオブジェクトを使用しています。

ので、私もfinallyブロックを追加し、NewReader.Close();とNewWriter.Close();を実行しています私は、表記using (XmlReader NewReader = XmlReader.Create(...))を使用してれる好ましい構文であることを知っているが、私は本当に好きではありません。

しかし、コード解析は、これらのオブジェクトが処分されていないと訴えているので、何とかDispose()メソッドを呼び出す必要があります。

これらのクラスでは、Dispose()メソッドが明示的に実装されているため、((IDisposable)(NewReader)).Dispose();と((IDisposable)(NewWriter)).Dispose();を使用する必要があります。

この手法には何らかの欠点がありますか？

Answer 1

C＃using文は常にあなたのためDisposeを呼び出します。それはroughtly以下に変換します。

XmlReader NewReader = XmlReader.Create(...); 
try 
{ 
    // do stuff on NewReader 
} 
finally 
{ 
    ((IDisposable)NewReader).Dispose(); 
} 

のでyouselfは、任意の値を追加しませんfinallyでそれを包みます。 CloseとDisposeはよく似ていますが、必ずしもそうではありません。このFxCopが正しいので、常にDisposeを呼び出す必要があります。これを行うときにはusingステートメントでこれを行うようにしてください。Closeを手動で呼び出す理由はありません。

Answer 2

usingステートメントは、実際には好ましい解決策です。これはC＃の慣用句です。これらのクラスは、すでにCloseの終了セマンティクスを持つメソッドを提供しているため、IDisposableを明示的に実装しています。私の賭けはDisposeがCloseに、あるいはその逆になることです。 しかし、それにはカウントしないでください。とにかくDisposeと呼ぶべきです。最後に

、すべてこれらは等価である：好ましい

1. 使用using、。
2. finallyブロックのCloseを呼び出し、静的解析の警告を抑制します。 finally上
3. コールDispose：((IDisposable)NewReader).Dispose();

Answer 3

usingを使用しないための十分な理由があります：オブジェクトの寿命は現在のブロック

よりも長いとusingを避けるための貧しい理由があるが生き残るために必要がある場合があります場合は

• は：
  • 「私は本当に好きではありません」

  コードに正当な理由がありますか？

  また、単純な拡張方法を使用すると、構文がきれいになり、もう一度きれいになることに注意してください。