Wie erzeugt man effizient N zufällige Ints von 0 bis num in C#?

Question

Ich konnte eine Funktion schreiben, die "eindeutige & & zufällige" Ganzzahlen innerhalb eines Bereichs generieren kann. Aber das in n . Ich benutze es für 6 zufällige Eingänge an einem Ort und 30 zufällige Eingänge an anderen, also wie können wir es verbessern, wenn es nötig ist, es zu verbessern?

private int[] genRands(int max, int totalRandomIntsRequired) 
    { 
     int[] nums = new int[totalRandomIntsRequired]; 

     Random r = new Random(); 
     for (int i = 0; i < totalRandomIntsRequired; i++) 
     { 
      nums[i] = r.Next(0, max + 1); 

      for (int j = i; j >= 0; j--) 
      { 
       if(nums[i] == nums[j]) 
       { 
        nums[i] = r.Next(0, max + 1); 
       } 
      } 
     } 
     return nums; 
    } 

Answer 1

hier ist ein Ansatz mit einer HashSet die Duplikate nicht zulässt, und als effiziente interne Dublettenprüfung.

public static int[] genRands(int total, int max) 
{ 
    if (max < total) 
    { 
     throw new IndexOutOfRangeException(); 
    } 
    Random _random = new Random(); 
    HashSet<int> Result = new HashSet<int>(); 
    while (Result.Count < total) 
    { 
     Result.Add(_random.Next(0, max)); 
    } 
    return Result.ToArray(); 
} 

Answer 2

Ich bin ein wenig verwirrt, was Sie fragen, ob Sie nur 6 oder n Zufallszahlen erzeugen müssen, warum zweimal Schleife? So oder so können Sie dies ausprobieren

using System; 
using System.Collections.Generic; 
using System.Linq; 

namespace ConsoleApplication4 
{ 
    class Program 
    { 
     static void Main(string[] args) 
     { 
      var rands = genRands(100, 6); 
      foreach (var num in rands.Select((n, i) => new { i= i + 1, n })) 
      { Console.WriteLine(string.Format("[{0}] {1}", num.i, num.n));}; 
      Console.ReadLine(); 
     } 
     private static List<int> genRands(int max, int total) 
     { 
      var nums = new List<int>(); 
      Random r = new Random(); 
      for (int i = 0; i < total; i++) 
       nums.Add(r.Next(0, max)); 
      return nums; 
     } 
    } 
} 

Es sollte eine Ausgabe so etwas wie dieses

[1] 45 
[2] 29 
[3] 40 
[4] 75 
[5] 29 
[6] 57