コードで競合状態が発生しますか？

Question

すべてのリソースへの基本パスを持つファイルがあります。たとえば：もちろん

build/scripts/script1.js 
build/scripts/script2.js 

私は、例えば、ベースパスを必要とする：

https://example.org/SuperDuperSite/build/scripts/script1.js 

私が何を期待していたどのようなパスで、起動時にグローバル辞書にファイルをロードしました。辞書は一度だけ読み込まれる必要があります。残念ながら、私が知る限り、基本パスは最初のリクエストまで利用できません。だからasp.net私はapplication_startではなくapplication_beginrequestを使用する必要があります。これについて悪い点は、マルチスレッドの問題に取り組まなければならないことです。私は本当に一度にそれをロードする必要がある場合には、すべて単一の要求のために呼び出されます

lock(_lock) { 
    if (_dictionary == null) { 
     LoadDictionary(); 
    } 
    } 

：次のコードの種類を書くために私を強制的に

。私は本当にこれが好きではありません。私は、パフォーマンス上の理由からすべてのリクエストをロックする必要はありません。一つの解決策は、私たちが思いついたの大学に話をした後、この溶液を用いだから、

if (_dictionary == null) 
{ 
    lock(_lock) { 
     if (_dictionary == null) { 
     LoadDictionary(); 
     } 
    } 
} 

私はすべて単一の要求をロックオンするために必要とされないだろうが、複数のスレッドが起動時に、このセクションを取得してしまった場合、私はそれから保護しますそのオブジェクトがロック内で再びヌルであるかどうかをチェックします。このコードは動作するのでしょうか、競合状態に陥るでしょうか？

Answer 1

二重チェックロックを使用する場合は注意してください。

はい、それはスレッドセーフですが、あなたの特定のコードにあなたは_dictionaryがは、他のスレッド（nullチェックを渡す）ことでをインスタンス化されているが、完全にはまだを取り込まない微妙なバグに遭遇するかもしれません部分的に人口が多い辞書にアクセスしようとする可能性があります。そして、これら二つのうちの一つで終わる：

1. をあなたは（もそのパフォーマンスへの影響が付属しています）ConcurrentDictionaryを使用している場合を除き、あなたがかもしれ辞書を読み込むときに、結果を欠落している、または
2. 悪化しています辞書に同時に読み書きしたり、デッドロックを引き起こしたりすることがあります（たとえば、Dictionaryクラスは多くのコードでデッドロックを引き起こすことが知られています）。

（ダブルチェック）bool _dictionaryLoadedフラグは、おそらく優れている、LoadDictionary()の終わりにtrueに裏返します。

.NET 4を使用している場合は、Lazy<>を使用してください。それはもっときれいです。LoadDictionaryをinit関数として使用するだけです。

編集：Lazy <>はinternally implemented with a double-checked lockです。