Números C# azar no están siendo "aleatoria"

Question

sé que la clase Random C# no hace "verdaderos azar" números, pero estoy subiendo con un problema con este código:

public void autoAttack(enemy theEnemy) 
    { 
     //Gets the random number 
     float damage = randomNumber((int)(strength * 1.5), (int)(strength * 2.5)); 

     //Reduces the damage by the enemy's armor 
     damage *= (100/(100 + theEnemy.armor)); 

     //Tells the user how much damage they did 
     Console.WriteLine("You attack the enemy for {0} damage", (int)damage); 

     //Deals the actual damage 
     theEnemy.health -= (int)damage; 

     //Tells the user how much health the enemy has left 
     Console.WriteLine("The enemy has {0} health left", theEnemy.health); 
    } 

I a continuación, llamar a la función aquí (me llamó 5 veces en aras de comprobar si los números eran al azar):

 if (thePlayer.input == "fight") 
     { 
      Console.WriteLine("you want to fight"); 
      thePlayer.autoAttack(enemy1); 
      thePlayer.autoAttack(enemy1); 
      thePlayer.autoAttack(enemy1); 
     } 

Sin embargo, al comprobar la salida, me da exactamente el mismo número por cada 3 llamadas a funciones. Sin embargo, cada vez que ejecute el programa, aparece un número diferente (que se repite 3 veces) así:

You attack the enemy for 30 damage. 
The enemy has 70 health left. 

You attack the enemy for 30 damage. 
The enemy has 40 health left. 

You attack the enemy for 30 damage. 
The enemy has 10 health left. 

haré luego reconstruir/debug/a ejecutar el programa, y ​​obtener un número diferente en lugar de 30 , pero repetirá las 3 veces.

Mi pregunta es: ¿cómo puedo asegurarme de obtener un número aleatorio diferente cada vez que llamo a esta función? Solo estoy obteniendo el mismo número "aleatorio" una y otra vez.

Aquí es la llamada clase al azar que he utilizado:

private int randomNumber(int min, int max) 
    { 
     Random random = new Random(); 
     return random.Next(min, max); 
    } 

Answer 1

Mi conjetura es que randomNumber crea una nueva instancia de Random ... cada vez que a su vez crea un nuevo generador de números pseudo-aleatorios basados ​​en la hora actual ... que no cambia tan a menudo como crees.

No hagas eso. Use la misma instancia de Random repetidamente ... pero no lo haga "soluciónelo" creando una variable estática Random. Eso tampoco funcionará bien a largo plazo, ya que Random no es seguro para subprocesos. Todo se verá bien en las pruebas, y misteriosamente recuperarás todos los ceros cuando tengas mala suerte con la simultaneidad :(

Afortunadamente, no es demasiado difícil conseguir algo que funcione con thread-locals, especialmente si eres en .NET 4. se termina con una nueva instancia de Random por hilo

he escrito un article on this very topic que pueden serle útil, incluyendo este código:.

using System; 
using System.Threading; 

public static class RandomProvider 
{  
    private static int seed = Environment.TickCount; 

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

    public static Random GetThreadRandom() 
    { 
     return randomWrapper.Value; 
    } 
} 

Si cambia su llamada new Random() a RandomProvider.GetThreadRandom() que probablemente hará todo lo que necesites ed (de nuevo, suponiendo .NET 4). Eso no se ocupa de la capacidad de prueba, pero un paso a la vez ...

Answer 2

¿Qué es randomNumber?

Normalmente se siembra un generador de números pseudoaleatorios (con una cosa relacionada con el tiempo, o algo aleatorio como un tiempo entre dos pulsaciones de tecla o paquetes de red o algo así).

No indica qué generador está utilizando ni cómo se siembra.

Answer 3

No nos mostró el código para randomNumber. Si se ve algo como

private int randomNumber(int m, int n) { 
    Random rg = new Random(); 
    int y = rg.Next(); 
    int z = // some calculations using m and n 
    return z; 
} 

Bueno, entonces está su problema.Si sigues creando nuevas instancias de Random, es posible que a veces tengan la misma semilla (la semilla predeterminada es el reloj del sistema que tiene una precisión limitada, créalas lo suficientemente rápido y obtienen la misma semilla) y luego la secuencia producida por este generador siempre será el mismo.

Para solucionar esto, se tiene que crear una instancia de Random vez:

private readonly Random rg = new Random(); 
private int randomNumber(int m, int n) { 
    int y = this.rg.Next(); 
    int z = // some calculations using m and n 
    return z; 
} 

Y para aclarar otro punto, incluso si lo hace, la salida de Random todavía no es "verdadero" al azar. Es solo psuedorandom.

Answer 4

si genera números aleatorios en el bucle, probablemente no sea aleatorio. porque los números aleatorios se crean básicamente internamente en la hora actual del sistema. Así coloque este código en el bucle:

Thread.Sleep(10); 

Así que el sistema se irá a dormir durante 10 m seg. Y obtendrás un nuevo número aleatorio nuevo. Es una solución garantizada. Pero esto también afectará el rendimiento del sistema.

Answer 5

Crea una instancia del objeto al azar fuera del método. (Aleatorio aleatorio = nuevo Aleatorio(); debe escribirse antes del método)

También es vital que entienda que al azar no es really random.