Problema en agenda con ficheros

[DickGumshoe]

¡Muchas gracias, do-while!

Creo que como me lo has explicado es mejor que creando un fichero nuevo para guardar los datos.

[DickGumshoe]

Me parece que tengo el programa terminado, a excepción  de un error.

Cuando pido los datos de las personas, "Introduce el nombre" e "Introduce la dirección" salen seguidos, sin que yo pueda introducir el primer dato.

Código (c) [Seleccionar] Expandir

case 1:
           
           printf("Introduce un nombre: ");
           fgets(datos[i].nombre, 10, stdin);
           printf("Introduce una direccion: ");
           fgets(datos[i].direccion, 15, stdin);
           printf("Introduce un movil: ");
           scanf("%d",&datos[i].movil);
           printf("Introduce correo electronico: ");
           fgets(datos[i].email, 50, stdin);
           printf("Introduce dia de nacimiento: ");
           scanf("%d",&datos[i].dia);
           printf("Introduce mes de nacimiento: ");
           scanf("%d",&datos[i].dia);
           printf("Introduce ano de nacimiento: ");
           scanf("%d",&datos[i].dia);
           i++;
           break;

¿Cuál creéis que es mi error?

Gracias.

Saludos.

[do-while]

¡Buenas!

Posiblemente hayas usado varios scanf antes de la llamada a tu funcion.

scanf deja siempre, al menos, un caracter '\n' en el buffer de entrada, y fgets lee hasta encontrar un salto de linea (que extrae del buffer de entrada). Por lo tanto lo que te pasa es que fgets esta leyendo los datos que scanf ha dejado en el buffer de entrada y por eso pasa a leer directamente el segundo dato.

Para que no te pase esto tendras que sacar toda la "basura" que te deje scanf en el buffer de entrada caracter a caracter. Incluye este codigo despues de cada scanf

Código (c) [Seleccionar] Expandir


while(getchar() != '\n');


Otros utilizan la formula fgets + sscanf.

Yo personalmente prefieco la de getchar(), ya que no estoy dependiendo de tamaños concretos de cadenas para pasar como argumento a fgets. Pero para gustos colores.

¡Saludos!

[DickGumshoe]

¡Muchas gracias! Ya lo he añadido a mi código.

Por ahora tengo esto:

Código (c) [Seleccionar] Expandir

/*Una agenda que maneje los siguientes datos: nombre, dirección, tlf móvil, email, y día,
mes y año de nacimiento (estos tres últimos datos deberán ser números enteros
cortos). Deberá tener capacidad para 100 fichas. Se deberá poder añadir un dato
nuevo, visualizar los nombres de las fichas existentes, o mostrar todos los datos de una
persona (se preguntará al usuario cual es el nombre de esa persona que quiere
visualizar). Al empezar el programa, leerá los datos de un fichero llamado "agenda.dat"
(si existe). Al terminar, guardará todos los datos en ese fichero.*/

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main()
{
    struct
    {
      char nombre[10];
      char direccion[15];
      int movil;
      char email[20];
      short int dia, mes, ano;
    }datos[100];
    char lineas[500],comprobar[10];
    FILE* fichero;
    int opcion,posicion,tamano,i,j,aux;
   
    fichero = fopen("agenda.dat", "r+t");
    if(fichero != NULL)
    {
   
    while(! feof(fichero)) //Leemos los datos que contiene al principio
    {
      fgets(lineas,25,fichero);
    }
   
    /*posicion actual*/
    posicion = ftell(fichero);

    /* final del fichero */
    fseek(fichero,0,SEEK_END);

    /* total de lineas */
    tamano = ftell(fichero);

   
    fseek(fichero,posicion,SEEK_SET);

    i = tamano / sizeof(datos);
   
    do
    {
    do
    {
    printf("Elija una opcion\n");
    printf("1. Introducir datos\n");
    printf("2. Visualizar nombres de las fichas existentes\n");
    printf("3. Mostrar todos los datos de una persona\n");
    printf("4. Salir");
    scanf("%d",&opcion);
   
    switch(opcion)
    {
      case 1:
           
           printf("Introduce un nombre: ");
           fgets(datos[i].nombre, 10, stdin);
           while(getchar() != '\n');
           printf("Introduce una direccion: ");
           fgets(datos[i].direccion, 8, stdin);
           printf("Introduce un movil: ");
           scanf("%d",&datos[i].movil);
           while(getchar() != '\n');
           printf("Introduce correo electronico: ");
           fgets(datos[i].email, 50, stdin);
           printf("Introduce dia de nacimiento: ");
           scanf("%d",&datos[i].dia);
           while(getchar() != '\n');
           printf("Introduce mes de nacimiento: ");
           scanf("%d",&datos[i].mes);
           while(getchar() != '\n');
           printf("Introduce ano de nacimiento: ");
           scanf("%d",&datos[i].ano);
           i++;
           break;
      case 2:
           for(j=0;j<=i;j++)
           {
             puts(datos[j].nombre);
           }
             
             
           break;
      case 3:
          printf("Introduce el nombre de la persona de la que desea visualizar los datos:");
          fgets(comprobar, 10, stdin);
         
          for(j=0;j<=i;j++)
          {
            if(strcmp(datos[j].nombre,comprobar)==0)
            {
              printf("Nombre: %s\n",datos[j].nombre);
              printf("Direccion: %s\n",datos[j].direccion);
              printf("Movil: %d\n",datos[j].movil);
              printf("Email: %s\n",datos[j].email);
              printf("Dia de nacimiento: %d\n",datos[j].dia);
              printf("Mes de nacimiento: %d\n",datos[j].mes);
              printf("Ano de nacimiento: %d\n",datos[j].ano);
            }
          }
           break;
      case 4:
           break;
      default:
              printf("Opcion no valida!\n");
    }
   
    }while(opcion<1||opcion>4);
    }while(opcion!=4);
    }
    aux=i;
    for(i=0;i<=aux;i++)
    {
      fprintf(fichero, "%s\n", datos[i].nombre);
      fprintf(fichero, "%s\n", datos[i].direccion);
      fprintf(fichero, "%d\n", datos[i].movil);
      fprintf(fichero, "%s\n", datos[i].email);
      fprintf(fichero, "%d\n", datos[i].dia);
      fprintf(fichero, "%d\n", datos[i].mes);
      fprintf(fichero, "%d\n", datos[i].ano);
      }
   fclose(fichero);
   
    getchar();
    printf("\n\nPulse una tecla para continuar");
    getchar();
    return 0;
}


¿Podríais decirme si voy bien? Ya he retirado del código "system("pause");", y lo he sustituido por getchar.

Gracias.

Saludos.

[do-while]

¡Buenas!

Esta parte del codigo esta mal (bueno, mal no esta, pero no funcionara como tu quieres):

Código (c) [Seleccionar] Expandir


           fgets(datos[i].nombre, 10, stdin);
           while(getchar() != '\n');


Te he dicho que lo de quitar caracteres de la entrada tienes que hacerlo despues de los scanf. Solo funcionara cuando la cadena introducida (contando el intro que pulses) tenga mas de 9 caracteres, ya que entonces al menos quedara un caracter en la entrada. En caso contrario, leera la cadena que introduzcas y se quedara esperando a que introduzcas mas datos para poder leerlos con getchar, por lo que perderas datos que te interesaria guardar, o a una mala, tendras que pulsar intro para que pase al siguiente mensaje solicitando datos.

Como te he dicho antes, fgets tambien extrae el caracter '\n' de la entrada, por lo tanto puedes controlar que se ha vaciado el buffer de entrada de la siguiente manera:

Código (c) [Seleccionar] Expandir


do{
    fgets(cadena,10,stdin);
}while(cadena[strlen(cadena) - 1] != '\n');

Asi te aseguras de que con fgets estas leyendo todos los datos de la entrada.

Otra alternativa seria la siguiente:

Código (c) [Seleccionar] Expandir


fgets(cadena,10,stdin);

if(cadena[strlen(cadena) - 1] != '\n')
    while(getchar() != '\n');


Pero lo que te he indicado en el post anterior solo deberias utilizarlo despues de un scanf.

Otra cosa, veo que estas abriendo el fichero en modo "r+t", no se si es bueno o te puede dar datos extraños el hecho de mezclar lectura/escritura en binario al abrir un fichero en modo texto. Cuando utilices structs de un tamaño fijo, intenta utilizar siempre acceso binario a los ficheros. Asi seguro que no te llevas sorpresas con feof().

¡Saludos!

[DickGumshoe]

He puesto
while(getchar()!='\n');
detrás de cuando pido

Código (c) [Seleccionar] Expandir

datos[i].nombre porque sino pasa directamente al segundo dato. Es decir, en la consola, me sale Introduce un nombre: Introduce una direccion:

Y ahí se detiene para que introduzca la dirección.

Ya he editado el código para abrirlo en binario en vez de en fichero de texto.

Gracias.

Saludos.

[do-while]

Y yo ya te he dicho en el primer post que he escrito que si te salen los dos mensajes seguidos es porque en alguna parte antes de llamar a tu funcion, has dejado informacion en la entrada. Tambien te he dicho que el codigo que te he dado tienes que usarlo despues de scanf, asi como que si pones despues de fgets los gechar no podras leer los datos de forma correcta (o si, pero sera una tombola).

Tambien te he corregido el hecho de que si pones despues del fgets los getchar, estaras leyendo informacion que no estaras guardando, asi que si te saltan los dos mensajes uno detras de otro poco arreglas leyendo caracteres que no guardas en ninguna parte, lo que estas haciendo es perder datos.

¡Saludos!

[DickGumshoe]

Gracias.

Entonces voy a buscar cuándo dejo información antes de llamar a la función.

Saludos.

[DickGumshoe]

Ya lo he solucionado.

Era poniendo getchar(); debajo de

Código (c) [Seleccionar] Expandir

printf("Elija una opcion\n");
    printf("1. Introducir datos\n");
    printf("2. Visualizar nombres de las fichas existentes\n");
    printf("3. Mostrar todos los datos de una persona\n");
    printf("4. Salir");
    scanf("%d",&opcion);
    getchar();

(Se me olvidó ese scanf).

[do-while]

¡Buenas!

Me alegro de que hayas encontrado el scanf. Ahora recuerda siempre que scanf deja siempre al menos un caracter '\n'. Por lo tanto toma como costumbre terminar de leer el input.

¡Saludos!