Mi programa guarda basura al leer un archivo

[joelfs18]

Buenos días a todos, me dirijo a ustedes con este primer post para ver si me pueden ayudar en el siguiente problema:

He creado un código que guarda datos de un paciente en un arreglo de structs (por el momento, luego lo implementaré en listas y otros TDAs), pero cuando guardo los datos del paciente, no solamente guardo esos, sino también basura a continuación de cada dato (símbolos raros, letras raras, etc). He pensado en otras maneras de hacerlo, pero me urge saber por qué es que mi programa guarda basura. A continuación les dejo el código y una captura para que vean de lo que les hablo. De antemano, muchas gracias!

http://gyazo.com/0fcbc9f70700df76ec257becb46ac950

Código C:

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
typedef struct datos
{
    char nombre[30];
    char apellido[30];
    char rut[30];
    char anios[30];
    int prio;
}DATOS;
 
int main()
{
    DATOS X[4];
    int i=0,j=0;
    char car;
 
    FILE*in;
    in=fopen("datos.txt","r");
 
    while(!feof(in))
    {
 
        do
        {
            car=fgetc(in);
            X[i].apellido[j]=car;
            j++;
        }
        while(car!=',');
 
        X[i].apellido[j-1]="\0";
        j=0;
 
        do
        {
            car=fgetc(in);
            X[i].nombre[j]=car;
            j++;
        }
        while(car!=',');
 
        X[i].nombre[j-1]="\0";
        j=0;
 
        do
        {
            car=fgetc(in);
            X[i].rut[j]=car;
            j++;
        }
        while(car!=',');
 
        X[i].rut[j-1]="\0";
        j=0;
 
        do
        {
            car=fgetc(in);
            X[i].anios[j]=car;
            j++;
        }
        while(car!='\n');
 
        X[i].anios[j-1]="\0";
        j=0;
 
 
        car=fgetc(in);
        fscanf(in,"%d",&X[i].prio);
        i++;
    }
 
    for(i=0;i<4;i++)
    {
 
            printf("\n%s",X[i].apellido);
            printf("\n%s",X[i].nombre);
            printf("\n%s",X[i].rut);
            printf("\n%s",X[i].anios);
            printf("\n%d",X[i].prio);
    }
 
 
return 0;
}
 

[vangodp]

Cuando creas un objeto de una estructura esa puede contener residuos. Para evitar eso cuando crees un objeto(declaración) de una estructura cualquiera haga tipoEstructura nombreEstructura = {0};. Con eso todos los campos estarán establecido a nulos y ya no tendrás residuos.

Otra forma de formatear una estructura es usar strcpy asi: strcpy (X[0].nombre, "");

haga tipoEstructura nombreEstructura = {0}; tanto antes de guardar como antes de leer el archivo, así puedes estar seguro que la estructura estará limpia.

[aguml]

Iniciado por vangodp

Cuando creas un objeto de una estructura esa puede contener residuos. Para evitar eso cuando crees un objeto(declaración) de una estructura cualquiera haga tipoEstructura nombreEstructura = {0};. Con eso todos los campos estarán establecido a nulos y ya no tendrás residuos.

Otra forma de formatear una estructura es usar strcpy asi: strcpy (X[0].nombre, "");

haga tipoEstructura nombreEstructura = {0}; tanto antes de guardar como antes de leer el archivo, así puedes estar seguro que la estructura estará limpia.

yo en vez de crear la estructura cada vez para limpiarla haria memset(&estructura,0,sizeof(estructura));

[vangodp]

yo en vez de crear la estructura cada vez para limpiarla haria memset(&estructura,0,sizeof(estructura));

Yo no he dicho eso. He dicho que a la hora de declararla se le de el valor de cero. Has entendido lo que has querido pero no es así.

[dmorill]

Hola, sé que has realizado tu programa en c, pero no sé si se puede hacer también como en c++ unos constructores para un sctruct, así evitas al crear una estructura de tipo datos, los miembros tengas valores inesperados.

Código C++:

struct datos{
    char nombre[30];
    char apellido[30];
    char rut[30];
    char anios[30];
    int prio;
    datos();
};
datos::datos(){
    for (int i = 0; i < 30; i++){
        nombre[i] = 0;
        apellido[i] = 0;
        rut[i] = 0;
        anios[i] = 0;
    }
    prio = 0;
}
 

Saludos.