Foro de elhacker.net

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



Porque 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:
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");