キャッシュにデータをロードするためのロック

Question

私はヘルパークラスをWebアプリケーションに持っていますが、一般的で不変なデータオブジェクトを静的プロパティとして提示することがあります。これらのオブジェクトを次のように読み込んでいます：

public static class MyWebHelper 
{ 
    #region - Constants & Fields 
    private const string LocationCacheKey = "MyWebHelper_Locations"; 
    private static object LocationLoadLock = new object(); 
    private static MemoryCache m_SiteCache; 
    #endregion 

    #region - Properties 
    /// <summary> 
    /// Gets the uneditable collection of locations. 
    /// </summary> 
    public static ReadOnlyCollection<Location> Locations 
    { 
     get 
     { 
      EnsureLocationsCache(); 
      return (ReadOnlyCollection<Location>)SiteCache[LocationCacheKey]; 
     } 
    } 

    /// <summary> 
    /// Gets the MemoryCache object specific for my site cache. 
    /// </summary> 
    public static MemoryCache SiteCache 
    { 
     get 
     { 
      if (m_SiteCache == null) 
      { 
       m_SiteCache = new MemoryCache("MyWeb_SiteCache"); 
      } 
      return m_SiteCache; 
     } 
    } 
    #endregion 

    #region - Methods 
    private static void EnsureLocationsCache() 
    { 
     lock (LocationLoadLock) 
     { 
      if (SiteCache[LocationCacheKey] == null) 
      { 
       // 
       // Load all Locations from the database and perform default sorting on location code. 
       // 
       List<Location> lLocations = DataAccess.GetAllLocations(); 
       lLocations.Sort(delegate(Location l1, Location l2) { return string.CompareOrdinal(l1.Code, l2.Code); }); 
       SiteCache[LocationCacheKey] = new ReadOnlyCollection<Location>(lLocations); 
      } 
     } 
    } 
    #endregion 
} 

私の質問は何ですか？私はデータベースへの呼び出しを減らそうとしていますが、オーバーヘッドを導入するだけのロックですか？キャッシュされたデータはサイト全体でかなり一般的に使用されており、ほとんど変更されません。また、私は正しい場所にロックしていますか？

Answer 1

サーバへの各Webリクエストで新しいスレッドが作成され、静的ヘルパーがこれらのスレッド間で共有されるため、いくつかのタイプのロックが便利です。ロックがなければ、データベース読み込みがすでに進行中のときにEnsureLocationsCacheメソッドへの入力を危険にさらします。これで2回ロードされたデータの正確さには影響しませんが、DB読み取りが高価な場合は、全体的なパフォーマンスに影響を与え、キャッシングの効果を無効にします。

これは、実際には、アプリケーションの起動でEnsureLocationsCache（）メソッドにアクセスしようとする同時スレッドの量に依存します。これは、各LocationCacheKeyに対して1回限りの呼び出しであるため低い可能性があります。

ロックを取得するオーバーヘッドは、キャッシュがすでにロードされている場合でもコードがロックを取得するため、有効な問題です。 @asawyer、@TomTom、@Joeはいくつかの選択肢を提案しています。

EDIT：

がstatic constructorでEnsureLocationsCache（）を呼び出すことを検討してください。この場合、ロックは一切必要ありません。

See discussion on thread-safety of static constructors。

Answer 2

ロックを行う場合は、double-checked lockingを使用して、キャッシュから読み取るたびにロックを取得するオーバーヘッドが発生しないようにします。

私はまったくロックする必要も疑問に思います。ロックしないと、複数のスレッドが同時にキャッシュをリフレッシュしようとすることがありますが、これはまれであり、非常に高額でない限り、静的な読み取り専用データに問題は発生しません。

それが高価である場合、あなたは、@ asawyerさんのコメントでアドバイスに従うことができ、ロックがLazy<T>によって処理されるように、キャッシュにLazy<ReadOnlyCollection<Location>>を挿入します。