Hola

Tengo un programa en el cual tengo que crear un nuevo tipo de dato struct en el cual voy a tener 3 variables, una que es el documento de la persona, la otra es el nombre de la persona y por ultimo la nacionalidad de la persona.

Al principio lo que tengo que hacer es cargar manualmente el nuevo tipo de dato, la carga va a finalizar cuando el documento sea igual a 0.

Luego va a pasar todo lo cargado a un archivo txt. (Hasta aca, va perfecto todo )

El problema surge cuando tengo que leer el archivo e imprimir el contenido . Lo que sucede es que cuando pongo while(!feof(fd)) la lectura del archivo no termina nunca, siempre me arroja los mismos valores, nunca termina el ciclo y ademas nunca me carga la nacionalidad.

El archivo se va a cargar de la siguiente manera.

38747443,Agustin,Argentina

Es decir que en cada linea del archivo txt voy a tener ese tipo de secuencia.

Este es el codigo que hice

#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
#define N 5
 
struct s_datos
{
    int doc;
    char nombre[50];
    char pais[50];
 
};
typedef struct s_datos dato;
 
int CARGA(dato charge[N]);
void CREACION(dato charge[N],int);
void IMPRESION();
 
int main()
{
    int i=0,j=0;
    dato charge[N];
    i=CARGA(charge);  /// i es la cantidad de personas que tengo.
    printf("\nEl valor de i es %d",i);
    for (j=0;j<i;j++)
    {
        printf("\n%d",charge[j].doc);
        printf("\n%s",charge[j].nombre);
        printf("\n%s",charge[j].pais);
    }
CREACION(charge,i);  ///Llamo a la funcion para crear un txt y le paso el nuevo dato struct y la cantidad de personas ingreadas.
printf("\n");
IMPRESION();
 
 
 
    return 0;
}
 
int CARGA(dato charge[N])
{
   int i=0,j=0,e=0;
   char letra='a';
   charge[i].doc=4;
   while (i<N)
   {
       printf("\nIngrese el numero de documento: ");
       scanf("%d",&charge[i].doc);
           if (charge[i].doc!=0)
           {
               printf("\nIngrese el nombre: ");
 
                   while(letra!='\r')
                   {
                      letra=getche();
                      charge[i].nombre[j]=letra;
                      j++;
                   }
                   charge[i].nombre[j]='\0';
               j=0;
               letra='a';
 
               printf("\nIngrese el pais: ");
 
                   while(letra!='\r')
                   {
                      letra=getche();
                      charge[i].pais[e]=letra;
                      e++;
                   }
                   charge[i].pais[e]='\0';
                   e=0;
               letra='a';
               i++;
           }
       if(charge[i].doc==0)
       {
           return i;
       }
   }
   return i;
 
}
 
void CREACION(dato charge[N],int i)
{
    FILE*arch;
    arch=fopen("C:\\Users\\Pablo-PC\\Desktop\\personas.txt","w");
    int j=0;
    while(j<i)
    {
        fprintf(arch,"%d,%s,%s\n",charge[j].doc,charge[j].nombre,charge[j].pais);
        j++;
    }
    fclose(arch);
}
 
void IMPRESION()
{
    FILE*fd;
 
    int doc,i=0;
    char nombre[50]={0},pais[50]={0};
 
    fd=fopen("C:\\Users\\Pablo-PC\\Desktop\\personas.txt","r");
 
    printf("Documento\tNombre\tPais\n");
    fscanf(fd,"%d,%s,%s\n",&doc,&nombre,&pais);
    while(!feof(fd) && i<4)
    {
        printf("%d\t%s\t%s\n",doc,nombre,pais);
        fscanf(fd,"%d,%s,%s\n",&doc,&nombre,&pais);
        i++;
    }
    fclose(fd);
}
 
 

La funcion que funciona mal es IMPRESION y no le veo defecto alguno.

Hola

Pero en ese caso me imprimiria con las comas incluidas en cada columna, como hago para que no me tome las comas?.

Muy buenas. Para solucionar tu problema con las comas puedes hacer uso de una funcionalidad un poco más avanzada de fscanf. Impresión queda así:

La cadenade control de fscanf ha cambiado un poco.
Tu idea de usar comas separadoras está bien, pues es el mismo formato que scanf va a recibir y va a desechar esas comas. El problema venía que recogías tanto nombre como país con el argumento %s que coge toda la cadena hasta que haya un espacio en blanco (tabulación, nueva línea, etc.) por lo que también te pillaba la coma.
Sin embargo haciendo uso de %[^,] le dices a fscanf: guarda la cadena hasta que encuentres el caracter coma (los corchetes indican los caracteres que debe recoger fscanf, el acento circunflejo al principio indica negación, es decir, que se debe recoger todos los caracteres hasta encontrar uno de esa lista). Así que capturarás toda la cadena hasta que llegues a la coma.
Después te encuentras la coma que vas a desechar y empieza otra captura de cadena hasta que llegue al final de ésta, el caracter de nueva línea, como marca %[^\n].

Al igual que scanf, printf tiene su técnica avanzada para formatear el texto.
Verás:
%s sabes que escribe una cadena.
%Ns siendo N un número entero constante hace que la cadena tenga como mínimo N caracteres y se alinee por la derecha. Si la cadena es más corta de lo que marca N los caracteres que faltan se rellenan con espacios.

%-Ns siendo N un número entero constante hace que la cadena tenga como mínimo N caracteres y se alinee por la izquierda. Si la cadena es más corta de lo que marca N los caracteres que faltan se rellenan con espacios.

%.Ms siendo M un número entero constante hace que se tomen de la cadena los M primero caracteres a escribir descartando los demás.

Por supuesto las dos formas se pueden mezclar:
%N.Ms
%-N.Ms

Ejemplos serían:
%5s
%-5s
%10.6s
%-10.6s

scanf y printf tienen aún muchas más opciones avanzadas. Te animo a que busques por internet su documentación y juegues con estas dos funciones. Puedes llegar a hacer muchas cosas con ellas.

Para que juegues con printf y después lo apliques en tu programa te dejo un ejemplo muy sencillo:

#include <stdio.h>
 
int main() {
    printf("|%15.5s|", "Mi_cadena");
 
    return 0;
}