Problema con rand()

[MrDev]

Pues tengo un problema, y es que no me da los valores como numeros, me los da como direcciones de memoria supongo que son. Igualmente al poner en el main al principio "in t p=rand();" me daba un 0 todo el rato, aqui dejo el codigo a ver si me podeis echar una manilla.

Código (c) [Seleccionar] Expandir

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

char* crearPassword();

int main(int argc, const char * argv[]) {
    srand (time(NULL));
   
    float p=rand();
   
    char* contrasena=crearPassword();
   
    printf("%s",contrasena);
   
    return 0;
}

char* crearPassword(){
    int i;
    char numero,aleatorio;
    char *password=malloc(8*sizeof(char))+1;
   
    for(i=0;i<4;i++){
        password[i]=(int)rand()%10;
    }
   
    for(i=4;i<8;i++){
        do{
            aleatorio=(char)rand()%123;
        }while(aleatorio<65 && aleatorio>122);
        password[i]=aleatorio;
    }
   
    return password;
}

[class_OpenGL]

Código (cpp) [Seleccionar] Expandir

do{
    aleatorio=(char)rand()%123;
}while(aleatorio<65 && aleatorio>122);

A ver una cosilla. El número aleatorio está entre 0 y 122, por lo que nunca va a ser mayor que 122. Lo que tendrías que hacer es calcular un número aleatorio entre 0 y la diferencia de 122 y 65, y luego sumarle 65 (aleatorio = (char)rand()%(122-65) + 65;).

También tienes que tener en cuenta que tienes que cambiar la semilla del generador de números aleatorios (rand()) con la función srand cada vez que generes un número aleatorio. Te aconsejo que uses srand(clock()).

[MrDev]

Código (cpp) [Seleccionar] Expandir

do{
   aleatorio=(char)rand()%123;
}while(aleatorio<65 && aleatorio>122);

A ver una cosilla. El número aleatorio está entre 0 y 122, por lo que nunca va a ser mayor que 122. Lo que tendrías que hacer es calcular un número aleatorio entre 0 y la diferencia de 122 y 65, y luego sumarle 65 (aleatorio = (char)rand()%(122-65) + 65;).

También tienes que tener en cuenta que tienes que cambiar la semilla del generador de números aleatorios (rand()) con la función srand cada vez que generes un número aleatorio. Te aconsejo que uses srand(clock()).

Gracias por la respuesta compi.

Edito, el problema es que en el primer bucle for de la funcion, no guarda en password los numeros

[MAFUS]

Código (c) [Seleccionar] Expandir

char *password=malloc(8*sizeof(char))+1;
Esto está mal. malloc te asigna una zona de memoria pero devuelves la posición 1 byte superior. Tal vez quisiste decir:

Código (c) [Seleccionar] Expandir

char *password=malloc((8+1)*sizeof(char)); para guardar la cadena más el caracter nulo.
Pero tampoco convence ya que rand podría generarte un 0 cuándo estás consiguiendo los números, con lo que la cadena resultante acabaría en ese momento. Por otra parte no estás escribiendo caracteres numéricos sino caracteres de control ASCII no imprimibles, por lo que a la hora de visualizar la cadena podrían aparecer efectos extraños en pantalla.
Tal vez quisiste decir:

Código (c) [Seleccionar] Expandir


for(i=0;i<4;i++)
   password[i]=(int)rand()%10+'0';;

Más. Si usas malloc para asignar memoria debes usar free para liberarla en cuanto no la uses. La penúltima instrucción de tu programa debería ser

Código (c) [Seleccionar] Expandir

free(contrasena);

[MrDev]

Código (c) [Seleccionar] Expandir

char *password=malloc(8*sizeof(char))+1;
Esto está mal. malloc te asigna una zona de memoria pero devuelves la posición 1 byte superior. Tal vez quisiste decir:

Código (c) [Seleccionar] Expandir

char *password=malloc((8+1)*sizeof(char)); para guardar la cadena más el caracter nulo.
Pero tampoco convence ya que rand podría generarte un 0 cuándo estás consiguiendo los números, con lo que la cadena resultante acabaría en ese momento. Por otra parte no estás escribiendo caracteres numéricos sino caracteres de control ASCII no imprimibles, por lo que a la hora de visualizar la cadena podrían aparecer efectos extraños en pantalla.
Tal vez quisiste decir:

Código (c) [Seleccionar] Expandir


for(i=0;i<4;i++)
   password[i]=(int)rand()%10+'0';;

Más. Si usas malloc para asignar memoria debes usar free para liberarla en cuanto no la uses. La penúltima instrucción de tu programa debería ser

Código (c) [Seleccionar] Expandir

free(contrasena);

Gracias por tu respuesta compi, ya esta el programa listo.

Una duda, en el

Código (c) [Seleccionar] Expandir


for(i=0;i<4;i++)
   password[i]=(int)rand()%10+'0';;

Por que no funciona sin el "'+0'"?

[MAFUS]

El hecho es que tu generas un carácter del ASCII 0 al ASCII 9, que son caracteres no imprimibles. Busca una tabla en internet de código ASCII y verás lo que digo. Pero al sumar '0' al resultado del rand lo que haces es llevar ese resultado al caracter ASCII '0' que ese sí ya es la representación del cero. ASCII dice que los caracteres numéricos van del cero hasta el nueve. Por tanto '0' + 0 es '0', '0' + 1 es '1', '0' + 2 es '2', etc.
Si te fijas lo mismo pasa con las letras: 'a' + 0 es 'a', 'a' + 1 es 'b'... 'A' + 0 es 'A', 'A' + 1 es 'B'...

[MrDev]

El hecho es que tu generas un carácter del ASCII 0 al ASCII 9, que son caracteres no imprimibles. Busca una tabla en internet de código ASCII y verás lo que digo. Pero al sumar '0' al resultado del rand lo que haces es llevar ese resultado al caracter ASCII '0' que ese sí ya es la representación del cero. ASCII dice que los caracteres numéricos van del cero hasta el nueve. Por tanto '0' + 0 es '0', '0' + 1 es '1', '0' + 2 es '2', etc.
Si te fijas lo mismo pasa con las letras: 'a' + 0 es 'a', 'a' + 1 es 'b'... 'A' + 0 es 'A', 'A' + 1 es 'B'...

Gracias, ya lo entiendo, se aprende bastante en foros