ตัวสร้างตัวเลขสุ่มจะสร้างตัวเลขสุ่มเพียงหนึ่งตัวเท่านั้น

ฉันมีฟังก์ชั่นต่อไปนี้:

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

ฉันจะเรียกมันว่า:

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

ถ้าฉันทำขั้นตอนนั้นวนซ้ำกับดีบักเกอร์ระหว่างรันไทม์ฉันจะได้รับค่าต่าง ๆ (ซึ่งเป็นสิ่งที่ฉันต้องการ) อย่างไรก็ตามถ้าฉันใส่จุดพักสองบรรทัดด้านล่างรหัสนั้นสมาชิกทุกคนในmacอาร์เรย์มีค่าเท่ากัน

ทำไมถึงเกิดขึ้น?

ทุกครั้งที่คุณทำnew Random()มันจะเริ่มต้นได้โดยใช้นาฬิกา ซึ่งหมายความว่าในวงแคบคุณจะได้รับค่าเท่ากันหลายครั้ง คุณควรเก็บอินสแตนซ์แบบสุ่มเดียวและใช้Next ต่อไปบนอินสแตนซ์เดียวกัน

//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);
    }
}

แก้ไข (ดูความคิดเห็น): ทำไมเราต้องมีlockที่นี่?

โดยทั่วไปNextกำลังจะเปลี่ยนสถานะภายในของRandomอินสแตนซ์ หากเราทำในเวลาเดียวกันจากหลายเธรดคุณสามารถโต้แย้งว่า "เราเพิ่งทำผลลัพธ์แบบสุ่มมากขึ้น" แต่สิ่งที่เราทำจริง ๆคือการทำลายการใช้งานภายในและเราอาจเริ่มได้รับตัวเลขเดียวกัน จากหัวข้อที่แตกต่างกันซึ่งอาจเป็นปัญหา - และอาจไม่ แม้ว่าการรับประกันว่าสิ่งที่เกิดขึ้นภายในจะเป็นปัญหาที่ใหญ่กว่าก็ตาม เนื่องจากRandomไม่ได้รับประกันความปลอดภัยของเธรด ดังนั้นจึงมีวิธีการที่ใช้ได้สองวิธี:

• ซิงโครไนซ์เพื่อให้เราไม่สามารถเข้าถึงได้ในเวลาเดียวกันจากกระทู้ที่แตกต่างกัน
• ใช้Randomอินสแตนซ์ต่าง ๆต่อเธรด

ทั้งสามารถปรับ; แต่การปิดการใช้งานอินสแตนซ์เดียวจากผู้โทรหลายคนพร้อมกันก็แค่ถามหาปัญหา

lockประสบความสำเร็จในครั้งแรก (และง่าย) ของวิธีการเหล่านี้ อย่างไรก็ตามวิธีการอื่นอาจเป็น:

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

นี่คือต่อเธรดดังนั้นคุณไม่จำเป็นต้องซิงโครไนซ์

เพื่อความสะดวกในการนำกลับมาใช้ใหม่ตลอดทั้งแอปพลิเคชันของคุณคลาสแบบสแตติกอาจช่วยได้

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; } }
}

คุณสามารถใช้แล้วใช้อินสแตนซ์สุ่มแบบคงที่กับรหัสเช่น

StaticRandom.Instance.Next(1, 100);

โซลูชันของ Mark อาจมีราคาแพงเนื่องจากต้องซิงโครไนซ์ทุกครั้ง

เราสามารถหลีกเลี่ยงการซิงโครไนซ์โดยใช้รูปแบบการจัดเก็บเฉพาะเธรด:

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);
    }
}

วัดการใช้งานทั้งสองและคุณควรเห็นความแตกต่างที่สำคัญ

คำตอบของฉันจากที่นี่ :

เพียงแค่ทำซ้ำคำตอบที่ถูกต้อง :

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

ดังนั้นคุณสามารถโทร:

var i = Util.GetRandom();

ทั้งหมดตลอด

หากคุณอย่างเคร่งครัดต้องมีวิธีการไร้สัญชาติคงจริงGuidเพื่อสร้างตัวเลขสุ่มคุณสามารถพึ่งพา

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

มันจะช้ากว่าเดิมเล็กน้อย แต่สามารถสุ่มได้มากกว่าRandom.Nextอย่างน้อยจากประสบการณ์ของฉัน

แต่ไม่ใช่ :

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

การสร้างวัตถุที่ไม่จำเป็นจะทำให้ช้าลงโดยเฉพาะอย่างยิ่งในลูป

และไม่เคย :

new Random().Next();

ไม่เพียง แต่จะช้ากว่า (ภายในวง) การสุ่มของมันคือ ... ไม่ดีตามฉันเลย ..

ฉันอยากจะใช้คลาสต่อไปนี้เพื่อสร้างตัวเลขสุ่ม:

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

1) ตามที่ Marc Gravell กล่าวว่าพยายามใช้เครื่องกำเนิดไฟฟ้าแบบสุ่มหนึ่งตัว มันยอดเยี่ยมเสมอที่จะเพิ่มสิ่งนี้ลงในตัวสร้าง: System.Environment.TickCount

2) หนึ่งเคล็ดลับ สมมติว่าคุณต้องการสร้างวัตถุ 100 ชิ้นและสมมติว่าแต่ละวัตถุควรมีเครื่องกำเนิดไฟฟ้าแบบสุ่มของตัวเอง (มีประโยชน์ถ้าคุณคำนวณจำนวนตัวเลขสุ่มในช่วงเวลาสั้น ๆ ) หากคุณจะทำเช่นนี้ในการวน (รุ่น 100 วัตถุ) คุณสามารถทำเช่นนี้ (เพื่อให้มั่นใจว่าการสุ่มเต็ม):

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);

ไชโย

ทุกครั้งที่คุณดำเนินการ

Random random = new Random (15);

ไม่สำคัญว่าคุณจะดำเนินการหลายล้านครั้งคุณจะใช้เมล็ดพันธุ์เดิมเสมอ

ถ้าคุณใช้

Random random = new Random ();

คุณจะได้รับหมายเลขสุ่มที่แตกต่างกันหากแฮ็กเกอร์เดาเมล็ดและอัลกอริทึมของคุณเกี่ยวข้องกับความปลอดภัยของระบบของคุณ - อัลกอริทึมของคุณเสีย ฉันคุณเรียกใช้ mult ในตัวสร้างนี้เมล็ดจะถูกระบุโดยนาฬิกาของระบบและหากมีหลายอินสแตนซ์ถูกสร้างขึ้นในช่วงเวลาสั้น ๆ (มิลลิวินาที) เป็นไปได้ว่าพวกมันอาจมีเมล็ดเดียวกัน

หากคุณต้องการตัวเลขสุ่มที่ปลอดภัยคุณต้องใช้ชั้นเรียน

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);
}

การใช้งาน:

int randomNumber = Next(1,100);

เพียงประกาศตัวแปรคลาส Random เช่นนี้

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

หากคุณต้องการรับหมายเลขสุ่มที่แตกต่างกันในแต่ละครั้งจากรายการของคุณให้ใช้

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

แต่ละครั้งโดยประกาศRandom r = new Random()เพียงครั้งเดียว

มีวิธีแก้ปัญหามากมายที่นี่ข้อหนึ่ง: ถ้าคุณต้องการเพียงแค่จำนวนลบตัวอักษรและวิธีการได้รับการสุ่มและความยาวผลลัพธ์

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);
}