Ayuda con fprintf y fgets [Solucionado]

[Luchoz95]

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

Código (c) [Seleccionar] Expandir

#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!

[Luchoz95]

Lo solucionee gente !! hice lo siguiente

Código (c) [Seleccionar] Expandir

void Imprimir_Persona()
{
char cadena[MAX_CHARS];
    miarchivo= fopen(nombrearchivo,"r");
    if(miarchivo==NULL)return -1;
    system("cls");
fgets(cadena,MAX_CHARS,miarchivo);
    while(!feof(miarchivo))
{
fputs(cadena,stdout);
fgets(cadena,MAX_CHARS,miarchivo);
}
printf("\nPresione una tecla para volver al menu ...");
system("pause>nul");


    fclose(miarchivo);
}

S2!

[rir3760]

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:

Código (c) [Seleccionar] Expandir

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:

Código (c) [Seleccionar] Expandir

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

Código (c) [Seleccionar] Expandir

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

Un saludo

[Luchoz95]

Muchas gracias por el consejo rir3760 como siempre, pero hay algo que no entiendo, supongamos que en el archivo de texto tenemos esto ...

Nombre1          Apellido1          Telefono1
Nombre2          Apellido2          Telefono2         
\n       (Invisible)

Porque utilizando esto :

Código (c) [Seleccionar] Expandir

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

no se imprime el \n ?

[rir3760]

Porque utilizando esto :

Código (c) [Seleccionar] Expandir

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

no se imprime el \n ?

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:

Código [Seleccionar] Expandir

Nombre1          Apellido1          Telefono1\n
Nombre2          Apellido2          Telefono2\n       
(nada)
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

[dato000]

Puede que sea porque el archivo no reconoce ese tipo de caracteres, no estoy seguro, pero talvez puede ser porque reconoce ese caracter como null y entienda como el final del archivo.

de todas maneras, porque no simplemente colocas los saltos de linea en la linea de salida del menú

Código (c) [Seleccionar] Expandir


printf("\n\nPresione una tecla para volver al menu ...");


Y se me hizo curioso ver, me hizo recordar mis clases con batch, ingenioso.

Código (c) [Seleccionar] Expandir


system("pause>nul");