Autor
|
Tema: Problema en agenda con ficheros (Leído 9,513 veces)
|
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: 1.276
¿Habra que sacarla de paseo?
|
¡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
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
|
- Doctor, confundo los números y los colores. - Vaya marrón. - ¿Marrón? ¡Por el culo te la hinco!
|
|
|
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 */ fseek(fichero ,posicion ,SEEK_SET ); i = tamano / sizeof(datos); do { do { printf("1. Introducir datos\n"); printf("2. Visualizar nombres de las fichas existentes\n"); printf("3. Mostrar todos los datos de una persona\n"); switch(opcion) { case 1: printf("Introduce un nombre: "); fgets(datos [i ]. nombre, 10, stdin ); printf("Introduce una direccion: "); fgets(datos [i ]. direccion, 8, 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 ]. mes); printf("Introduce ano de nacimiento: "); scanf("%d",&datos [i ]. ano); i++; break; case 2: for(j=0;j<=i;j++) { } 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); } printf("\n\nPulse una tecla para continuar"); 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: 1.276
¿Habra que sacarla de paseo?
|
¡Buenas! Esta parte del codigo esta mal (bueno, mal no esta, pero no funcionara como tu quieres): fgets(datos [i ]. nombre, 10, stdin );
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{ }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: if(cadena [strlen(cadena ) - 1] != '\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
|
- Doctor, confundo los números y los colores. - Vaya marrón. - ¿Marrón? ¡Por el culo te la hinco!
|
|
|
DickGumshoe
|
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.
|
|
« Última modificación: 10 Febrero 2012, 16:57 pm por DickGumshoe »
|
En línea
|
|
|
|
do-while
Desconectado
Mensajes: 1.276
¿Habra que sacarla de paseo?
|
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!
|
|
|
En línea
|
- Doctor, confundo los números y los colores. - Vaya marrón. - ¿Marrón? ¡Por el culo te la hinco!
|
|
|
DickGumshoe
|
Gracias.
Entonces voy a buscar cuándo dejo información antes de llamar a la función.
Saludos.
|
|
|
En línea
|
|
|
|
DickGumshoe
|
Ya lo he solucionado. Era poniendo getchar(); debajo de printf("1. Introducir datos\n"); printf("2. Visualizar nombres de las fichas existentes\n"); printf("3. Mostrar todos los datos de una persona\n");
(Se me olvidó ese scanf).
|
|
|
En línea
|
|
|
|
do-while
Desconectado
Mensajes: 1.276
¿Habra que sacarla de paseo?
|
¡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!
|
|
|
En línea
|
- Doctor, confundo los números y los colores. - Vaya marrón. - ¿Marrón? ¡Por el culo te la hinco!
|
|
|
|
|