Tengo la siguiente función que genera un número aleatorio entre -10 y 2:
#include <stdlib.h> #include <time.h> static int getRandomReturnCode(void) { int N = 2, M = -10; srand(time(NULL)); r = M + rand() / (RAND_MAX / (N - M + 1) + 1); if (r == 0){ getRandomReturnCode(); } return r; }Actualmente, si se devuelve un código de retorno de 0 (éxito), llamará recursivamente a la función hasta que se cumpla y se devuelva un código de retorno distinto de cero. ¿Qué puedo hacer para mejorar mi código de modo que el 0 quede excluido del rango de números elegidos al azar?
Hagas lo que hagas, no vuelvas a dibujar si un valor devuelto es 0: eso introducirá un sesgo estadístico en el resultado.
Lo mejor que puede hacer aquí es dibujar entre -10 y 1 inclusive y agregar 1 a cualquier salida no negativa.
Finalmente, llame a srand una vez más arruinará las propiedades estadísticas del generador.
(Brevemente, dado que este comentario es más para el sitio de matemáticas, y restringiendo el argumento a un generador congruente lineal, lo que tiende a suceder si omite los valores generados es que aumenta la varianza de la distribución resultante para que ya no sea uniforme. El pequeño número extraído se relacionará linealmente con el dibujo posterior, ya que el módulo del generador no tendrá efecto. Esta autocorrelación, necesaria para que la distribución resultante sea uniforme, de los dibujos adyacentes se reducirá si se descartan los dibujos y eso aumenta la varianza de la muestra.)