Penghasil bilangan acak hanya menghasilkan satu bilangan acak

Saya memiliki fungsi berikut:

//Function to get random number
public static int RandomNumber(int min, int max)
{
    Random random = new Random();
    return random.Next(min, max);
}

Bagaimana saya menyebutnya:

byte[] mac = new byte[6];
for (int x = 0; x < 6; ++x)
    mac[x] = (byte)(Misc.RandomNumber((int)0xFFFF, (int)0xFFFFFF) % 256);

Jika saya langkah loop itu dengan debugger selama runtime saya mendapatkan nilai yang berbeda (yang saya inginkan). Namun, jika saya meletakkan breakpoint dua baris di bawah kode itu, semua anggota macarray memiliki nilai yang sama.

Mengapa itu terjadi?

Setiap kali Anda melakukannya new Random()diinisialisasi menggunakan jam. Ini berarti bahwa dalam loop ketat Anda mendapatkan nilai yang sama banyak kali. Anda harus menyimpan satu instance Acak dan terus menggunakan Next pada instance yang sama .

//Function to get a random number 
private static readonly Random random = new Random(); 
private static readonly object syncLock = new object(); 
public static int RandomNumber(int min, int max)
{
    lock(syncLock) { // synchronize
        return random.Next(min, max);
    }
}

Sunting (lihat komentar): mengapa kita perlu di locksini?

Pada dasarnya, Nextakan mengubah keadaan internal Randominstance. Jika kami melakukan itu secara bersamaan dari beberapa utas, Anda dapat berdebat "kami baru saja membuat hasilnya lebih acak", tetapi apa yang sebenarnya kami lakukan berpotensi melanggar implementasi internal, dan kami juga bisa mulai mendapatkan angka yang sama dari utas berbeda, yang mungkin menjadi masalah - dan mungkin tidak. Namun, jaminan apa yang terjadi secara internal adalah masalah yang lebih besar; karena Randomtidak tidak membuat jaminan benang-keselamatan. Jadi ada dua pendekatan yang valid:

• Sinkronisasi agar kami tidak mengaksesnya secara bersamaan dari utas yang berbeda
• Gunakan Randomcontoh berbeda per utas

Baik bisa baik-baik saja; tetapi mutexing satu contoh dari beberapa penelepon pada saat yang sama hanya meminta masalah.

The lockmencapai pertama (dan sederhana) dari pendekatan ini; Namun, pendekatan lain mungkin:

private static readonly ThreadLocal<Random> appRandom
     = new ThreadLocal<Random>(() => new Random());

ini kemudian per-utas, jadi Anda tidak perlu melakukan sinkronisasi.

Untuk kemudahan penggunaan ulang di seluruh aplikasi Anda, sebuah kelas statis dapat membantu.

public static class StaticRandom
{
    private static int seed;

    private static ThreadLocal<Random> threadLocal = new ThreadLocal<Random>
        (() => new Random(Interlocked.Increment(ref seed)));

    static StaticRandom()
    {
        seed = Environment.TickCount;
    }

    public static Random Instance { get { return threadLocal.Value; } }
}

Anda dapat menggunakan lalu menggunakan contoh acak statis dengan kode seperti

StaticRandom.Instance.Next(1, 100);

Solusi Mark bisa sangat mahal karena perlu melakukan sinkronisasi setiap kali.

Kita dapat menyiasati kebutuhan untuk sinkronisasi dengan menggunakan pola penyimpanan khusus thread:

public class RandomNumber : IRandomNumber
{
    private static readonly Random Global = new Random();
    [ThreadStatic] private static Random _local;

    public int Next(int max)
    {
        var localBuffer = _local;
        if (localBuffer == null) 
        {
            int seed;
            lock(Global) seed = Global.Next();
            localBuffer = new Random(seed);
            _local = localBuffer;
        }
        return localBuffer.Next(max);
    }
}

Ukur kedua implementasi dan Anda akan melihat perbedaan yang signifikan.

Jawaban saya dari sini :

Hanya mengulangi solusi yang tepat :

namespace mySpace
{
    public static class Util
    {
        private static rnd = new Random();
        public static int GetRandom()
        {
            return rnd.Next();
        }
    }
}

Jadi, Anda dapat menelepon:

var i = Util.GetRandom();

semua di seluruh.

Jika Anda benar-benar membutuhkan metode statis stateless sejati untuk menghasilkan angka acak, Anda bisa mengandalkan a Guid.

public static class Util
{
    public static int GetRandom()
    {
        return Guid.NewGuid().GetHashCode();
    }
}

Ini akan menjadi sedikit lebih lambat, tetapi bisa jauh lebih acak daripada Random.Next, setidaknya dari pengalaman saya.

Tapi tidak :

new Random(Guid.NewGuid().GetHashCode()).Next();

Pembuatan objek yang tidak perlu akan membuatnya lebih lambat terutama di bawah satu lingkaran.

Dan tidak pernah :

new Random().Next();

Tidak hanya lebih lambat (di dalam satu lingkaran), keacakannya adalah ... yah tidak benar-benar bagus menurut saya ..

Saya lebih suka menggunakan kelas berikut untuk menghasilkan angka acak:

byte[] random;
System.Security.Cryptography.RNGCryptoServiceProvider prov = new System.Security.Cryptography.RNGCryptoServiceProvider();
prov.GetBytes(random);

1) Seperti yang dikatakan Marc Gravell, coba gunakan ONE-generator acak. Itu selalu keren untuk menambahkan ini ke konstruktor: System.Environment.TickCount.

2) Satu tip. Katakanlah Anda ingin membuat 100 objek dan anggap masing-masing dari mereka harus memiliki generator acak sendiri (berguna jika Anda menghitung LOADS angka acak dalam waktu yang sangat singkat). Jika Anda melakukan ini dalam satu lingkaran (generasi 100 objek), Anda bisa melakukan ini seperti itu (untuk memastikan keacakan sepenuhnya):

int inMyRandSeed;

for(int i=0;i<100;i++)
{
   inMyRandSeed = System.Environment.TickCount + i;
   .
   .
   .
   myNewObject = new MyNewObject(inMyRandSeed);  
   .
   .
   .
}

// Usage: Random m_rndGen = new Random(inMyRandSeed);

Bersulang.

Setiap kali Anda mengeksekusi

Random random = new Random (15);

Tidak masalah jika Anda mengeksekusinya jutaan kali, Anda akan selalu menggunakan seed yang sama.

Jika Anda menggunakan

Random random = new Random ();

Anda mendapatkan urutan nomor acak yang berbeda, jika seorang hacker menebak benih dan algoritma Anda terkait dengan keamanan sistem Anda - algoritma Anda rusak. Saya Anda menjalankan mult. Dalam konstruktor ini, seed ditentukan oleh jam sistem dan jika beberapa instance dibuat dalam periode waktu yang sangat singkat (milidetik), mungkin saja mereka memiliki seed yang sama.

Jika Anda membutuhkan nomor acak yang aman, Anda harus menggunakan kelas

System.Security.Cryptography.RNGCryptoServiceProvider

public static int Next(int min, int max)
{
    if(min >= max)
    {
        throw new ArgumentException("Min value is greater or equals than Max value.");
    }
    byte[] intBytes = new byte[4];
    using(RNGCryptoServiceProvider rng = new RNGCryptoServiceProvider())
    {
        rng.GetNonZeroBytes(intBytes);
    }
    return  min +  Math.Abs(BitConverter.ToInt32(intBytes, 0)) % (max - min + 1);
}

Pemakaian:

int randomNumber = Next(1,100);

cukup deklarasikan variabel kelas acak seperti ini:

    Random r = new Random();
    // ... Get three random numbers.
    //     Here you'll get numbers from 5 to 9
    Console.WriteLine(r.Next(5, 10));

jika Anda ingin mendapatkan nomor acak yang berbeda setiap kali dari daftar Anda maka gunakan

r.Next(StartPoint,EndPoint) //Here end point will not be included

Setiap kali dengan mendeklarasikan Random r = new Random()sekali.

Ada banyak solusi, di sini satu: jika Anda hanya ingin menghapus nomor surat dan metode menerima secara acak dan panjang hasilnya.

public String GenerateRandom(Random oRandom, int iLongitudPin)
{
    String sCharacters = "123456789ABCDEFGHIJKLMNPQRSTUVWXYZ123456789";
    int iLength = sCharacters.Length;
    char cCharacter;
    int iLongitudNuevaCadena = iLongitudPin; 
    String sRandomResult = "";
    for (int i = 0; i < iLongitudNuevaCadena; i++)
    {
        cCharacter = sCharacters[oRandom.Next(iLength)];
        sRandomResult += cCharacter.ToString();
    }
    return (sRandomResult);
}