[解決済み] IDisposableが全クラスに広がるのを防ぐには?
質問
これらの簡単なクラスから始めてください...
このような簡単なクラスのセットがあるとします。
class Bus
{
Driver busDriver = new Driver();
}
class Driver
{
Shoe[] shoes = { new Shoe(), new Shoe() };
}
class Shoe
{
Shoelace lace = new Shoelace();
}
class Shoelace
{
bool tied = false;
}
A
Bus
には
Driver
は、その
Driver
は2つの
Shoe
があり、それぞれ
Shoe
には
Shoelace
. すべて非常に愚かなことです。
靴ひもにIDisposableオブジェクトを追加する
後で、私は
Shoelace
に対するある操作がマルチスレッドになる可能性があると判断し、そのために
EventWaitHandle
を追加しています。そのため
Shoelace
はこのようになります。
class Shoelace
{
private AutoResetEvent waitHandle = new AutoResetEvent(false);
bool tied = false;
// ... other stuff ..
}
靴ひもにIDisposableを実装する
しかし、現在は マイクロソフトの FxCop は文句を言うでしょう。 Shoelace' は次の IDisposable タイプのメンバーを作成するので、IDisposable を実装してください: 'EventWaitHandle'."
さて、私が実装したのは
IDisposable
で
Shoelace
で、私の小さなクラスはこのようなひどい混乱に陥ります。
class Shoelace : IDisposable
{
private AutoResetEvent waitHandle = new AutoResetEvent(false);
bool tied = false;
private bool disposed = false;
public void Dispose()
{
Dispose(true);
GC.SuppressFinalize(this);
}
~Shoelace()
{
Dispose(false);
}
protected virtual void Dispose(bool disposing)
{
if (!this.disposed)
{
if (disposing)
{
if (waitHandle != null)
{
waitHandle.Close();
waitHandle = null;
}
}
// No unmanaged resources to release otherwise they'd go here.
}
disposed = true;
}
}
あるいは、(コメント欄で指摘されているように、)以降に
Shoelace
には管理されていないリソースがないため、より単純な dispose の実装を使うこともできます。
Dispose(bool)
とDestructorを必要としない、よりシンプルなdisposeの実装を使うかもしれません。
class Shoelace : IDisposable
{
private AutoResetEvent waitHandle = new AutoResetEvent(false);
bool tied = false;
public void Dispose()
{
if (waitHandle != null)
{
waitHandle.Close();
waitHandle = null;
}
GC.SuppressFinalize(this);
}
}
IDisposableの拡散を恐る恐る見る
なるほど、これで解決ですね。しかし、今度は FxCop が次のように文句を言うでしょう。
Shoe
を作成します。
Shoelace
というように
Shoe
は
IDisposable
も必要です。
そして
Driver
は
Shoe
だから
Driver
は、必ず
IDisposable
.
そして
Bus
は
Driver
だから
Bus
は、必ず
IDisposable
といった具合になります。
突然ですが、私の小さな変更で
Shoelace
への小さな変更が多くの作業を引き起こし、上司はなぜ私が
Bus
を変更するために
Shoelace
.
質問
どのようにしたら、この
IDisposable
が広がるのを防ぎつつ、アンマネージドオブジェクトが適切に廃棄されるようにするにはどうしたらよいでしょうか。
どのように解決するのですか?
IDisposableの拡散を本当に"防止"することはできません。のように、ディスポーザブルにする必要があるクラスもあります。
AutoResetEvent
のように、廃棄される必要があるクラスもあり、最も効率的な方法は
Dispose()
メソッドで行うのが最も効率的です。しかし、このメソッドは何らかの形で呼び出されなければならないので、まさにあなたの例のように、IDisposableをカプセル化したり含むクラスはこれらを廃棄しなければならないので、同様に廃棄されなければならない、などです。それを回避する唯一の方法は
- 可能な限り IDisposable クラスの使用を避ける、単一の場所でイベントをロックまたは待機する、単一の場所で高価なリソースを保持する、など。
-
必要なときだけ作成し、直後に廃棄する(
using
パターン)
IDisposable はオプションのケースをサポートしているため、無視できるケースもあります。 例えば、WaitHandleは、名前付きMutexをサポートするためにIDisposableを実装しています。 名前が使用されていない場合、Disposeメソッドは何もしません。 MemoryStreamもその一例で、システムリソースを一切使わず、Disposeの実装も何もしない。アンマネージドリソースが使用されているかどうかを注意深く考えることは、有益な情報になります。また、.net ライブラリの利用可能なソースを調査したり、デコンパイラーを使用することもできます。
関連
-
[解決済み】文字列が有効な DateTime " format dd/MM/yyyy " として認識されなかった。
-
[解決済み】ソケットのアドレス(プロトコル/ネットワークアドレス/ポート)は、通常1つしか使用できない?
-
[解決済み] DBNullから他の型にオブジェクトをキャストすることができない
-
[解決済み】リソースの読み込みに失敗した:ステータス500(内部サーバーエラー)のサーバーの応答)
-
[解決済み】取り消せないメンバはメソッドのように使えない?
-
[解決済み] [Solved] 不正な文字列値: '\xEFxBFxBD' for column
-
[解決済み】Swashbuckle/Swagger + ASP.Net Core: "Failed to load API definition" (API定義の読み込みに失敗しました
-
[解決済み】値をNULLにすることはできません。パラメータ名:source
-
[解決済み】Linq 構文 - 複数列の選択
-
[解決済み] 他のスレッドからGUIを更新するにはどうすればよいですか?
最新
-
nginxです。[emerg] 0.0.0.0:80 への bind() に失敗しました (98: アドレスは既に使用中です)
-
htmlページでギリシャ文字を使うには
-
ピュアhtml+cssでの要素読み込み効果
-
純粋なhtml + cssで五輪を実現するサンプルコード
-
ナビゲーションバー・ドロップダウンメニューのHTML+CSSサンプルコード
-
タイピング効果を実現するピュアhtml+css
-
htmlの選択ボックスのプレースホルダー作成に関する質問
-
html css3 伸縮しない 画像表示効果
-
トップナビゲーションバーメニュー作成用HTML+CSS
-
html+css 実装 サイバーパンク風ボタン
おすすめ
-
[解決済み】指定されたキャストが有効でない?
-
[解決済み】GDI+、JPEG画像をMemoryStreamに変換する際にジェネリックエラーが発生しました。
-
[解決済み】スクリプトクラスが見つからないので、スクリプトコンポーネントを追加できない?
-
[解決済み】C#におけるtypedefの等価性
-
[解決済み] [Entity Framework 4.1でエンティティに関連オブジェクトを追加する際に、エンティティオブジェクトをIEntityChangeTracker.の複数のインスタンスから参照できない。
-
[解決済み】リソースの読み込みに失敗した:ステータス500(内部サーバーエラー)のサーバーの応答)
-
[解決済み】「...は'型'であり、与えられたコンテキストでは有効ではありません」を解決するにはどうすればよいですか?(C#)
-
[解決済み】Visual Studio: 操作を完了できませんでした。パラメータが正しくありません
-
[解決済み】エラー「必要なフォーマルパラメータに対応する引数が与えられていない」を解決する?
-
[解決済み】スレッド終了またはアプリケーションの要求により、I/O操作が中断されました。