[c] Problema numeros aleatorios mayores de 1000

[dmacias]

Hola a todos de nuevo, os vengo con otro problemilla en mis estudios 

El caso es que estoy haciendo este ejercicio



Y he hecho tal como he creido que iria bien, me funcionan todos los resultados, excepto el rango de 1000 <= n <= 1112 y no se porque pasa eso, supongo que será alguna "trampa" del lenguaje, porque he observado que falla en cuando el rango llega a 1000, antes va perfecto asta con los negativos.

Os pego el codigo

Código (c) [Seleccionar] Expandir


/*Programa del libro, Tema 5
* ejercicio 5.13, pagina 170
* problema funcion de numeros aleatorios*/

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int aleatorio(int x, int y);

int main()
{
     int x, y;
     
     printf("Introduce dos numeros para hacer un rango (1 15 p.ej.): ");
     scanf("%d%d", &x, &y);
     
     printf("%d <= %d <= %d\n", x, aleatorio(x, y), y);
     
     return 0;
     
     }
     
int aleatorio(int x, int y){
     srand(time(NULL));
     int z;
     z = rand()%y + x;
     return z;
     
     }




Y unas capturas de que funcionar funciona, excepto con el rango que pasa de 1000.







Pues eso, que si alguien me da norte por donde esta el problema, se lo agradezco mucho 

Un saludo

[amchacon]

A la Y le tienes que restar lo que vale X.

El problema esta en que al hacer %1112, te sale un número del 0 al 1111. Si a eso le sumas 1000 la lias parda completamente.

[noele1995]

El código esta mal para todos pero empezando desde el 1 o el 0 no lo notas

Código (cpp) [Seleccionar] Expandir

int aleatorio(int intervalo_abajo, int intervalo_arriba)
{
    int num;
    srand(time(NULL));
    //Genera un numero de la diferencia del intervalo (+1 por <=) y se lo suma a la base
    num = rand() % (intervalo_arriba - intervalo_abajo + 1) + intervalo_abajo;
   
    return num;
}

Saludos

[dmacias]

Ok, parece que no habia terminado de entender bien como funciona rand, pense que y era el limite y +x el inicio.

Ya esta corregido

Código (c) [Seleccionar] Expandir


/*Programa del libro, Tema 5
* ejercicio 5.13, pagina 170
* problema funcion de numeros aleatorios*/

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int aleatorio(int x, int y);

int main()
{
    int x, y;
   
    printf("Introduce dos numeros para hacer un rango (1 15 p.ej.): ");
    scanf("%d%d", &x, &y);
   
    printf("%d <= %d <= %d\n", x, aleatorio(x, y), y);
   
    return 0;
   
    }
   
int aleatorio(int x, int y){
    srand(time(NULL));
    int z;
    z = rand()%(y -x +1) + x;
    return z;
   
    }



Ahora si funciona perfecto, pero no entiendo como antes funcionaba bien si no llegabamos a 1000  

Un saludo

[noele1995]

Ahora si funciona perfecto, pero no entiendo como antes funcionaba bien si no llegabamos a 1000  

Generabas un numero del intervalo superior y luego le sumabas la base, si la base es 1 o 0 puede que no lo notes, pero ya si es 1000 entonces te pasas tres pueblos xD

Saludos

[amchacon]

Por cierto el srand SOBRA.

La semilla debe inicializarse una sola vez en todo el programa. Si la inicializas cada vez que llamas a la funcion, obtendras siempre valores muy similares (o identicos). Esto se puede ver poniendo un bucle para que te saque 10 aleatorios.

En cuanto al problema, no era de rand sino de como usas los operadores de C. Antes te iba bien porque tus valores inferiores eran muy pequeños, asi que era mas dificil que te saliera un caso erroneo.

[dmacias]

Eso de la semilla no lo entendi del todo
He probado el programa en el rango de 1 a 100 varias veces, rapido para que ande mas o menos por el mismo tiempo (que solo varien segundos al captar la semilla 01:01:xx por poner un ejemplo) y no salen ni parecidos los resultados, quizas me estoy haciendo un lio xD

[amchacon]

Código (cpp) [Seleccionar] Expandir

/*Programa del libro, Tema 5
* ejercicio 5.13, pagina 170
* problema funcion de numeros aleatorios*/

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int aleatorio(int x, int y);

int main()
{
    int x, y;
    int i;
    printf("Introduce dos numeros para hacer un rango (1 15 p.ej.): ");
    scanf("%d%d", &x, &y);

    for (i = 0; i < 100; i++)
        printf("%d <= %d <= %d\n", x, aleatorio(x, y), y);

    return 0;

}

int aleatorio(int x, int y)
{
    srand(time(NULL));
    int z;
    z = rand()%(y -x +1) + x;
    return z;

}


Solución: Poner el srand al principio del codigo:

Código (cpp) [Seleccionar] Expandir

[code=cpp]/*Programa del libro, Tema 5
* ejercicio 5.13, pagina 170
* problema funcion de numeros aleatorios*/

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int aleatorio(int x, int y);

int main()
{
    srand(time(NULL));
    int x, y;
    int i;
    printf("Introduce dos numeros para hacer un rango (1 15 p.ej.): ");
    scanf("%d%d", &x, &y);

    for (i = 0; i < 100; i++)
        printf("%d <= %d <= %d\n", x, aleatorio(x, y), y);

    return 0;

}

int aleatorio(int x, int y)
{
    int z;
    z = rand()%(y -x +1) + x;
    return z;
}



     [/code]

[dmacias]

Despejada la duda con la demostracion