Me crashea el programa

[jaxoR]

Tengo el libro de como programar en C/C++ de deitel. Pero en el libro nunca especifica de terminar el rand.

PD: Seguí adelante con el programa, pero me sigue crasheando, en cuanto ingreso 1 me crashea:

Código (cpp) [Seleccionar] Expandir

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

int main()
{
    int a, i, j, k, f;
    int b[] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9};
    char c[] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', '\0'};
    printf("==========================\n== Generador de codigos ==\n==========================\n\n* Ingrese 1 si quiere generar un codigo\n");
    scanf("%i", &a);

    char letra[5];
    int numero[5];
    i = 0;

    if (a != 1)
    {
        printf("Necesitas ingresar el numero 1 para generar un codigo\n");
    }

    if (a == 1)
    {
        for(j=0; j < 5; j++)
        {
            i++;
            k = 0;
            k = 1 + rand() % 23;
            letra[i] = c[k];
        }

        i = 0;
        j = 0;
        f = 0;

        for(j=0; j < 5; j++)
        {
            i++;
            f = 0;
            f = 1 + rand() % 9;
            numero[i] = b[f];
        }


        printf("%s %s %s %s %s %i %i %i %i %i", letra[0], letra[1], letra[2], letra[3], letra[4], numero[0], numero[1], numero[2], numero[3], numero[4]);

    }

    return 0;
}

[eferion]

si haces

Código (c) [Seleccionar] Expandir

printf("%s", letra[0] );

estás imprimiendo una cadena... una cadena es una secuencia de caracteres que acaba en el caracter '\0'. Dado que en tu caso, letra tiene una secuencia de caracteres que no termina en '\0', el programa seguirá imprimiendo el contenido de la memoria adyacente a letra y eso hace que el programa se muera.

Si lo que quieres es imprimir un caracter tienes que cambiar el printf por

Código (c) [Seleccionar] Expandir

printf( "%c", letra[0] );

[leosansan]

Yo le aconsejaría revisarlo todo ya que hay variables de más, falta la librería time.h, no inicializa rand, haría falta bajo mi punto de vista un bucle por si no introduce 1.

En fin, te lo dejo funcionando. Ya es cosa tuya mejorarlo:

Código (cpp) [Seleccionar] Expandir


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

int main(){
   srand((unsigned) time(NULL));
   int a, j, k=0, f=0;
   int b[] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9};
   char c[] = {"abcdefghijklmnopqrstuvwxys"};
   char letra[5];
   int numero[5];
   do{
     printf("==========================\n== Generador de codigos ==\n==========================\n\n* Ingrese 1 si quiere generar un codigo\n");
     scanf("%d", &a);
     if (a != 1)
       printf("Necesitas ingresar el numero 1 para generar un codigo\n");
   }while(a != 1);
   for(j=0; j < 5; j++) {
     k = 1 + rand() % 25;
     letra[j] = c[k];
   }
     for(j=0; j < 5; j++){
     f = 1 + rand() % 9;
     numero[j] = b[f];
   }
   printf("%c %c %c %c %c %d %d %d %d %d", letra[0], letra[1], letra[2], letra[3], letra[4], numero[0], numero[1], numero[2], numero[3], numero[4]);
   return 0;
}

¡¡¡¡ Saluditos! ..... !!!!

[jaxoR]

Dos preguntas:

Para que sirve el include time.h y que hace esto?

Código (cpp) [Seleccionar] Expandir

srand((unsigned) time(NULL));

[eferion]

Deberías conseguir y leer con tranquilidad un manual de C. Te dará muchas más satisfacciones que darte de cabezazos con estas cosas. Es un consejo.

los includes sirven para enlazar con código externo.

Código (c) [Seleccionar] Expandir

#include <time.h>

se sustituye por el contenido del archivo "time.h". De esta forma el compilador es capaz de encontrar las funciones definidas dentro de este archivo ( en este caso la función time().

Para saber qué hace la línea

Código (c) [Seleccionar] Expandir

srand((unsigned)time(NULL));

Deberías descomponerla en instrucciones simples:

Código (c) [Seleccionar] Expandir


// Esta linea te devuelve la fecha y la hora actuales.
// Realmente almacena el numero de segundos desde el 01/01/1970 a las 00:00
time_t tiempo = time(NULL);

// Esta linea convierte el valor anterior en un tipo basico de C, un entero sin signo.
unsigned int valor = (unsigned)tiempo;

// Esta linea inicializa el generador de numeros aleatorios.
// Pasarle el tiempo es una forma de que la semilla inicial sea mas o menos aleatoria.
srand(valor);

En cualquier caso, en serio, consigue un buen manual de C y empieza a leerlo sin prisa. Adquirir una buena base es más importante que aprender a tirar líneas de código.

[rir3760]

Dos preguntas:

Para que sirve el include time.h y que hace esto?

Código (cpp) [Seleccionar] Expandir

srand((unsigned) time(NULL));

Tienes que leer con mas cuidado el tema, eso lo respondí en mi primer mensaje:

Aparte de todo lo dicho falta inicializar la semilla para los números aleatorios, esto puede hacerse con:

Código (c) [Seleccionar] Expandir

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

/* ... */

int main(void)
{
   /* ... */
   
   srand((unsigned) time(NULL));
   
   /* Uso de rand ...  */
}

Antes de utilizar rand y srand se debe incluir <stdlib.h>, antes de time <time.h>.


Un saludo

[leosansan]

Una pequeña observación: no guardas el código entero en ninguna parte.

Esto lo puedes solventar usando el mismo array letra con las letras y los números.

Otra cosita: el código puede considerarse débil, en el sentido de que van letras seguidas de números. Podrías hacerlo más fuerte entremezclando los números con las letras como te indico a continuación:

Código (cpp) [Seleccionar] Expandir


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

int main(){
    srand((unsigned) time(NULL));
    int a, j, k=0, f=0,cont=0;
    char c[] = {"abcdefghijklmnopqrstuvwxys0123456789"};
    char letra[11],codigo[11];
    for(j=0; j < 11; j++)
      codigo[j]='A';
    do{
      printf("==========================\n== Generador de codigos ==\n==========================\n\n* Ingrese 1 si quiere generar un codigo\n");
      scanf("%d", &a);
      if (a != 1)
        printf("Necesitas ingresar el numero 1 para generar un codigo\n");
    }while(a != 1);
    for(j=0; j < 5; j++) {
      k = rand() % 26;
      letra[j] = c[k];
    }
    for(j=5; j < 10; j++){
      f = 26 + rand() % 10;
      letra[j] = c[f];
    }
    letra[j]='\0';
    printf("%s\n", letra);
    ///printf("%d   %s\n", j,letra);
    for (j=0;j<10;j++){
    j=0;
    do{
      k=rand ()%10;
      if (codigo[j]=='A' && letra[k]!='A'){
        codigo[j]=letra[k];
        letra[k]='A';
        j++;
      }
      }while (j<10);
      codigo[10]='\0';
    }
    printf("%s\n", codigo);
    return 0;
}

También puedes hacerlo aún más fuerte usando más caracteres en el array letra, como /*/#@, etc.

¡¡¡¡ Saluditos! ..... !!!!