静的読取り専用配列のスレッドセーフとforeach

Question

「静的読取り専用」として定義されたchar []とstring []型の配列がいくつかあります。彼らのアイテムは決して変化しません。

'foreach'と 'generics'列挙型はスレッドセーフですか？

private static readonly string[] staticReadOnlyArray = new string[] { "someKey0", "someKey1", "someKey2", ... }; 

public bool SomeThreadSharedCall(string toCheck) 
{ 
    // #1 
    foreach (string s in staticReadOnlyArray) 
    { 
     if (s == toCheck) 
      return true; 
    } 
    return false; 

    // #2 
    return staticReadOnlyArray.Contains(toCheck); 

    // or #3 
    return staticReadOnlyArray.Any(s => string.Compare(toCheck, s, StringComparison.OrdinalIgnoreCase) == 0); 

    // or #4 
    staticReadOnlyArray.ForEach(s => someAction(s, toCheck)); 
} 

Answer 1

あなたのメソッドの操作は、あなたの主張に基づいてのみ「スレッドセーフ」ですTheir items never change.;ただし、指定されたコードの中には何も保証されていません。

用語threadsafeは通常意味するために使用されている問題のコード保証データが変更されないこと、またはそれが変更、操作が正しい結果を生成しますすることができた場合には。

コードを本当にスレッドセーフにするには、your own synchronization logicを実行する必要があります。あなたのコメントへ

===回答：

GetEnumerator()の戻り値 - あなたはiteratorを呼び出しているものは - 呼び出し側スレッドによる排他的使用のために安全そのものであるが、それはここでの問題はありません。それは変更可能なので、スレッドセーフではない基底のコレクション（配列）です。

ループをforeachからforに変更しても、コードはスレッドセーフではありません。コレクションへのアクセスを同期するか、コレクションを不変にする必要があります。

あなたのケースでは、配列のデータが一定であることを前提として、後者をお勧めします。具体的には、具体的なコードをご紹介します。

private static readonly IEnumerable<string> staticReadOnlyData = Array.AsReadOnly(new string[] { "someKey0", "someKey1", "someKey2", ... }); 

public bool SomeThreadSharedCall(string toCheck) 
{ 
    // #1 
    foreach (string s in staticReadOnlyData) 
    { 
     if (s == toCheck) 
      return true; 
    } 
    return false; 

    // #2 
    return staticReadOnlyData.Contains(toCheck); 

    // or #3 
    return staticReadOnlyData.Any(s => string.Compare(toCheck, s, StringComparison.OrdinalIgnoreCase) == 0); 
} 

Answer 2

よくわからない場合は、テストしてください。しかし、はい、スレッドセーフです。