Foro de elhacker.net

Hola amigos,
tengo un problema con este código basado en archivos y no entiendo porque
en esta parte del programa me lee toda la cadena, cuando esperaba solo leyera y me mostrara por pantalla el nombre de la persona y no toda la cadena seguida de nombre, ciudad y edad.

 
        fgets(persona.nombre,40,fichero);
        puts(persona.nombre);
 
 

Este es el código completo:

 
#include <stdio.h>
#include <string.h>
 
struct informe{
    char nombre[40];
    char ciudad[40];
    int edad;
}persona;
 
int main()
{
    char name[20]="gente.dat";
    FILE *fichero;
    fichero=fopen(name,"w");
    if(fichero==NULL){
        printf("\nEl archivo no existe\n");
        exit(1);
    }
    else{
        /*Escribo el archivo*/
        printf("Introduzca nombre: ");
        gets(persona.nombre);
        fprintf(fichero,"%s",persona.nombre);
        printf("Introduzca ciudad: ");
        gets(persona.ciudad);
        fprintf(fichero,"%s",persona.ciudad);
        printf("Introduzca edad: ");
        scanf("%d",&persona.edad);
        fprintf(fichero,"%d",persona.edad);
        fclose(fichero);
    }
 
    /* Leo el archivo */
    printf("\nEsto es lo que se ha introducido en el archivo:\n\n");
    printf("_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _\n\n");
    fichero=fopen(name,"r");
    if(fichero==NULL){
        printf("\nEl archivo no existe\n");
        exit(1);
    }
    while(feof(fichero)==0){
 
        fgets(persona.nombre,40,fichero);
        puts(persona.nombre);
        fgets(persona.ciudad,40,fichero);
        puts(persona.ciudad);
        fscanf(fichero,"%d",&persona.edad);
        puts(persona.edad);
    }
    fclose(fichero);
return 0;
}
 

Gracias, un saludo

El problema se genera por la forma en que guardas los datos en el archivo:

/*Escribo el archivo*/
printf("Introduzca nombre: ");
gets(persona.nombre);
fprintf(fichero,"%s",persona.nombre);
 
printf("Introduzca ciudad: ");
gets(persona.ciudad);
fprintf(fichero,"%s",persona.ciudad);
 
printf("Introduzca edad: ");
scanf("%d",&persona.edad);
fprintf(fichero,"%d",persona.edad);
 
fclose(fichero);

Por ejemplo si el nombre es "X", la ciudad "Y" y la edad "5" los datos se envían sin separación alguna como la secuencia "XY5".

Para solucionarlo debes enviar cada campo seguido de un avance de linea:

/*Escribo el archivo*/
printf("Introduzca nombre: ");
gets(persona.nombre);
fprintf(fichero, "%s\n", persona.nombre);
 
printf("Introduzca ciudad: ");
gets(persona.ciudad);
fprintf(fichero, "%s\n", persona.ciudad);
 
printf("Introduzca edad: ");
scanf("%d", &persona.edad);
fprintf(fichero, "%d\n", persona.edad);
 
fclose (fichero);

Otros errores que debes corregir:

* Antes de utilizar la función exit debes incluir el encabezado <stdlib.h>.

* No puedes imprimir un entero (la edad) con la función puts:

fscanf(fichero,"%d",&persona.edad);
puts(persona.edad);

En lugar de puts utiliza printf.

* Después de leer el ultimo campo de tipo int (de nuevo la edad) debes descartar el resto de la linea.

* La condición del bucle de lectura garantiza un iteración de mas, en otras palabras si tienes un registro se trataran de mostrar dos. Para evitarlo debes verificar (por lo menos) el resultado de la lectura del primer campo. Combinado con el punto anterior:

while (fgets(persona.nombre, 40, fichero) != NULL){
   printf("%s", persona.nombre);
 
   fgets(persona.ciudad, 40, fichero);
   printf("%s", persona.ciudad);
 
   fscanf(fichero,"%d",&persona.edad);
   printf("%d\n", persona.edad);
 
   {
      int ch;
 
      while ((ch = fgetc(fichero)) != EOF && ch != '\n')
         ;
   }
}
fclose(fichero);

Por ultimo debes evitar el uso de magic numbers y las funciones gets y feof. Mas información en la pagina |Lo que no hay que hacer en C/C++. Nivel basico| (http://foro.elhacker.net/programacion_cc/lo_que_no_hay_que_hacer_en_cc_nivel_basico-t277729.0.html).

Un saludo

Hola rir3760,

pufff que nivel, como te agradezco tu inmejorable explicación. La verdad que más claro y educativo imposible.

Saludos master!!

Pdata: buen post el que me indicaste, muchas veces tenemos las cosas enfrente de nuestros ojos y nos las vemos o el subsconciente nos juega malas pasadas, jajaja