Observaciones:
1.- Estás definiendo un array de estructuras con ámbito global y luego vuelves a definir otro array con el mismo nombre pero con ámbito local.
Es decir, este código:
struct datos{
char nombre[30];
int clave;
}alumnos[];
Es como si hicieras esto:
struct datos{
char nombre[30];
int clave;
};
struct datos alumnos[];
No necesitas definir este array de estructuras porque ya lo haces en la función main
. Además, en C no puedes omitir el tamaño del array (de ley debes especificarle una longitud).
2.- La variable indice
nunca es inicializada, por lo tanto, tiene contenido basura. En este caso, es valor por defecto de esta variable debería ser 0
, ya que para recorrer el array se empieza desde la posición 0
.
3.- No necesitas invocar a la función fopen
en cada iteración, no hace falta. Esto solo ocasiona que ocurra fuga de memoria, porque perdemos la referencia del bloque anterior.
4.- Falta invocar la función fclose
. Es buena práctica cerrar los recursos cuando ya no lo necesites.
5.- La variable agregar
es innecesaria. Porque el programa solo pedirá información al usuario hasta cierto límite (en este caso 3
), así que no puedes usar esta condición como salida del bucle:
agregar==1
De lo contrario, si se llegara a pedir más información del límite establecido, ocurriría un desbordamiento de búfer.
La condición de salida debería ser así:
indice < personas
Con las observaciones mencionadas anteriormente, el código quedaría así:
struct datos{
char nombre[30];
int clave;
};
int main()
{
FILE *archivo =fopen("lista.txt","wb");
struct datos alumnos[personas];
int indice = 0;
do
{
printf("Ingresa el nombre del alumno #%d",(indice+1));
scanf("%s", alumnos[indice].nombre);
fflush(stdin);
printf("Ingresa la clave #%d",(indice+1));
scanf("%d",&alumnos[indice].clave);
fflush(stdin);
fprintf(archivo,"\n %s\n %i",alumnos[indice].nombre, alumnos[indice].clave);
indice++;
}while(indice < personas);
fclose(archivo);
return 0;
}
El código de arriba escribirá en el archivo después de pedir los tres datos al usuario.
La función fflush no limpia el búfer stdin, sino el stdout. Por más que funcione en algunos sistemas, no se recomienda su uso por cuestión de portabilidad.
Una manera de hacerlo de forma estándar (que funcione en cualquier plataforma) es de esta manera:
while(getchar() != '\n');
El código completo (junto con fgets) quedaría así:
int main()
{
FILE *archivo =fopen("lista.txt","wb");
struct datos alumnos[personas];
int indice = 0;
do
{
printf("Ingresa el nombre del alumno #%d",(indice+1));
fgets(alumnos[indice].nombre, 30, stdin);
printf("Ingresa la clave #%d",(indice+1));
scanf("%d",&alumnos[indice].clave);
//Limpiamos el búfer del teclado
while(getchar() != '\n');
fprintf(archivo,"\n %s\n %i",alumnos[indice].nombre, alumnos[indice].clave);
indice++;
}while(indice < personas);
fclose(archivo);
return 0;
}
Nota: Se debe tomar en cuenta que la función fgets
incluirá el salto de línea en el array nombre
si no lee la cantidad máxima de caracteres que le especificamos en el segundo parámetro (en este caso es 30
).