将ThreadLocal保持在并行循环之间是一个好/坏的想法?

时间:2016-10-30 07:42:49

标签: c# random parallel-processing thread-local

我正在尝试将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构造函数,允许访问所有值(我已经改变了上面的代码)。

我还尝试了10万次myClass.Execute()执行,发现只创建了17个RandomGenerator实例。所以我认为(我希望)这种方法一切都好。

1 个答案:

答案 0 :(得分:1)

如果您不要求列出所有值,则每个线程的值都以线程为根。因此,如果线程死亡,他们就不会保持他们的价值观。你的代码很好。

如果使用[ThreadStatic] static RandomGenerator rng;,则会变得更加简单。

注意,线程本地访问有点慢。如果你让每个线程尽可能少地拉动RNG并保持一段时间(在架构上可能的话),你可能会更好。

此外,.NET RNG非常慢且质量低。考虑使用XorShift和变体。