Foro de elhacker.net

Programación => Programación C/C++ => Mensaje iniciado por: pacosn1111 en 20 Mayo 2015, 22:04 pm



Título: Error en programa generar clave aleatoria
Publicado por: pacosn1111 en 20 Mayo 2015, 22:04 pm
Hola a tod@s, acabo de escribir un simple código en C que te pregunta con un scanf de cuantos caracteres quieres la clave y te genera una clave aleatoria con mayusculas minusculas y números aleatorios, cuando uso un numero de caracteres alto como 25 todo va bien, pero cuando uso uno pequeño, como 5, obtengo lo siguiente:

x81Zw@

Es decir, esa arroba de más no se por que me aparece, dejo aquí el código:

Código
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <time.h>
  4.  
  5. void main() {
  6.  
  7. srand(time(NULL));
  8.  
  9. int cifras, x;
  10. char min[]="abcdefghijklmnopqrstuvwxyz";
  11. char may[]="ABCDEFGHIJKLMNOPQRSTUVWXYZ";
  12. char num[]="0123456789";
  13.  
  14. printf("Escribe el número de cifras del string a crear...");
  15. scanf("%d", &cifras);
  16. printf("\n");
  17.  
  18. char aleatorio[cifras];
  19.  
  20. for(x; x<cifras; x++) {
  21. int eleccion=(int)(rand() % 3+1);
  22.  
  23. switch ( eleccion ) {
  24. case 1:
  25. aleatorio[x]=min[rand() % 25+1];
  26. break;
  27. case 2:
  28. aleatorio[x]=may[rand() % 25+1];
  29. break;
  30. case 3:
  31. aleatorio[x]=num[rand() % 9+1];
  32. break;
  33. default:
  34. continue;
  35. }
  36.  
  37. aleatorio[cifras+1]='\0';
  38. }
  39. printf("%s\n", aleatorio);
  40. }
  41.  

Saludos.


Título: Re: Error en programa generar clave aleatoria
Publicado por: DarK_FirefoX en 20 Mayo 2015, 22:38 pm
No estas inicializando la variable x del ciclo, y te debe estar obteniendo un valor que esté almacenado en ese espacio en memoria. (el que esté)

Espero te ayude.


Título: Re: Error en programa generar clave aleatoria
Publicado por: pacosn1111 en 20 Mayo 2015, 22:48 pm
Gracias por tu respuesta, pero ya lo he corregido y sigue dando el mismo fallo.


Título: Re: Error en programa generar clave aleatoria
Publicado por: engel lex en 20 Mayo 2015, 22:56 pm
el codigo tiene una lista de errores...

1-
Código
  1. char aleatorio[cifras+1];

debe ser 1 más de largo para que puedas colocarle el caracter de terminación

2-
aquí es raro que no te diera error... estás accediendo a una posición invalida
Código
  1. aleatorio[cifras+1]='\0';


debe ser
Código
  1. aleatorio[cifras]='\0';

ya que el ciclo es x menor que cifras (nunca llega a cifras)

3-
lo tipico  con la linea 21 y el switch es que
Código
  1. int eleccion=(int)(rand() % 3)

y que el switch empieze en 0... sin embargo el metodo funciona, ya que se evalua primero el "%" y luego "+"

pero en 25, 28 y 31 tienes un error...
los array son de 26 y 10 de largo..

cuando haces
Código
  1. aleatorio[x]=num[rand() % 9+1];

se toma un numero entre 0 y 8, luego le sumas 1... así que "a", "A" y "0" nunca se darán



Título: Re: Error en programa generar clave aleatoria
Publicado por: pacosn1111 en 20 Mayo 2015, 23:10 pm
el codigo tiene una lista de errores...

1-
Código
  1. char aleatorio[cifras+1];

debe ser 1 más de largo para que puedas colocarle el caracter de terminación

2-
aquí es raro que no te diera error... estás accediendo a una posición invalida
Código
  1. aleatorio[cifras+1]='\0';


debe ser
Código
  1. aleatorio[cifras]='\0';

Gracias, solucionado.

ya que el ciclo es x menor que cifras (nunca llega a cifras)

3-
lo tipico  con la linea 21 y el switch es que
Código
  1. int eleccion=(int)(rand() % 3)

y que el switch empieze en 0... sin embargo el metodo funciona, ya que se evalua primero el "%" y luego "+"

pero en 25, 28 y 31 tienes un error...
los array son de 26 y 10 de largo..

cuando haces
Código
  1. aleatorio[x]=num[rand() % 9+1];

se toma un numero entre 0 y 8, luego le sumas 1... así que "a", "A" y "0" nunca se darán




Título: Re: Error en programa generar clave aleatoria
Publicado por: engel lex en 20 Mayo 2015, 23:45 pm
Quisiste decir algo con la cita?


Título: Re: Error en programa generar clave aleatoria
Publicado por: pacosn1111 en 22 Mayo 2015, 10:05 am
Si, que ya estaba solucionado, pero sin darme cuenta he generado una cita en blanco.

Saludos.