copiar un archivo en otro archivo distinto en C

[vangodp]

No use feof para la condicion del bucle. Mejor no lo uses nunca XDDD. O casi nunca.

Mejor usar el proprio valor de retorno de fgetc que retorna un entero igual a EOF si llegamos al fin del archivo, o la letra leida en formato decimal. Resultado.. No necesitamos a feof en ese ejemplo.

Código C:

#include <stdio.h>
 
int main () {
 
    FILE *file1 , *file2;
    int data1 =0;
    
    file1 = fopen ( "original.txt", "r" );
    file2 = fopen ( "copia.txt" , "w" );
    
    while ( (data1 = fgetc ( file1 )) != EOF ) {
        fputc ( data1, file2 );
    }
    
    fclose ( file1 );
    fclose ( file2 );
    
    return 0;
}
 

La razon por la que es mejor usar el valor de retorno de la propria funcion fgetc y no el de feof es que feof solo va retornar 0 si falla fgetc. Una posible solucion es poner antes del bucle una lectura previa con fgetc de esta forma:

Código C:

data1 = fgetc(file1);
    while( !feof(file1) ){
        fputc(data1, file2);
        data1 = fgetc(file1);
    }
 

En este caso si falla fgetc o resulta que no hay nada que leer no se entra al bucle. =)
Si haces como en tu ejemplo veras que se imprime una letra de más en el archivo y esto es por lo que comente antes, fgetc retorna EOF en caso llegado al fin del archivo, y solo entonces despues de eso es cuando feof falla, pero no si antes imprimir ese EOF sobrante con el fputc.

Código C:

while( !feof(file1) ){
        data1 = fgetc(file1);
        fputc(data1, file2);
    }
 

Total, son mil razones para no usar feof, tenga en cuanta que feof depende de otras funciones para que retorne 0, pero esas mismas funciones tienen sus valores de retorno, es solo investigar como funcionan todas ellas y tendras más control sobre tu programa.
No en todos los casos se desaconceja usar feof, solo que debes investigar más sobre su funcionamiento.
http://www.cplusplus.com/reference/cstdio/feof/
http://www.cplusplus.com/reference/cstdio/fgetc/
http://www.cplusplus.com/reference/cstdio/fputc/