Ficheros.

[yournightmare86]

Hola buenas, trás mucho tiempo vuelvo al uso de ficheros en C(algo en lo que no soy precisamente un experto). Aquí les dejo mi código.
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#define RUTA "/home/madrid/C/registro_alumnos.txt"

//Prototipos de las funciones.

int menu();
void anyadir_alumno();
//void modificar_alumno();
//void visualizar();

typedef struct
{
int dia;
int mes;
int anyo;
}fecha;

typedef struct
{
char nombre[20];
char apellidos1[20];
char apellidos2[20];
fecha fecha_nac;
char nota_evaluacion[4];
char nota_final[4];
}alum;


main()
{
int i,op;

while((op = menu()) != 4)
{
switch(op)
{
case 1:
anyadir_alumno();
break;
case 2:
//modificar_alumno();
break;
case 3:
//visualizar();
break;
case 4:
return 0;
default:
printf("\n Opcion incorrecta. Seleccione otra opcion.\n");
}
}
return 0;
}

int menu()
{
int op,ok;

printf("\n 1) Anyadir alumno.");
printf("\n 2) Modificar alumno.");
printf("\n 3) Visualizar alumnos.");
printf("\n 4) Salir.");

do{
printf("\n\n Seleccione opcion: ");
ok=scanf("%d",&op);
} while(!ok);
return op;
}

void anyadir_alumno()
{
alum * alumno;
FILE * fd;
static int i=0;

i++;
fd = fopen(RUTA,"a");
if(fd == NULL) return;

printf("\n\n\t Datos del alumno numero %d",i);
printf("\n\t Nombre: "); scanf("%s",&alumno[i].nombre);
printf("\n\t Apellido 1: "); scanf("%s",&alumno[i].apellidos1);
printf("\n\t Apellido 2: "); scanf("%s",&alumno[i].apellidos2);
printf("\n\t\t Fecha: ");
printf("\n\t\t Dia: "); scanf("%d",&alumno[i].fecha_nac.dia);
printf("\n %d has introducido",alumno[i].fecha_nac.dia);

fwrite(alumno[i].nombre,1,strlen(alumno[i].nombre),fd);
fwrite("-",1,1,fd);
fwrite(alumno[i].apellidos1,1,strlen(alumno[i].apellidos1),fd);
fwrite("-",1,1,fd);
fwrite(alumno[i].apellidos2,1,strlen(alumno[i].apellidos2),fd);
fwrite("-",1,1,fd);
fwrite(alumno[i].fecha_nac.dia,1,sizeof(alumno[i].fecha_nac.dia),fd);
fwrite("-",1,1,fd);
fclose(fd);
}

De momento estoy con la primera función, la de añadir un nuevo registro(por eso he hecho comentarios las llamadas a las demás funciones). Cuando intento guardar en el fichero la variable día, me da error en la línea que marqué en rojo, y no se porqué. Alguna sugerencia?Debería usar fprintf en vez de fwrite?
Gracias por su paciencia y ayuda.
Saludos.

[Eternal Idol]

Cuando pongas codigo en el foro usa la etiqueta code entre [] para que salga asi (ahi esta el codigo correcto por cierto):

Código:

fwrite(&alumno[i].fecha_nac.dia, sizeof(alumno[i].fecha_nac.dia), 1, fd);

Código:

size_t fwrite(const void *buffer, size_t size, size_t count, FILE *stream);

Como vemos en el prototipo de fwrite lo primero es un buffer, entonces le pasamos la direccion de memoria donde esta el valor de dia (si eso estuviera bien ...). Despues va el tamaño de cada elemento y despues el elemento. Por ultimo el stream. Si lo que querias era escribir el numero en decimal (ocupando sizeof(int)) eso es lo que tenes que hacer, si queres escribirlo como texto podes probar con fprintf.

Ojo que eso compila pero estas usando alum *alumno sin darle ningun valor a ese puntero ... y despues accedes como si fuera un array ...

[yournightmare86]

Muchas gracias por tu respuesta Eternal Idol pero creo que no entendí bien lo último, que valor debo asignarle al puntero? No sé muy bien como hacer...
Por cierto cambié la línea por la que me propusiste y sigue sin funcionar...compila pero se cuelga.Dentro del fwrite, el 1 no debe ir en segundo lugar al igual que en las líneas

Código:

fwrite(alumno[i].apellidos1,1,strlen(alumno[i].apellidos1),fd);
fwrite(alumno[i].apellidos2,1,strlen(alumno[i].apellidos2),fd);

Saludos.

[Eternal Idol]

Iniciado por yournightmare86

Muchas gracias por tu respuesta Eternal Idol pero creo que no entendí bien lo último, que valor debo asignarle al puntero? No sé muy bien como hacer...

No uses un puntero a menos que vayas a usar memoria dinamica (malloc/new).

Iniciado por yournightmare86

Por cierto cambié la línea por la que me propusiste y sigue sin funcionar...compila pero se cuelga.

Si, te lo habia advertido: "Ojo que eso compila pero estas usando alum *alumno sin darle ningun valor a ese puntero ... y despues accedes como si fuera un array ..."

Iniciado por yournightmare86

Dentro del fwrite

Esta explicado claramente arriba, mejor no puedo hacerlo y si es valido para cualquier llamada que hagas a esa funcion.

[yournightmare86]

Creo que empezaré de nuevo, que estoy hecho un lío. Probaré de otra manera. Muchas gracias.

[Nivel7]

Pues yo veo muy clara la explicacion.

la funcion fwrite, tienen su prototipo el cual te an mostrado. puedes verificarlo en stdio.h.

el problema en la funcion anyadir_alumno, es que estas declarando un puntero y posteriormente lo usas sin antes reservar memoria para este.

alum *alimnos = malloc(sizeof(alum)*10);//por ejemplo, y podrias redimencionar al requerir mas.

o si tienes un numero de registros determinado mejor usa un array:
alum alumnos[10];

agregar que en el while((op = menu()) != 4); y luego haces un case 4:
no crees que esta de sobra.
no tiene nada de malo pero puedes evitar una.

por como guardas tus datos en el fichero sería mejor usar fprintf y fscanf.
pero esto es segun tus requerimientos.

[yournightmare86]

Gracias por la respuesta. Ya voy entendiendo mejor que hace poco me hicieron un cacao en la cabeza...
Saludos.