静的メソッド間の呼び出し - スレッドの安全性を脅かすでしょうか？

Question

私は次のローカルメソッドを持っているパブリック静的クラスで：このパブリックメソッドその

public static bool IsIntegrityOfDataSetGood(DataSet dataSet, KeyValuePair<string, List<int>> tableAndColumnIndexes) 
    { 
     return IsIntegrityOfDataSetGood(dataSet, tableAndColumnIndexes.Key) && 
       dataSet.Tables[tableAndColumnIndexes.Key].Columns.Count > GetMaxFromList(tableAndColumnIndexes.Value); 
    } 

注：このメソッドは、クラス内のパブリックメソッドの一部（例えば）によって呼び出され

private static int GetMaxFromList(IEnumerable<int> list) 
    { 
     var result = 0; 

     foreach (var i in list) 
     { 
      if (i > result) result = i; 
     } 

     return result; 
    } 

クラス内の別のパブリックメソッドを呼び出します。

クラスメソッドは、パラメータを介して送信されたデータでのみ動作しますが、静的メソッド間の呼び出しによってスレッドの安全性が損なわれることが懸念されます。私は心配する権利はありますか？

このコードは、.NET 2.0プロジェクトに存在します。

Answer 1

いいえ、これはではありません。はあなたのスレッドの安全を守ります。

しかし、本当にスレッドセーフであるかどうかは、tableAndColumnIndexesがどこから来ているのか、ほかのどのスレッドがアクセスできるのかによって異なります。言い換えれば、それは呼び出しコードに依存します。

Answer 2

他のスレッドがlistシーケンスを変更していない限り、それは絶対に良いはずです。それは共有状態にはアクセスせず、パラメータを使用するだけです。他のスレッドがの場合、InvalidOperationExceptionとなります。

スレッドの安全性と静的メソッドについて誤解を感じるかもしれませんが、正確には何かを伝えるのは難しいです...あなたはEric Lippertのブログ投稿"What is this thing you call thread safe?"が役に立ちます。