Nombre aléatoire dans une boucle

Déplacez la déclaration du générateur de nombres aléatoires hors de la boucle.

La génération de nombres aléatoires commence à partir d'une valeur de graine. Si la même graine est utilisée à plusieurs reprises, la même série de nombres est générée. Un moyen de produire différentes séquences est de rendre la valeur de la graine dépendante du temps, produisant ainsi une série différente avec chaque nouvelle instance de hasard. Par défaut, le constructeur sans paramètre de la classe aléatoire utilise l'horloge système pour générer sa valeur de graine, ...

source < p> En faisant la déclaration dans la boucle, vous appelez efficacement le constructeur de la même valeur et encore, vous obtenez les mêmes numéros.

Donc, votre code devrait donc devenir: xxx

Random r = new Random(); 
for ... 
    string += r.Next(4); 
new Random() will initialize the (pseudo-)random number generator with a seed based on the current date and time. Thus, two instances of Random created at the same date and time will yield the same sequence of numbers.You created a new random number generator in each iteration and then took the first value of that sequence. Since the random number generators were the same, the first value of their sequences was the same. My solution will create one random number generator and then return the first, the second, etc... value of the sequence (which will be different).

Vous devez utiliser la même instance aléatoire tout au lieu de la création d'une nouvelle à chaque fois.

Comme vous l'avez: P>

Random r = new Random()
for ...
    string += r.Next(4);

Déplacer le Random R = Nouveau aléatoire (); en dehors de la boucle et appelez simplement à l'intérieur de la boucle.