Ayuda - Programa en C que muestra algo que no debe mostrar (FILE,fprintf,fscanf)

[El-Brujito]

Hola, que tal, soy un hombre de 26 años, la primera ves que yo entre a esta pagina web fue hace mas de 7 años, yo me habia registrado para entonces con otro nick, y bueno esta pagina me trae muchos recuerdos, pero vamos al punto.

Sucede que estoy intentando crear un programa en el lenguaje de programación C, pero necesito ayuda, ya que el programa tiene un problema, y es que durante el tiempo de ejecucion, luego de la compilacion, presenta un error.

Es un programa que crea un archivo de registros, y tambien lo lee e imprime su contenido en pantalla.

El problema esta al momento de imprimir en pantalla los registros creados en el archivo, ya que si usted crea el archivo con 5 registros, el programa va a imprimir 6 registros, es decir, siempre imprime un registro mas que no fue solicitado, y esto no es correcto, porque yo quiero que imprima el contenido del archivo, y no registros "fantasmas".

Asi que necesito a algun experto que me ayude.

Este es el codigo en cuestion


Un programa con dos funciones, CreateFile: crea un archivo con la cantidad de registros ingresada por el usuario; y ReadFile: lee un archivo e imprime en pantalla los registros que contiene ese archivo, parece ser que en esta funcion (ReadFile) es donde esta el problema pero no puedo comprender o encontrar el error

Código (c) [Seleccionar] Expandir

// testing file structure and visibility
#include "stdio.h"
#include "stdlib.h"

void CreateFile();
void ReadFile();

struct Record
{
char Charac[20];
};

int main()
{
int Command=-1,i=0;
struct Record DefaultRecord;

for(i=0;i<=19;i++)
{

if(i==19)
{
DefaultRecord.Charac[i]='@';
}
else
{
DefaultRecord.Charac[i]='#';
}

}

while(Command!=0)
{

printf("Main menu:\n");
printf("\n1 - Create a new file.\n");
printf("2 - read a file.\n");
printf("0 - end program.\n");
scanf("%d",&Command);

switch(Command)
{
case 1:
CreateFile();
break;
case 2:
ReadFile();
break;
default:
printf("%d is not a valid command.\n",Command);
}

}

return 0;
}

void CreateFile()
{
int nR=0,i=0,n=0,x=0;
char FileName[20]; // FIX
FILE *fPtr;
struct Record XRecord;

for(i=0;i<=19;i++)
{

if(i==19)
{
XRecord.Charac[i]='@';
}
else
{
XRecord.Charac[i]='#';
}

}

printf("\nPlease enter the number of records to create the file: ");
scanf("%d",&nR);
printf("\nPlease enter the name of the file including his type (aaa.aa): ");
scanf("%s",&FileName);

if((fPtr=fopen(FileName,"w"))==NULL)
{
printf("nFile %s could not be opened.\n",FileName);
}
else
{
for(i=1;i<=nR;i++)
{
   for(x=0;x<=19;x++)
   {
   fprintf(fPtr,"%c",XRecord.Charac[x]);
   }
n++;
printf("n=%d\n",n);
}
fclose(fPtr);
}
}

void ReadFile()
{
int v=0,i=0,n=0,x=0;
FILE *fPtr;
char FileName[20];
struct Record XRecord;

for(i=0;i<=19;i++)
{
if(i==19)
{
XRecord.Charac[i]='Y';
}
else
{
XRecord.Charac[i]='x';
}
}
printf("\nPlease enter FileName: ");
scanf("%s",&FileName);

if((fPtr=fopen(FileName,"r"))==NULL)
{
printf("%s could not be opened.\n",FileName);
}
else
{
printf("\n");

while(!feof(fPtr))
{
   n++;

   for(x=0;x<=19;x++)
   {
   fscanf(fPtr,"%c",&XRecord.Charac[x]);
   }
printf("Record Number:%d\n",n);

for(i=0;i<=19;i++)
{
printf("%c",XRecord.Charac[i]);
}
printf("\n");
}

}
fclose(fPtr);
}


Alguien ayudeme por favor  

[AlbertoBSD]

Hola después de entender mas o menos tu codigo y encontrar algunos detalles no veo bien cual sea el problema.

Si solo si escribes el archivo cuantos bytes pesa?

Detalles vistos:

En la funcion readfile el ciclo de inicialización sale sobrando.

Código (c) [Seleccionar] Expandir

for(i=0;i<=19;i++)
{
if(i==19)
{
XRecord.Charac[i]='Y';
}
else
{
XRecord.Charac[i]='x';
}
}

Por que no usar fwrite para escribir toda la estrucuctura de golpe?

Ejemplos:

[youtube=640,360]https://www.youtube.com/watch?v=DOqtEMW1Nqk[/youtube]

Saludos

[El-Brujito]

Hola después de entender mas o menos tu codigo y encontrar algunos detalles no veo bien cual sea el problema.

Hola amigo, gracias por responder, dios te bendiga, el problema esta en que la funcion ReadFile imprime en pantalla un registro adicional el cual no fue solicitado, por ejemplo si usted crea un archivo con 2 registros, ReadFile mostrara en pantalla 3 registros, de los cuales el tercero no tengo idea de donde habra salido; si usted crea el archivo con 3 registros, ReadFile mostrara 4,

hay un error porque si el archivo es creado con X numero de registros, esa cantidad es la que debe de mostrar, pero ReadFile esta mostrando X+1 registros

Si solo si escribes el archivo cuantos bytes pesa?

Pesa 20 bytes, segun windows, (quizas el tamaño varie segun el computador)

Detalles vistos:

En la funcion readfile el ciclo de inicialización sale sobrando.

¿Sobrando mi amigo querido?, usted dice que se desborda el buffer? de verdad que no entiendo porque si el array tiene 20 elementos, esta inicialización, la cual es solo para rellenar, en realidad sus valores no se usan, pues, no le veo nada de malo

Código (c) [Seleccionar] Expandir

for(i=0;i<=19;i++)
{
if(i==19)
{
XRecord.Charac[i]='Y';
}
else
{
XRecord.Charac[i]='x';
}
}

Por que no usar fwrite para escribir toda la estrucuctura de golpe?

Primero estaba probando con fwrite y fread, pero como el programa continuo presentando el error, intente con fprintf y fscanf, para ver si se arreglaba, pero no se arreglo.

Ejemplos:

[youtube=640,360]https://www.youtube.com/watch?v=DOqtEMW1Nqk[/youtube]

Saludos

Gracias por ese ejemplo AlbertoBSD, lo tomare en cuenta

[AlbertoBSD]

Hola

¿Sobrando mi amigo querido?, usted dice que se desborda el buffer? de verdad que no entiendo porque si el array tiene 20 elementos, esta inicialización, la cual es solo para rellenar, en realidad sus valores no se usan, pues, no le veo nada de malo

me refiero a que : ¿Para que inicializar el buffer con datos? si al final vas a terminar sobreescribiendo sobre el con los datos desde el archivo.

Intenta escribir el archivo con este ciclo:

Código (c) [Seleccionar] Expandir

for(i=0;i<nR;i++)
{
   for(x=0;x<20;x++)
   {
   fprintf(fPtr,"%c",XRecord.Charac[x]);
   }
n++;
printf("n=%d\n",n);
}

Personalmente no me gusta usar <= en los codigos, prefiero usar simplemente mejor que.

Saludos

[class_OpenGL]

He compilado el código y me ha dado unas cuantas advertencias. Deberías vigilarlas. Yo no voy a mostrar las advertencias pues lo puedes mirar tu mismo, pero aun así es preferible que un programa no las tenga, pues son potenciales errores que después cuesta solucionar (lo digo por experiencia).

Respecto al error, es bastante fastidiante (me ha pasado hace poco xD). Lo que pasa es que el indicador de feof se activa solo cuando se intenta leer un carácter pero ya no quedan más. Entonces, cuando terminas de leer el último registro, no llegas a activar el feof y por lo tanto el while se ejecuta una vez más.

Para solucionarlo, puedes comprobar que no has llegado al final del archivo antes de mostrar el supuesto registro:

Código (c) [Seleccionar] Expandir

while(!feof(fPtr)) {
n++;

for(x = 0; x <= 19; x++)
fscanf(fPtr, "%c", &XRecord.Charac[x]);

if(!feof(fPtr)) { // Compruébalo aquí
printf("Record Number:%d\n",n);

for(i=0;i<=19;i++)
printf("%c", XRecord.Charac[i]);

printf("\n");
}
}

[AlbertoBSD]

Si es cierto!!!!

Una solucion alterna a este problema es determinar la longitud del archivo y dividirlo entre el tamaño la estructura y asi solo leer el numero exacto de datos.

Posiblemente te pasaba lo mismo con el fread.

Saludos

[MAFUS]

Eso es porqué al leer el tercer registro todavía no has llegado al final del archivo por eso el bucle itera una última vez.

Crea una variable tipo char (yo la llamo c) y cambia las líneas 132 y 133 por las siguientes:

Código [Seleccionar] Expandir

while((c = fgetc(fPtr)) != EOF) {
    ungetc(c, fPtr);

[El-Brujito]

Gracias a los que respondieron, anotare sus nombres por alli para agredecerles cuando pueda, he resuelto el problema utilizando este codigo, sin embargo sus respuestas (AlbertoBSD, class_OpenGL, MAFUS), son muy utiles, ya que tambien resuelven el problema mediante codigos similares

Este es el que he usado, sustituyendo el while por este:

Código (c) [Seleccionar] Expandir

while(1)
{
   n++;

   for(x=0;x<=19;x++)
   {
   fscanf(fPtr,"%c",&XRecord.Charac[x]);
   }
   if(feof(fPtr))
   {
       break;
   }
printf("Record Number:%d\n",n);

for(i=0;i<=19;i++)
{
printf("%c",XRecord.Charac[i]);
}
printf("\n");
}

He compilado el código y me ha dado unas cuantas advertencias. Deberías vigilarlas. Yo no voy a mostrar las advertencias pues lo puedes mirar tu mismo, pero aun así es preferible que un programa no las tenga, pues son potenciales errores que después cuesta solucionar (lo digo por experiencia).

Respecto al error, es bastante fastidiante (me ha pasado hace poco xD). Lo que pasa es que el indicador de feof se activa solo cuando se intenta leer un carácter pero ya no quedan más. Entonces, cuando terminas de leer el último registro, no llegas a activar el feof y por lo tanto el while se ejecuta una vez más.

Para solucionarlo, puedes comprobar que no has llegado al final del archivo antes de mostrar el supuesto registro:

Código (c) [Seleccionar] Expandir

while(!feof(fPtr)) {
n++;

for(x = 0; x <= 19; x++)
fscanf(fPtr, "%c", &XRecord.Charac[x]);

if(!feof(fPtr)) { // Compruébalo aquí
printf("Record Number:%d\n",n);

for(i=0;i<=19;i++)
printf("%c", XRecord.Charac[i]);

printf("\n");
}
}

Excelente respuesta mi amigo!

Si es cierto!!!!

Una solucion alterna a este problema es determinar la longitud del archivo y dividirlo entre el tamaño la estructura y asi solo leer el numero exacto de datos.

Posiblemente te pasaba lo mismo con el fread.

Saludos

Tu respuesta es muy util

Eso es porqué al leer el tercer registro todavía no has llegado al final del archivo por eso el bucle itera una última vez.

Crea una variable tipo char (yo la llamo c) y cambia las líneas 132 y 133 por las siguientes:

Código [Seleccionar] Expandir

while((c = fgetc(fPtr)) != EOF) {
   ungetc(c, fPtr);

Teneis toda la razon, tu respuesta es muy buena, y explica claramente como resolver el problema.

Os aprecio a todos por la ayuda que me han dado