ThreadLocalを並列ループの間に置くことは良い/悪い考えですか？

Question

私は、並列ランダムジェネレータにThreadLocalを使用しようとしています。私のテストコードは、次のようなものです：

class Program 
{ 
    static void Main() 
    { 
     using (MyClass myClass = new MyClass()) 
     { 
      for (int i = 0; i < 3; i++) 
      { 
       Console.WriteLine("\nPass {0}: ", i + 1); 
       myClass.Execute(); 
      } 

      Console.WriteLine("\nRandom Generators used: {0}", myClass.ListRNG.Count); 
     } 

     Console.WriteLine("\nPress any key..."); 
     Console.ReadKey(); 
    } 
} 

sealed class MyClass : IDisposable 
{ 
    ThreadLocal<RandomGenerator> threadRNG = new ThreadLocal<RandomGenerator>(() => 
                new RandomGenerator(), true); 

    public IList<RandomGenerator> ListRNG { get { return threadRNG.Values; } } 

    public void Execute() 
    { 
     Action<int> action = (i) => 
     { 
      bool repeat = threadRNG.IsValueCreated; 
      List<int> ints = new List<int>(); 
      int length = threadRNG.Value.Next(10); 
      for (int j = 0; j < length; j++) 
       ints.Add(threadRNG.Value.NextInt(100)); 
      Console.WriteLine("Action {0}. ThreadId {1}{2}. Randoms({3}): {4}", 
       i + 1, Thread.CurrentThread.ManagedThreadId, 
       repeat ? " (repeat)" : "", length, string.Join(", ", ints)); 
     }; 

     Parallel.For(0, 10, action); 
    } 

    public void Dispose() 
    { 
     threadRNG.Dispose(); 
    } 
} 

class RandomGenerator : Random 
{ 
    public RandomGenerator() : base(Guid.NewGuid().GetHashCode()) 
    { 
    } 

    public int NextInt(int maxValue) 
    { 
     return base.NextDouble() >= .5 ? base.Next(maxValue) : -base.Next(maxValue); 
    } 
} 

並列ループの複数の実行の間にThreadLocalを維持するのは良いことか悪いのですか？私は、RandomGeneratorの未使用のインスタンスが蓄積するかもしれないと思います。特に私は何千人もの執行をしています。

更新： 私は別のThreadLocalコンストラクタで別のバージョンのテストを試み、すべての値にアクセスできるようにしました（私は上記のコードを変更しました）。

また、myClass.Execute（）を100,000回実行したところ、RandomGeneratorインスタンスが17個しか作成されていませんでした。だから、私は（私が望む）そのようなアプローチですべてがOKだと思う。

Answer 1

スレッド単位の値は、すべての値を指定する必要がない場合は、スレッドをルーツにしています。したがって、スレッドが死んでも、値を保持することはできません。あなたのコードは問題ありません。

[ThreadStatic] static RandomGenerator rng;を使用するとさらに簡単になります。

スレッドローカルアクセスは少し遅いことに注意してください。 RNGを可能な限り小さくして、しばらくの間（各スレッドが構造的に可能な限り）RNGを引っ張るようにすれば、より良いかもしれません。

また、.NET RNGは実際には低速で低品質です。 XorShiftとバリアントの使用を検討してください。