El bucle while con ¡feof me duplica el último registro [C]

[rdv316]

Hola.
Tengo un inconveniente con el bucle while y es que al imprimir el archivo me duplica el último registro, encontré este mismo inconveniente posteado en distintos foros pero ningún ej. me dio resultado.-
¿Me podrían decir cómo puedo hacer para solucionarlo?.-

Código C:

#include <stdio.h>
#include <string.h>
 
struct registro{
    char marcado;
    char ingles[30];
    char espaniol[30];
}est;
 
int main(void){
    printf("\n ingl%cs.......: ", 130);
    gets(est.ingles);
 
    printf("\n traducci%cn...: ", 162);
    gets(est.espaniol);
 
    FILE * ptr;
 
    if( ( ptr=fopen("archivo.txt","a+") ) == NULL ){
        printf("\n No se pudo abrir el archivo...");    
    }
    else{
        fwrite(&est,sizeof(est),1,ptr);
        est.marcado = '*';
    
        rewind(ptr);
 
        printf("\n  Borrado   Ingl%cs       Espa%col", 130, 164);   
        while(!feof(ptr)){
            fread(&est,sizeof(est),1,ptr);
            printf("\n %8d %8s %13s", est.marcado ,est.ingles, est.espaniol);   
        }
 
        fclose(ptr);        
    }
 
    return 0;
}
 

Saludos.
Daniel

[razpeitia]

Tienes un error en la linea 13! (Solo bromeo, no puedo ver tu código).

[aguml]

fread retorna el numero de elementos leidos, simplemente comprueba que fread retorne el numero de elementos que quieres leer, en este caso 1, y si no retorna ese valor es porque huvo algun error o porque llegó al oef por lo que no imprimes y listo. Otra cosa, usa ferror() dentro del bucle por si hay algun error salir de el tambien.

[rdv316]

Hola aguml.-
Gracias, con esta sintaxis me funciona correctamente.-

Código C:

...........
while (fread(&est, sizeof est, 1, ptr) == 1){
    printf("\n %8d %8s %13s", est.marcado ,est.ingles, est.espaniol);  
}
...........
 

Saludos.
Daniel

[vangodp]

El error de tu programa consiste en los siguientes errores:
Imaginas que se han leído todas las estructuras y estas en la ultima aun por leer, fread lee esa ultima estructura(Todo correcto nada falla), printf la imprime. El amiguete feof() no ve ningún problema ya que la ultima lectura ha sido correcta, el bucle se repite una vez más(No debería, ya imprimimos la ultima estructura guardada.), entonces esta vez si que falla la lectura con fread ya que no hay nada mas que leer, no se guarda nada en la estructura( Eso porque fallo fread.), pero recuerde que la estructura aun conserva lo ultimo guardado anteriormente, feof() ahora si va hacer de chivato a causa de fread haber fallado pero no sin antes el printf haber imprimido una vez más.

Como vemos el programa hace lo que se pide, el problema es que feof solo va dar error después de que fread haya fallado, pero como imprimimos una vez más antes de salir sale malo ya que se imprime 2 veces lo ultimo.

Mejor usar los valores de retorno que te proporcionan las funciones que usas para leer desde los archivos para que eso no pase. Además de usar ferror y feof pero a modo de comprobación solamente.

Suerte

[rdv316]

Hola vangodp, creme que lo pusiste en tu post lo voy a colgar de un cuadro, que interesante (por lo menos para mí) cuando encuentro este tipo de documentación, mira que antes de dar de alta el post leí en mi libro y en la web y creme que no encontré una documentación parecida a lo tuyo, ni parecida.-

Saludos.
Daniel

[vangodp]

La verdad es que en muchos sitios te enseñan a usar el feof y no es que este mal, lo malo es que pensamos que para todo va servir y no es así.
Lo mejor es casi siempre usar los valores de retorno de las funciones como bien comenta aguml.
Créame si te digo que fue hace poco que me entere del mismo XD
Aquí puedes leer algo al respecto, también ver ejemplos de como usar algunas funciones para leer archivos de una forma más eficaz en C/C++: http://stackoverflow.com/questions/5...s-always-wrong

No me recuerdo quien me paso el enlace este, pero fue en este mismo foro no hace mucho.

Bueno.. Espero que te sirva.