8

そのため、DI / IoCコンテナOpenNETCF.IoCに取り組んでおり、コンテナコレクション内のIDisposableアイテムのライフサイクル管理の形式を追加するための（合理的な）機能リクエストがあります。

私の現在の考えでは、オブジェクトをクエリして破棄されたかどうかを確認することはできず、オブジェクトが破棄されたときのイベントを取得できないため、開発者が必要とするオブジェクトのラッパーを作成する必要があります。管理するフレームワーク。

現在、オブジェクトはAddNewで追加できます（簡単にするために、オーバーロードは1つだけで、Addはないと仮定します）。

public TTypeToBuild AddNew<TTypeToBuild>() { ... }

私が検討しているのは、新しいメソッドを追加することです（それらのグループですが、全体像を把握できます）。

public DisposableWrappedObject<IDisposable> AddNewDisposable<TTypeToBuild>()
    where TTypeToBuild : class, IDisposable
{
    ...
}

DisposableWrappedObjectは次のようになります。

public class DisposableWrappedObject<T>
    where T : class, IDisposable
{
    public bool Disposed { get; private set; }
    public T Instance { get; private set; }

    internal event EventHandler<GenericEventArgs<IDisposable>> Disposing;

    internal DisposableWrappedObject(T disposableObject)
    {
        if (disposableObject == null) throw new ArgumentNullException();

        Instance = disposableObject;
    }

    ~DisposableWrappedObject()
    {
        Dispose(false);
    }

    public void Dispose()
    {
        Dispose(true);
    }

    protected virtual void Dispose(bool disposing)
    {
        lock(this)
        {
            if(Disposed) return;

            EventHandler<GenericEventArgs<IDisposable>> handler = Disposing;
            if(handler != null)
            {
                Disposing(this, new GenericEventArgs<IDisposable>(Instance));
            }

            Instance.Dispose();

            Disposed = true;
        }
    }
}

これで、アイテムがAddNewDIsposableを介してコンテナーに追加されると、イベントハンドラーも追加されるため、（ラッパーを介して）Disposedを取得すると、フレームワークはそのアイテムを基になるコレクションから削除します。

私は実際にこれを実装し、単体テストに合格していますが、これがどこで壊れているのか、または消費する開発者にとってより「友好的」になる方法についての意見を探しています。

編集1

Disposedイベントがどのように使用されるかについて質問があったので、ここにいくつかのコードがあります（重要なものにトリミングされています）：

private object AddNew(Type typeToBuild, string id, bool wrapDisposables)
{
    ....

    object instance = ObjectFactory.CreateObject(typeToBuild, m_root);

    if ((wrapDisposables) && (instance is IDisposable))
    {
        DisposableWrappedObject<IDisposable> dispInstance = new
               DisposableWrappedObject<IDisposable>(instance as IDisposable);
        dispInstance.Disposing += new 
               EventHandler<GenericEventArgs<IDisposable>>(DisposableItemHandler);
        Add(dispInstance as TItem, id, expectNullId);
        instance = dispInstance;
    }

    ....

    return instance;
}

private void DisposableItemHandler(object sender, GenericEventArgs<IDisposable> e)
{
    var key = m_items.FirstOrDefault(i => i.Value == sender).Key;
    if(key == null) return;
    m_items.Remove(key);
}
4

1 に答える 1

3

何かが足りないかもしれませんが、なぜAPIに新しいメソッドを追加するのですか？オブジェクトがコンテナに追加されると、キャストしてIDが可能かどうかを確認し、可能であれば適切に処理できます。

また、デストラクタが必要かどうか疑問に思っています。コンテナーがIDisposable（Unityのように）であると仮定すると、Basic Dispose Patternを実装するだけで、GCのオーバーヘッドを大幅に節約できます。

該当する可能性のあるいくつかの質問：

于 2009-10-13T04:13:53.580 に答える