14

静的メソッドを持つコードでアンマネージ ライブラリを使用する必要があります。ライブラリ操作を依存関係としてコードに導入したいと思います。また、静的メソッドのほかに、ライブラリには初期化メソッドと設定メソッドがあり、どちらもグローバルです。したがって、これをインスタンス クラスにラップすることはできません。1 つのインスタンスが設定を変更すると、他のすべてのインスタンスが影響を受け、1 つのインスタンスが初期化されると、他のすべてのインスタンスが再初期化されるからです。

シングルトンクラスとして導入することを考えました。このようにインスタンス クラスになりますが、インスタンスは 1 つしかないため、設定の変更や初期化について心配する必要はありません。このアプローチについてどう思いますか？私は依存性注入パターンにかなり慣れていないので、シングルトン パターンが適切な解決策であるかどうかわかりません。同様のケースに対するあなたの解決策は何ですか？

編集:初期化にもパラメーターが必要なため、メソッド呼び出しをロックして、呼び出されるたびに設定を再初期化して変更することはできません。

編集 2:いくつかのメソッドのシグネチャは次のとおりです。

public static void Initialize(int someParameter)
// Parameter can only be changed by re-initalization which
// will reset all the settings back to their default values.

public static float[] Method1(int someNumber, float[] someArray)

public static void ChangeSetting(string settingName, int settingValue)
4

2 に答える 2

10

起動時に一度だけ設定する必要がある場合は、独自の静的コンストラクターで静的クラスのすべての初期化を行う非静的ラッパークラスを作成することをお勧めします。そうすれば、それが1回だけ発生することを保証できます。

public class MyWrapper
{
    public MyWrapper()
    {
        // Do any necessary instance initialization here
    }

    static MyWrapper()
    {
        UnManagedStaticClass.Initialize();
        UnManagedStaticClass.Settings = ...;
    }

    public void Method1()
    {
        UnManagedStaticClass.Method1();
    }
}

ただし、呼び出すたびに設定を変更する必要があり、インスタンスをスレッドセーフにしたい場合は、静的オブジェクトをロックして、静的設定が誤って上書きされないようにすることをお勧めします。別のスレッドでまだ使用中：

public class MyWrapper
{
    public MyWrapper()
    {
        // Do any necessary instance initialization here
    }

    static MyWrapper()
    {
        UnManagedStaticClass.Initialize();
    }

    static object lockRoot = new Object();

    public void Method1()
    {
        lock (lockRoot)
        {
            UnManagedStaticClass.Settings = ...;
            UnManagedStaticClass.Method1();
        }
    }
}

クラスのインスタンスコンストラクターに初期化パラメーターを渡す必要がある場合は、静的フラグフィールドを使用することでそれを行うこともできます。

public class MyWrapper
{
    public MyWrapper(InitParameters p)
    {
        lock (lockRoot)
        {
            if (!initialized)
            {
                UnManagedStaticClass.Initialize(p);
                initialized = true;
            }
        }
    }

    static bool initialized = false;
    static object lockRoot = new Object();

    public void Method1()
    {
        lock (lockRoot)
        {
            UnManagedStaticClass.Settings = ...;
            UnManagedStaticClass.Method1();
        }
    }
}

毎回再初期化する必要があるが、再初期化が遅すぎるためにパフォーマンスが心配な場合、（恐ろしいシングルトン以外の）他の唯一のオプションは、再初期化が必要かどうかを自動検出することです。必要な場合にのみ実行してください。少なくとも、それが発生するのは、2つのスレッドが2つの異なるインスタンスを同時に使用している場合のみです。あなたはこのようにそれを行うことができます：

public class MyWrapper
{
    public MyWrapper(InitParameters initParameters, Settings settings)
    {
        this.initParameters = initParameters;
        this.settings = settings;
    }

    private InitParameters initParameters;
    private Settings settings;
    static MyWrapper currentOwnerInstance;
    static object lockRoot = new Object();

    private void InitializeIfNecessary()
    {
        if (currentOwnerInstance != this)
        {
            currentOwnerInstance = this;
            UnManagedStaticClass.Initialize(initParameters);
            UnManagedStaticClass.Settings = settings;
        }
    }

    public void Method1()
    {
        lock (lockRoot)
        {
            InitializeIfNecessary();
            UnManagedStaticClass.Method1();
        }
    }
}
于 2013-02-15T13:06:10.637 に答える
1

ステートレス サービス クラスを使用し、各メソッド呼び出しで静的クラスの状態情報を渡します。クラスの詳細を知らなくても、ac# 静的クラスを使用した別の例を示します。

public static class LegacyCode
{
    public static void Initialize(int p1, string p2)
    {
        //some static state
    }
    public static void ChangeSettings(bool p3, double p4)
    {
        //some static state
    }
    public static void DoSomething(string someOtherParam)
    {
        //execute based on some static state
    }
}

public class LegacyCodeFacadeService
{
    public void PerformLegacyCodeActivity(LegacyCodeState state, LegacyCodeParams legacyParams)
    {
        lock (_lockObject)
        {
            LegacyCode.Initialize(state.P1, state.P2);
            LegacyCode.ChangeSettings(state.P3, state.P4);
            LegacyCode.DoSomething(legacyParams.SomeOtherParam);
            //do something to reset state, perhaps
        }
    }
}

空白を少し埋める必要がありますが、アイデアが得られることを願っています。要点は、必要最小限の時間だけ静的オブジェクトの状態を設定し、その間ずっとアクセスをロックして、他の呼び出し元がグローバル状態の変更の影響を受けないようにすることです。このクラスを使用するには、このクラスの新しいインスタンスを作成する必要があるため、完全に注入可能でテスト可能です (インターフェースを抽出するステップを除きますが、簡潔にするためにスキップしました)。

ここでの実装には多くのオプションがあります。たとえば、LegacyCodeState を大幅に変更する必要があるが、少数の特定の状態のみに変更する必要がある場合は、それらの状態を管理する作業を行うオーバーロードを使用できます。

編集

これは多くの点でシングルトンよりも好ましいですが、最も重要なことは、蓄積してグローバル状態に結合することができないということです。これにより、グローバル状態が静的クラスへの唯一のエントリ ポイントである場合、グローバル状態が非グローバル状態に変わります。 . ただし、シングルトンが必要になった場合は、コンストラクターをここにカプセル化することで簡単に切り替えることができます。

public class LegacyCodeFacadeService
{
    private LegacyCodeFacadeService() { }

    public static LegacyCodeFacadeService GetInstance()
    {
        //now we can change lifestyle management strategies later, if needed
        return new LegacyCodeFacadeService();
    }

    public void PerformLegacyCodeActivity(LegacyCodeState state, LegacyCodeParams legacyParams)
    {
        lock (_lockObject)
        {
            LegacyCode.Initialize(state.P1, state.P2);
            LegacyCode.ChangeSettings(state.P3, state.P4);
            LegacyCode.DoSomething(legacyParams.SomeOtherParam);
            //do something to reset state, perhaps
        }
    }
}
于 2013-02-15T13:33:44.460 に答える