Gente sigo con problemas similares al anterior tema que publique , miren este codigo
es un directorio de telefono que estoy probando ...

#include <stdio.h>
#include <windows.h>
#include<string.h>
 
#define MAX_CHARS 256
 
 
 
typedef struct persona
{
	char nombre[MAX_CHARS],apellido[MAX_CHARS],telefono[20];
}sPersona;
 
void Agregar_Presona();
void Imprimir_Persona();
FILE* miarchivo;
char* nombrearchivo= "Directorio.txt";
 
 
 
int main()
{
	int a,opc;
	while(1)
	{
		system("cls");
	printf("1. Agregar Persona\n");
	printf("2. Ver personas\n");
	printf("3. Salir\n\n");
	printf("Elige una opcion y precione enter : ");
	scanf("%d",&opc);
	switch(opc)
	{
	case 1:
		{
			Agregar_Presona();
			break;
		}
	case 2:
		{
			Imprimir_Persona();
			break;
		}
	case 3:
		{
			ExitProcess(1);
		}
	default :
		{
			MessageBox(NULL,"Opcion Incorrecta","Directorio",MB_OK | MB_ICONERROR);
		}
	}
	}
}
 
 
void Agregar_Presona()
{
	sPersona nPersona;
	char *p;
	miarchivo= fopen(nombrearchivo,"a"); //abro el archivo "a" sirve para agregar al final
    if(miarchivo==NULL)return -1;
 
	system("cls");
	fflush(stdin);
 
	printf("Introduce el nombre del persona: ");
	fflush(stdin);
	fgets(nPersona.nombre, MAX_CHARS, stdin);
	//Elimina el avance de linea ... Gracias rir3760 !
	p = strchr(nPersona.nombre, '\n');
	if (p != NULL)
	*p = '\0';
 
 
 
	printf("Introduce el apellido de la persona: ");
	fflush(stdin);
	fgets(nPersona.apellido, MAX_CHARS, stdin);
	//Elimina el avance de linea ... Gracias rir3760 !
	p = strchr(nPersona.apellido, '\n');
	if (p != NULL)
      *p = '\0';
 
 
 
	printf("introduzca el telefono: ");
	fflush(stdin);
	fgets(nPersona.telefono, 20, stdin);
 
	//Elimina el avance de linea ... Gracias rir3760 !
	p = strchr(nPersona.telefono, '\n');
	if (p != NULL)
       *p = '\0';
 
 
	nPersona.nombre[0]=toupper(nPersona.nombre[0]); //Funcion que convierte la primera letra de la palabara en mayuscula
	nPersona.apellido[0]=toupper(nPersona.apellido[0]); //Funcion que convierte la primera letra de la palabara en mayuscula
 
 
 
	fprintf(miarchivo,"%-20s\t%-20s\t%-20s\n",nPersona.nombre,nPersona.apellido,nPersona.telefono);
 
    fclose(miarchivo);
 
 
}
 
 
 
void Imprimir_Persona()
{
	char cadena[MAX_CHARS];
	//char* cpToken;
    miarchivo= fopen(nombrearchivo,"r");
    if(miarchivo==NULL)return -1;
    system("cls");
    while(!feof(miarchivo))
	{
			fgets(cadena,MAX_CHARS,miarchivo);
			fputs(cadena,stdout);
	}
	printf("Presione una tecla para volver al menu ...");
	system("pause>nul");
 
 
    fclose(miarchivo);
}
 

el problema esta que cuando se lee el archivo se lee tambien el ultimo '\n' y pasa lo siguiente ...

como vemos se repite la ultima linea ... algun consejo ? S2!

En esa función no puedes retornar -1 ya que ella no retorna un valor (indicado mediante su tipo de retorno: void). Cambia la sentencia de retorno a:

if (miarchivo == NULL)
   return;

Y no necesitas utilizar la función feof (salvo ciertas excepciones es mejor evitar su uso) en su lugar se puede utilizar el valor de retorno de la función utilizada. En tu caso cambia esta parte:

fgets(cadena, MAX_CHARS, miarchivo);
while (!feof(miarchivo)){
   fputs(cadena,stdout);
   fgets(cadena,MAX_CHARS,miarchivo);
}

Por esta:

while (fgets(cadena, MAX_CHARS, miarchivo) != NULL)
   fputs(cadena, stdout);

Un saludo

Para explicar esto lo mejor es dividirlo en dos partes, el contenido del archivo y como opera la función feof.

1) En C un archivo de texto (mas bien un stream en modo texto) consiste de cero o mas lineas seguidas de ... nada. Por ejemplo:
El archivo se procesa carácter por carácter o (como es tu caso) por linea. Eventualmente se procesan todos los caracteres y, al no haber nada mas, se activa el estado de fin de archivo o eof.

2) El porque la ultima linea se procesa dos veces se debe al uso de feof, una explicación en detalle se encuentra en el tema Re: exec y sus derivadas.

Un saludo