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 ...
(http://img248.imageshack.us/img248/6058/problemafgets.png)
como vemos se repite la ultima linea ... algun consejo ? S2!
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!
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
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 ?
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 (http://foro.elhacker.net/buscador2-t342883.0.html;msg1680098#msg1680098).
Un saludo
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");