2

私がやろうとしているのは、プロジェクトごとの静的辞書を用意して、コードのどこからでも単一のリストにアクセスできるようにすることです。

今まで、私はこの解決策、パブリック プロパティを持つ public static クラスを思いつきました。

public static class Common
{
    public static StringDictionary Domains
    {
        get
        {
            StringDictionary list = new StringDictionary();

            list.Add("212", "Location A");
            list.Add("555", "Location B");
            list.Add("747", "Location C");
            list.Add("000", "Location D");

            return list;
        }
    }
}

私はこのように使用します（グリッドビューのセルのコンテンツを置き換えるために使用します）：

if (Common.Domains.ContainsKey(e.Row.Cells[5].Text))
{
    e.Row.Cells[5].Text = Common.Domains[e.Row.Cells[5].Text];
}
else
{
    e.Row.Cells[5].Text = "n/a";
}

しかし、これが効率的な解決策であるかどうか、および/またはこれを行う他の（より良い）方法があるかどうかはわかりません...誰かがヒントを与えることができますか?

前もってありがとう、アンドレア。

4

4 に答える 4

3

プロパティにアクセスするたびにリストを再作成したくないでしょう。構築を静的コンストラクターに移動します。

public static class Common
{
    private static StringDictionary _domains;
    static Common()
    {
        _domains = new StringDictionary();
        _domains.Add("212", "Location A");
        _domains.Add("555", "Location B");
        _domains.Add("747", "Location C");
        _domains.Add("000", "Location D");
    }
    public static StringDictionary Domains
    {
        get
        {
            return _domains;
        }
    }
}

また、返される辞書は読み取り専用ではないことに注意してください。したがって、このクラスのクライアントはコレクションを変更できます (たとえば、エントリの追加/削除)。

于 2009-04-16T15:44:41.853 に答える
0

@Jeff Fritzは彼の答えで正しいですが、複数のスレッド間でこのフィールドの共有アクセスについてもう少し追加したかっただけです。StringDictionaryは読み取り操作に対してのみスレッド セーフであり、書き込み操作に対してはスレッド セーフではないため、適切なロックなしでこのコレクションを変更しないように注意する必要があります。また、コレクションを反復処理するときにもコレクションをロックする必要があります。

繰り返すには:

lock(Common.Domains) {
    foreach(var domain in Common.Domains) {
    }
}

静的コンストラクターの外部にドメインを追加するには:

lock(Common.Domains) {
    Common.Domains.Add("111", "Location 3");
}

一般に、フィールドが変更されないように、静的フィールドを介して非読み取り専用コレクションを共有する場合は、非常に注意します。代わりにStringDictionaryをReadOnlyCollectionにして、フィールドを「読み取り専用」としてマークすることもできます。

public static readonly ReadOnlyCollection<Pair<string, string>> domains

O(1) ルックアップは取得できませんが、このコレクションは非常に小さいと想定しているため、問題にはなりません。

于 2009-04-16T19:24:38.687 に答える
0

乗るたびに辞書がリロードされますDomains

listインスタンス ディクショナリの読み取り専用の静的コピーをメモリにロードしてスタッシュする静的コンストラクタを構成します。

于 2009-04-16T15:44:24.350 に答える
0

代わりにシングルトン パターンの使用を検討してください。

http://en.wikipedia.org/wiki/Singleton_pattern

すべてのプログラムには独自の特別なニーズがあるため、必須ではありません。このパターンについて読んで、設計を再考することをお勧めします。

プログラムがマルチスレッド化されている場合は、StringDictionary がスレッド セーフであるかどうかを確認したい場合があります。スレッド セーフでない (そしてプログラムがマルチスレッド化されている) 場合は、クラスに同期メカニズムを追加する必要があります。

于 2009-04-16T15:47:11.330 に答える