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

// 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  

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,

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

Detalles vistos:

¿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

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

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.


Gracias por ese ejemplo AlbertoBSD, lo tomare en cuenta

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:

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");
	}
}

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: