Problema en C con fread y fwrite

[eferion]

Código C:

if(strlen(Buffer)>0) Buffer[strlen(Buffer)-1]='\0';
 

strlen() tiene internamente un contador... al pasarle un puntero empieza a recorrerlo hasta que encuentra un caracter nulo ('\0'), incrementando su contador interno por el camino. Una vez localizado este carácter devuelve el valor del contador.

Dicho esto, Buffer[strlen(Buffer)-1]='\0' va a eliminar SIEMPRE el último caracter de la secuencia leída. Es decir, si tenías 'HOLA\0' te vas a quedar con 'HOL\0'. No entiendo la utilidad de esa línea.

Y, por cierto, aquí te sucede lo mismo:

Código C:

fgets(Datos[X].Name,sizeof(Datos[X].Name),stdin);
Datos[X].Name[strlen(Datos[X].Name)-1]='\0';
 

Seguimos...

Código C:

while(!feof(Agenda)){
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
}
 

la condición del "while" no va a fallar nunca porque feof() no se actualiza hasta que no realizas una lectura fallida... y entre el último feof() del bucle y la condición del while no hay ningúna operación sobre el fichero...

puedes dejarlo como

Código C:

while( 1 )
{
  // ...
}
 

o así

Código C:

do
{
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
    // ...
    if(feof(Agenda)) break;
    // ...
    // if(feof(Agenda)) break; <-- comentado
} while( !feof( ) );
 

más que nada porque la condición del while, tal y como está el código, no va a devolver false en la vida.

Código C:

fgets(Buffer,200,Agenda);
if(feof(Agenda)) break;
if(strlen(Buffer)>0) Buffer[strlen(Buffer)-1]='\0';
strcpy(Datos[X].Name,Buffer);
 

No se si te has dado cuenta, pero te puedes ahorrar el uso de "Buffer"... primero haces una copia del fichero a "Buffer" y después copias ese mismo contenido de "Buffer" a la estructura "Datos"... ¿Por qué no haces una copia directa a "Datos"?

Algo así:

Código C:

fgets(Datos[X].Name,200,Agenda);
if(feof(Agenda)) break;
 

Por otro lado, no se si te has dado cuenta, pero "Buffer" tiene tamaño 200 y "Name", "Address" y "Email" tienen tamaño 100... si editas el archivo a mano y aumentas la longitud de uno de esos campos por encima de 100 vas a pisar memoria... para evitar eso puedes valerte de "sizeof"

Código C:

fgets(Datos[X].Name, sizeof(Datos[0].Name), Agenda );
if(feof(Agenda)) break;
 

Así te evitas el desbordamiento del buffer.

Otra cosilla: dado que utilizas 'X' para controlar el número máximo de elementos... "Algodon" es otra variable que te puedes ahorrar... básicamente cambia el if por:

Código C:

if( X == 0 ){
    printf("No hay datos que leer...\n\n");
}
 

Y listo.

Y, bueno, entiendo que este programa te funciona... ¿Qué problemas tienes al cambiar fgets por fread???