乱数を生成する静的メソッドを使用することをお勧めしますか？

Question

ランダムfloatとdoubleの番号を生成するために、次のコードを書いています。

public static class Statics 
{ 
    private static readonly Random random = new Random(); 
    private static readonly object syncLock = new object(); 

    public static double getRandomDouble(double min, double max) 
    { 
     lock (syncLock) 
     { 
      return random.NextDouble() * (max - min) + min; 
     } 
    } 

    public static float getRandomFloat(float min, float max) 
    { 
     lock (syncLock) 
     { 
      return (float)random.NextDouble() * (max - min) + min; 
     } 
    } 
} 

それは私がstaticclassと乱数を生成するための方法を使用していることをOKですか？

私のプログラムは、これらのメソッドに大きく依存しているので、生成された数値が本当にランダムであることを確認したいと思います。これらのジェネレータは多くのオブジェクトによって使用されていますが、同時には使用されていません。

Answer 1

Randomはスレッドセーフではないことがあり、静的コンテキストでRandomを使用することが問題になる理由は、ありませんしかし、心に留めておく（そして、それはあなたがやったようです）。

lockを使用してThreadを同期することを選択しました。

ただし、ThreadStaticまたはThreadLocalを使用してランダムジェネレータを作成する（better for that causeでどのようにあなたが別の種でなまけインスタンスを初期化することができます）、その方法は、あなたがロックする必要されていないのパフォーマンスを獲得することができます。彼らはあなたがthis答えに見ることができると同時に初期化される可能性があるため

または、各Threadに異なる種でRandomを初期化することを確認してください。

public static class StaticRandom 
{ 
    static int seed = Environment.TickCount; 

    static readonly ThreadLocal<Random> random = 
     new ThreadLocal<Random>(() => new Random(Interlocked.Increment(ref seed))); 

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