Foro de elhacker.net

Hola.

Haciendo un ejercicio me he encontrado una duda.

Es este:

/*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];
    FILE* fichero;
    int opcion;
 
    fichero = fopen("agenda.dat", "rt");
    if(fichero != NULL)
    {
 
    while(! feof(fichero)) //Leemos los datos que contiene al principio
    {
      fgets(lineas,25,fichero);
    }
    fclose(fichero);
 
    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");
    scanf("%d",&opcion);
 
    switch(opcion)
    {
      case 1:
           //En construcción, lo editaré según las respuestas que obtenga al preguntaros.
           break;
      case 2:
           //En construcción, lo editaré según las respuestas que obtenga al preguntaros.
           break;
      case 3:
           //En construcción, lo editaré según las respuestas que obtenga al preguntaros.
           break;
      default:
              printf("Opcion no valida");
    }
    }
 
 
 
 
    system("pause");
    return 0;
}
 
 

La idea que estoy siguiendo es que primero abro el archivo en modo de lectura. Entonces pongo los casos en los que tiene que leer el archivo, y los casos en los que no. Cierro el archivo.

Después, creo un switch, en el que elijo una de las tres opciones, y ahí planteo cada cosa.

Y, finalmente, abro de nuevo el archivo, pero esta vez para escribir, y meto los datos obtenidos en el switch.

Mi problema es que no sé cómo hacer para que al cerrar el programa y volver a abrirlo sepa que ya he metido un número determinado de fichas (no sé si me explico). Creo que solo es con un do-while en el switch de "añadir datos", sumándole uno a "i", pero no estoy seguro.

También me gustaría saber si veis bien como he planteado el problema, para ver cómo puedo cambiar las cosas.

Muchísimas gracias.

Saludos.

Si la idea del contador es que funcione siempre y cuando mantengas la cuenta al abrir y cerrar el programa, lo que necesitas es algún modo de almacenarlo externamente. Si usas un contador en el programa, en memoria, se borrará ese dato al cerrar el programa.
Ahora bien, puedes usar otro archivo para guardar el dato de la cantidad de fichas (supongo que al hacer este ejercicio sabes algo de manejo de archivos). Entonces, al abrir el programa, deberás leer este archivo que contiene el contador y listo, empiezas de vuelta la cuenta desde donde quedaste la última vez.

Ahora bien, no entiendo exactamente lo que quieres lograr con esto:

 while(! feof(fichero)) //Leemos los datos que contiene al principio
    {
      fgets(lineas,25,fichero);
    }
    fclose(fichero);

Pretendo que mientras el fichero no llegue a su fin, lea todas sus líneas (pero por cómo has reaccionado, me parece que voy a tener que quitarlo, seguramente esté mal...).

No había caído en que debía almacenar el resultado de "i++" en un fichero.

Muchas gracias.

Saludos.

Si con "cerrar al programa" te refieres a darle clic a la "cruz roja" matando el proceso, lógicamente no puedes hacer nada con variables porque se almacenan en la RAM que es una memoria volátil. En este caso debes leer el fichero y comprobar el número de contactos que hay (cada vez que leas un nombre sumas uno al contador hasta llegar al final, asi calcularás el número de elementos ya guardados).

Pero, si lo que quieres decir es después de guardar un contacto, es tan sencillo como un contador... y en el bucle que te va imprimiendo el menú pones como condición de continuación "contador < 100".

Tu código está hecho a base de librerías standard, queda feo que lo destroces con "system("pause")", aunque en general su uso no es correcto. Pon cualquier instrucción de lectura del teclado para pausar el programa.

Saludos.

Muchas gracias a los dos. Ahora me tengo que ir ya, pero ya mañana termino el programa y dejo mi código completo.

Saludos.

Jaja, no reaccioné mal, sólo preguntaba porque no entiendo el por qué de cerrar el archivo justo después de las lecturas.
Quizás dependa de lo que pretendas hacer en las 3 opciones, pero ¿No sería mejor abrir el archivo para lectura y escritura y cerrarlo antes de terminar el programa? Es decir, abrir y cerrar una única vez.

¿Entonces está bien esto?

while(! feof(fichero)) //Leemos los datos que contiene al principio
    {
      fgets(lineas,25,fichero);
    }
    fclose(fichero);

Lo hago por partes (me refiero a lo de abrir y cerrar el fichero para escribir/leer) porque en el manual que estoy siguiendo en la parte que hay antes del ejercicio no viene nada de cómo hacer las dos cosas (aunque he visto que viene justo después, pero bueno).

Ahora me voy a poner a terminar el programa.

Gracias.

Saludos.

Mira, lo que haces con esa parte del code es leer todas las líneas, llegar al final del archivo, y después cerrarlo. Yo no le veo utilidad así como está!

¿Al cerrar el fichero "se van" los datos obtenidos con ese fgets del programa?

¡Buenas!

Supongo que la lectura/escritura en el fichero sera en binario. Si vas aumentando el tamaño del fichero segun vas metiendo datos, lo estaras haciendo de forma que añada bloques del tamaño del struct que utilizas. Por lo tanto, para saber cuantos elementos tienes en un momento dado solo tendras que calcular la longitud del fichero y dividirla por el tamaño del tipo de dato que guarda:

 
FILE *f;
unsigned long posicion, tamanio, ndatos;
 
f = fopen(tu_archivo,"rb");
 
/* las instrucciones que quieras */
 
/* leemos la posicion actual del fichero */
posicion = ftell(f);
 
/* nos situamos al final del fichero */
fseek(f,0,SEEK_END);
 
/* leemos el numero de bytes que tiene el fichero */
tamanio = ftell(f);
 
/* volvemos a dejarlo como estaba */
fseek(f,posicion,SEEK_SET);
 
/* si el fichero solo contiene un tipo de datos esta division siempre sera exacta */
ndados = tamanio / sizeof(tu_struct);
 

¡Saludos!
¡Saludos!

¡Muchas gracias, do-while!

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

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.

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.

¡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

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!

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

Por ahora tengo esto:

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

¡Buenas!

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

           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:

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:

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!

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

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.

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!

Gracias.

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

Saludos.

Ya lo he solucionado.

Era poniendo getchar(); debajo de

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

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

Este es el código que me ha quedado al hacer el programa:

/*  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];
           int dia, mes, ano;
        }datos[100];
        char lineas[500],comprobar[10];
        FILE* fichero;
        int opcion,posicion,tamano,i=0,j,aux;
       int c;
        fichero = fopen("agenda.dat", "r+b");
        if(fichero != NULL)
        {
 
        while(!feof(fichero)) //Leemos los datos que contiene el archivo y los almacenamos en datos[100]
        {
          c=fscanf(fichero,"%s",datos[i].nombre);
          fscanf(fichero,"%s",datos[i].direccion);
          fscanf(fichero,"%d",&datos[i].movil);
          fscanf(fichero,"%s",datos[i].email);
          fscanf(fichero,"%d",&datos[i].dia);
          fscanf(fichero,"%d",&datos[i].mes);
          fscanf(fichero,"%d",&datos[i].ano);
          if(c!=-1)i++; // evita incrementar erroneamente
        }
 
 
       rewind(fichero); 
 
        do
        {
        do
        {
        printf("Fichas guardadas: %i\n",i);
        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\n");
        scanf("%d",&opcion);
        getchar();
 
        switch(opcion)
        {
          case 1:
               printf("Introduce un nombre: ");
               scanf("%s",datos[i].nombre);
 
               printf("Introduce una direccion: ");
 
               scanf("%s",datos[i].direccion);
               printf("Introduce un movil: ");
               scanf("%d",&datos[i].movil);
               while(getchar() != '\n');
               printf("Introduce correo electronico: ");
 
               scanf("%s",datos[i].email);
               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++)
               {
                 printf("%s\n",datos[j].nombre);
               }
 
 
               break;
          case 3:
              printf("Introduce el nombre de la persona de la que desea visualizar los datos:");
             scanf("%s",comprobar);
              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);
 
        printf("\n\nPulse una tecla para continuar");
        getchar();
        return 0;
    }
 

Saludos.

o.O
Exelente, solamente lo que yo talvez agregaria es que a la hora de Agregar un nuevo dato, seria comprobar que no haya mas de 100 datos ya dentro, porque si ya hay 100 e intentas agregar uno nuevo pues dara error, creo que seria lo unico que yo le agregaria.

¡Muchas gracias! Se me olvidó poner eso. Ya lo he añadido.

Saludos.

Un consejo cuando sean registros estructurados es mejor usar una BDD en este caso te recomiendo que uses SQLite, asi tus consultas son mas rápidas y no te complicas tanto con códigos largos y complejos y en si el desarrollo de tu aplicación sera mas rápido y simple.

http://www.sqlite.org/

Dulces Lunas!¡.

Gracias por la recomendación.

He estado mirando la página que me has pasado y otras que he encontrado por Internet, y parece muy buena la librería. Todavía no he empezado a usar librerías no incluidas en el compilador, pero cuando termine de leer el manual que estoy siguiendo y de hacer los ejercicios que vienen en él, empezaré a usar librerías que no vienen en el compilador (entre ellas esta que me has pasado).

Saludos y gracias.