Question

这是一篇很好的文章，描述了随机数的线程安全性：Getting random numbers in a thread-safe way

但我坚持使用“RandomGen2”示例：

public static class RandomGen2 
{ 
    private static Random _global = new Random(); 
    [ThreadStatic] 
    private static Random _local;

    public static int Next() 
    { 
       Random inst = _local; 
       if (inst == null) 
       { 
           int seed; 
           lock (_global) seed = _global.Next(); 
           _local = inst = new Random(seed); 
       } 
       return inst.Next(); 
   }

}

为什么线程静态字段被复制到局部变量：Random inst = _local; ？为什么不简单地使用

if (_local == null) .... return _local.Next()

Answer 1

注意：自从写完这个答案以来，我已经意识到创建多个Random实例的问题，即使它听起来应该有效。我通常发现一个更好的选择是拥有一个Random实例，然后锁定它。虽然这只是一个潜在的瓶颈，但在大多数应用程序中，它不会导致问题。

我怀疑这只是为了避免多次读取线程静态变量的成本。与读取局部变量相比，这样做相对效率低。

你的建议会起作用，但效率会略低一些。在其他情况下，读取之间的值可能会发生变化 - 当然，在这种情况下，这不是问题，因为它是线程局部变量。

使用.NET 4及更高版本，使用ThreadLocal<T>：

会更简单

public static class RandomGen2 
{ 
    private static Random _global = new Random(); 
    private static ThreadLocal<Random> _local = new ThreadLocal<Random>(() =>
    {
        int seed;
        lock (_global) seed = _global.Next();
        return new Random(seed);
    });


    public static int Next() 
    { 
        return _local.Value.Next();
    } 
}

以线程安全的方式获取随机数

1 个答案: