Estructuras en C

[n00b17]

Buenos días. Tengo un percance con un programa que estoy tratando de hacer.
Se trata de hacer dos estructuras, una que contenga un cliente y su información personal, otra para etiquetado e inventariado de CD, luego, si es necesario, crear una nueva para el alquiler de los CD. Tengo planteado algo, pero el programa se me cierra al ingresar alguna opción a realizar, no detecto el error. Quisiera saber si me pueden ayudar: (trabajo con Codeblocks en windows)

Código:

#include <stdio.h>
#include <string.h>


struct cliente
{
    int cedula;
    char nombre[10];
    char apellido[10];
    char fdn[15];
    char sexo[10];
    int edad;
}clien;

struct cd
{
   int codigocd;
   char autor[10];
   char album[10];
   char titulo[10];
   int npista;
}disco;

struct alquilercd
{
    char cdalquilado[30];
    char fechaalquiler[15];
    int c;
}alqui;

     void insertarcd();
     void insertarcliente();
     void alquilarcd();
     void retornarcd();

void insertarcd(){
                 printf("Codigo de CD\n");
                 scanf("%d", &disco.codigocd);
                 printf("Nombre del Autor\n");

                 scanf("%s", &disco.autor);
                 printf("Titulo del Album\n");
                 scanf("%s", &disco.album);
                 printf("Nombre del Titulo\n");
                 scanf("%s", &disco.titulo);
                 printf("Numero De Pistas\n");
                 scanf("%d", disco.npista);
                 FILE *archivo;
                 archivo=fopen("c:\\disco.txt", "w");
                 if(archivo==NULL)
                return 1;
    fprintf(archivo, "\n");
    fclose(archivo);
                }

void insertarcliente(){
                    printf("Digite la C.C\n");
                    scanf("%d", &clien.cedula);
                    printf("Digite el Nombre\n");
                    scanf("%s", &clien.nombre);
                    printf("Digite el Apellido\n");
                    scanf("%s", &clien.apellido);
                    printf("Digite La Fecha Nacimiento\n");
                    scanf("%s", &clien.fdn);
                    printf("Digite el sexo\n");
                    scanf("%s", &clien.sexo);
                    printf("Digite la Edad\n");
                    scanf("%d", &clien.edad);
                        FILE *archivo;
                 archivo=fopen("c:\\clientes.txt", "w");
                 if(archivo==NULL)
                 return 1;
    fprintf(archivo,"\n",clien.cedula,clien.nombre,clien.apellido,clien.fdn,clien.sexo,clien.edad);
    fclose(archivo);
                        }
void alquilarcd(){
       printf("digite la cc\n");
       scanf("%d", &clien.cedula);
       FILE *fichero;
    fichero = fopen("c:\\clientes.txt", "r");
    while(!feof(fichero));
    rewind(fichero);
    fclose(fichero);
    getchar();
}
void retornarcd(){
}

int main(){
           int a;
           do{
            printf("alquiler de CDs\n");
            printf("Menu\n");
            printf("Digite Opcion\n");
            printf("1.insertar cliente\n");
            printf("2.insertar CD\n");
            printf("3.alquilar CD\n");
            printf("Opcion\n");
            gets(a);

                  switch(a){
                  case 1:
                    insertarcliente();
                    break;
                  case 2:
                      insertarcd();
                      break;
                  case 3:
                      alquilarcd();
                      break;
                            }
           }
           while(a!=10);
}

[CapacProg]

Hola, veo un error, borra dentro del scanf para los campos de tipo cadena el & delante.

Antes con &

Código C:

printf("Titulo del Album\n");
 scanf("%s", &disco.album);
 

Ahora sin &

Código C:

printf("Titulo del Album\n");
 scanf("%s", disco.album);
 

Luego comentas como te fue y sino menciona el error y linea donde falla.

[amchacon]

Estructuras globales? *_*

Un truco para saber donde está el error es ir poniendo getchar() linea a linea. De esa forma podrás identificar que línea es la que destroza el programa.

[n00b17]

Capaprog, me sigue sucediendo lo mismo, ejecuta pero al ingresar una opción, se crashea.
Igual sucede si uso gets simplemente. No sé en sí si el código me servirá para lo que quiero entonces.

amchacon. Trataré de ver si me resulta para identificar el problema, pero no debería ser tan engorroso.

[amchacon]

Iniciado por n00b17

Trataré de ver si me resulta para identificar el problema, pero no debería ser tan engorroso.

Realmente, lo ideal es hacerlo con un depurador. Pero a falta de depurador es un sistema bastante eficaz.

No tienes que ponerlo en todas las lineas, solo antes de las lineas que puedan fallar.

[L3m0n]

Buenas, de entrada tienes un par de warnings que saltan en el compilador, las funciones, si son void no deben devolver nada, deberías borrar el return.


Ahora vamos con el problema, yo te propongo que no uses gets() ya que es una funcion que induce a problemas con el buffer del input. Así que yo de ti crearía algo así, substituyendo el gets por un fgets y un sscanf que es lo que se debe usar para no tener problemas con el input.

Código C:

//dentro del main
int a;
char cad[15];
 
//antes del switch
fgets(cad,sizeof(cad),stdin);
sscanf(cad,"%d",&a);
 
//y ahora el switch
switch(a)
{
...
}
 

Y con eso se debería solucionar el problema



Saludos

[CapacProg]

Hola:

Con tantos errores que vi, me cansaría de subirlos y que luego lo interpretes, así que mejor te paso el código que lo corregido y si tienes dudas las comentas.

Código C:

//.
//.
//.
 
void insertarcd();
void insertarcliente();
void alquilarcd();
void retornarcd();
 
 
 
int main() {
    int a;
    do {
        printf("alquiler de CDs\n");
        printf("Menu\n");
        printf("Digite Opcion\n");
        printf("1.insertar cliente\n");
        printf("2.insertar CD\n");
        printf("3.alquilar CD\n");
        printf("4.salir\n");
        printf("Opcion\n");
        scanf("%d", &a);
 
        switch (a) {
        case 1:
            insertarcliente();
            break;
        case 2:
            insertarcd();
            break;
        case 3:
            alquilarcd();
            break;
        }
    } while (a != 4);
    return 0;
}
 
void insertarcd() {
    printf("Codigo de CD\n");
    scanf("%d", &disco.codigocd);
    printf("Nombre del Autor\n");
 
    scanf("%s", disco.autor);
    printf("Titulo del Album\n");
    scanf("%s", disco.album);
    printf("Nombre del Titulo\n");
    scanf("%s", disco.titulo);
    printf("Numero De Pistas\n");
    scanf("%d", &disco.npista);
    FILE *archivo;
    archivo = fopen("c:\\disco.txt", "w");
    if (archivo == NULL)
        return;
    fprintf(archivo, "\n");
    fclose(archivo);
}
 
void insertarcliente() {
    printf("Digite la C.C\n");
    scanf("%d", &clien.cedula);
    printf("Digite el Nombre\n");
    scanf("%s", clien.nombre);
    printf("Digite el Apellido\n");
    scanf("%s", clien.apellido);
    printf("Digite La Fecha Nacimiento\n");
    scanf("%s", clien.fdn);
    printf("Digite el sexo\n");
    scanf("%s", clien.sexo);
    printf("Digite la Edad\n");
    scanf("%d", &clien.edad);
    FILE *archivo;
    archivo = fopen("c:\\clientes.txt", "w");
    if (archivo == NULL)
        return;
    fprintf(archivo, "%d,%s,%s,%s,%s,%d\n", clien.cedula, clien.nombre, clien.apellido, clien.fdn, clien.sexo, clien.edad);
    fclose(archivo);
}
void alquilarcd() {
    printf("digite la cc\n");
    scanf("%d", &clien.cedula);
    FILE *fichero;
    fichero = fopen("c:\\clientes.txt", "r");
    while (!feof(fichero))
        ;
    rewind(fichero);
    fclose(fichero);
    getchar();
}
void retornarcd() {
}