Ayuda con fprintf y fgets [Solucionado]

Iniciado por Luchoz95, 21 Abril 2013, 06:21 AM

0 Miembros y 1 Visitante están viendo este tema.

Luchoz95

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!

Luchoz95

Lo solucionee gente !! hice lo siguiente
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:
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
C retains the basic philosophy that programmers know what they are doing; it only requires that they state their intentions explicitly.
--
Kernighan & Ritchie, The C programming language

Luchoz95

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


CitarNombre1          Apellido1          Telefono1
Nombre2          Apellido2          Telefono2         
\n       (Invisible)

Porque utilizando esto :
while (fgets(cadena, MAX_CHARS, miarchivo) != NULL)
   fputs(cadena, stdout);


no se imprime el \n ?

rir3760

Cita de: Luchoz95 en 21 Abril 2013, 07:53 AMPorque utilizando esto :
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:
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
C retains the basic philosophy that programmers know what they are doing; it only requires that they state their intentions explicitly.
--
Kernighan & Ritchie, The C programming language

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ú


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


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

system("pause>nul");