Tema destacado: Entra al canal IRC oficial de #elhacker.net
Autor
|
Tema: Problema en agenda con ficheros (Leído 1,110 veces)
|
|
DickGumshoe
|
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.
|
|
|
|
|
En línea
|
|
|
|
Ferno
Desconectado
Mensajes: 282
|
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);
|
|
|
|
|
En línea
|
|
|
|
|
DickGumshoe
|
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.
|
|
|
|
|
En línea
|
|
|
|
eleon
Desconectado
Mensajes: 56
|
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. 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.
|
|
|
|
« Última modificación: 7 Febrero 2012, 23:29 por eleon »
|
En línea
|
|
|
|
|
DickGumshoe
|
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.
|
|
|
|
|
En línea
|
|
|
|
Ferno
Desconectado
Mensajes: 282
|
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...).
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.
|
|
|
|
|
En línea
|
|
|
|
|
DickGumshoe
|
¿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.
|
|
|
|
|
En línea
|
|
|
|
Ferno
Desconectado
Mensajes: 282
|
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á!
|
|
|
|
|
En línea
|
|
|
|
|
DickGumshoe
|
¿Al cerrar el fichero "se van" los datos obtenidos con ese fgets del programa?
|
|
|
|
|
En línea
|
|
|
|
do-while
Desconectado
Mensajes: 604
Cuando me afeito, recuerdo porque me dejo barba.
|
¡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!
|
|
|
|
|
En línea
|
¡¡¡Feliz año nuevo!!!
|
|
|
|
DickGumshoe
|
¡Muchas gracias, do-while!
Creo que como me lo has explicado es mejor que creando un fichero nuevo para guardar los datos.
|
|
|
|
|
En línea
|
|
|
|
|
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. 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.
|
|
|
|
|
En línea
|
|
|
|
do-while
Desconectado
Mensajes: 604
Cuando me afeito, recuerdo porque me dejo barba.
|
¡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!
|
|
|
|
|
En línea
|
¡¡¡Feliz año nuevo!!!
|
|
|
|
DickGumshoe
|
¡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.
|
|
|
|
|
En línea
|
|
|
|
do-while
Desconectado
Mensajes: 604
Cuando me afeito, recuerdo porque me dejo barba.
|
¡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!
|
|
|
|
|
En línea
|
¡¡¡Feliz año nuevo!!!
|
|
|
|
|