1

Al momento de querer guardar los valores solo me captura el primer nombre y valor y los demás no los guarda ni los muestra, ademas, si quiero intentar usar fgets no me deja poner el nombre.

Código:

#include <stdlib.h>
#define personas 3

struct datos{

    char nombre[30];
    int clave;
}alumnos[];

int main()
{
    FILE *archivo;
    struct datos alumnos[personas];
    int indice,agregar;
    do
    {
        archivo=fopen("lista.txt","wb");
        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++;

        scanf("%d",&agregar);

    }while(agregar==1);

    return 0;
}
0

1 respuesta 1

1

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).

9
  • hola, me funciono perfecto pero al momento de ponerlo adentro de un switch me salta para poner el primer nombre al igual que si lo pongo en un arrreglo, eso a que se debe?
    – Jose Cast
    Commented el 3 jun. 2020 a las 17:43
  • @JoseCast Se te salta el nombre, es decir, ¿no te deja ingresar el nombre?
    – MrDave1999
    Commented el 3 jun. 2020 a las 17:48
  • si, no me deja ingresar el nombre , solo aparece:" ingresar nombre #1 ingresar clave #1 :"
    – Jose Cast
    Commented el 3 jun. 2020 a las 17:49
  • @JoseCast Antes de pedir el nombre, invocas a la función scanf?
    – MrDave1999
    Commented el 3 jun. 2020 a las 17:50
  • 1
    ya me funciono , puse un ` while(getchar() != '\n');` antes del ingresar el nombre y ya me dejo ingresarlo , gracias
    – Jose Cast
    Commented el 3 jun. 2020 a las 17:54

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.