Problema con ficheros

[mikelalos]

Buenas, tengo una especie de agenda hecha en c y me gustaria guardar los datos en un fichero que luego pueda leer. El codigo que tengo es este:

Código C:

[QUOTE]#include <stdio.h>
#include <stdlib.h>
#include "agenda.h"
int main(){
  FILE *f;
  int opcion, i=0;
  struct agenda ag;
  inicializar_agenda(&ag);
  do{
  printf("\n1.-Añadir contacto");
  printf("\n2.-Buscar contacto");
  printf("\n3.-Mostrar todos los contactos");
  printf("\n4.-Abrir agenda guardada");
  printf("\n5.-Salir");
  printf("\n¿que quiere hacer? ");
  scanf("%d", &opcion);
  switch(opcion){
  case 1:
    [B]f = fopen ("agenda.bin", "wb");
    if (f == NULL) {
     printf ("Error abriendo alumnos.dat\n") ; 
     return 1 ;
     }
    anyadir_contacto(&ag);
    fwrite (&ag, sizeof(ag), 1, f);
    fclose(f);[/B]
  break;
  case 2:
    if (esta_vacia(ag))
      printf("La agenda esta vacia");
    else
      mostrar_contacto(ag);
  break;
  case 3:
    listar(ag);
    break;
  case 4:
    [B]f = fopen ("agenda.bin", "rb");
    while (fread(&ag, sizeof(ag), 1, f) > 0)
      for(i=0;i<ag.numContactos;i++)
      fprintf(f,"%s %d %s %s\n", ag.conjunto[i].nombre, ag.conjunto[i].telefono, ag.conjunto[i].email, ag.conjunto[i].web) ;[/B]
  }
  }while(opcion!=5);
  return 0;   
}
    [/QUOTE]
 

El fichero se crea en el caso 1, pero luego no lo saca por pantalla en el caso 4 ¿Que puede estar mal?

[sam90]

PRobaste abrir el archivo si escribe o no?? el archivo se crea??

Podemos poner la funcion anyadir_contacto. Creeria que el error viene de ahi.

[vnvnation]

· r abre un fichero para lectura. Si el fichero no existe devuelve error.
· w abre un fichero para escritura. Si el fichero no existe se crea, si el
fichero existe se destruye y se crea uno nuevo.
· a abre un fichero para añadir datos al final del mismo. Si no existe
se crea.
· + símbolo utilizado para abrir el fichero para lectura y escritura.
· b el fichero es de tipo binario.
· t el fichero es de tipo texto. Si no se pone ni b ni t el fichero es de
texto. Los modos anteriores se combinan para conseguir abrir
el fichero en el modo adecuado.

Yo meteria mejor esto si no a cada rato borras el contenido de tu archivo te dejo las aperturas de archivos

Código C++:

case 1:
    [B]f = fopen ("agenda.bin", "ab");
    if (f == NULL) {
     printf ("Error abriendo alumnos.dat\n") ; 
     return 1 ;
     }
    anyadir_contacto(&ag);
    fwrite (&ag, sizeof(ag), 1, f);
    fclose(f);[/B]
  break;
 

[mikelalos]

Iniciado por sam90

PRobaste abrir el archivo si escribe o no?? el archivo se crea??

Podemos poner la funcion anyadir_contacto. Creeria que el error viene de ahi.

La funcion funciona bien, no creo que esté ahi el problema porque el archivo se crea y se escribe en el. El problema es cuando le doy la orden de sacarlo por pantalla que no hace absolutamente nada.

Iniciado por vnvnation

· r abre un fichero para lectura. Si el fichero no existe devuelve error.
· w abre un fichero para escritura. Si el fichero no existe se crea, si el
fichero existe se destruye y se crea uno nuevo.
· a abre un fichero para añadir datos al final del mismo. Si no existe
se crea.
· + símbolo utilizado para abrir el fichero para lectura y escritura.
· b el fichero es de tipo binario.
· t el fichero es de tipo texto. Si no se pone ni b ni t el fichero es de
texto. Los modos anteriores se combinan para conseguir abrir
el fichero en el modo adecuado.

Yo meteria mejor esto si no a cada rato borras el contenido de tu archivo te dejo las aperturas de archivos

Código C++:

case 1:
    [B]f = fopen ("agenda.bin", "ab");
    if (f == NULL) {
     printf ("Error abriendo alumnos.dat\n") ; 
     return 1 ;
     }
    anyadir_contacto(&ag);
    fwrite (&ag, sizeof(ag), 1, f);
    fclose(f);[/B]
  break;
 

Gracias por el consejo, ya lo he cambiado.

[Munire]

Iniciado por mikelalos

El fichero se crea en el caso 1, pero luego no lo saca por pantalla en el caso 4 ¿Que puede estar mal?

el error es que has utilizado fprintf() que sirve para ESCRIBIR en el fichero, en vez de utilizar fscanf() para LEER del fichero. Luego tienes que usar un printf() para escribir por pantalla lo que has leido.

[mikelalos]

Iniciado por Munire

el error es que has utilizado fprintf() que sirve para ESCRIBIR en el fichero, en vez de utilizar fscanf() para LEER del fichero. Luego tienes que usar un printf() para escribir por pantalla lo que has leido.

Lo he hecho como tu dices y lo que hace ahora es sacar un 0 por pantalla

Código C:

f = fopen ("agenda.bin", "rb");
    while (fread(&ag, sizeof(ag), 1, f) > 0){
      for(i=0;i<ag.numContactos;i++)
      fscanf(f,"%s %d %s %s\n", ag.conjunto[i].nombre, &ag.conjunto[i].telefono, ag.conjunto[i].email, ag.conjunto[i].web) ;
      printf("%s %d %s %s\n", ag.conjunto[i].nombre, ag.conjunto[i].telefono, ag.conjunto[i].email, ag.conjunto[i].web) ;
    }
 

[Munire]

Código C:

for(i=0;i<ag.numContactos;i++)
      fscanf(f,"%s %d %s %s\n", ag.conjunto[i].nombre, &ag.conjunto[i].telefono, ag.conjunto[i].email, ag.conjunto[i].web) ;
      printf("%s %d %s %s\n", ag.conjunto[i].nombre, ag.conjunto[i].telefono, ag.conjunto[i].email, ag.conjunto[i].web) ;
 

Mucho cuidado con no poner llaves por que es engañoso. parece que el for abarca hasta el printf pero NO ES ASI.

Lo que has escrito es igual a esto:

Código C:

for(i=0;i<ag.numContactos;i++)
{
    fscanf(f,"%s %d %s %s\n", ag.conjunto[i].nombre, &ag.conjunto[i].telefono, ag.conjunto[i].email, ag.conjunto[i].web) ;
}
printf("%s %d %s %s\n", ag.conjunto[i].nombre, ag.conjunto[i].telefono, ag.conjunto[i].email, ag.conjunto[i].web) ;
 

el no poner llaves (para for, if...) solo se puede hacer si vas a poner una sola sentencia.

Por ejemplo este sencillo programa:

Código C:

int main()
{
    int i=1;
    if(i==0)
        printf("#####");
        printf("#");
 
    system("pause");
    return 0;
}
 

la salida es: #

[mikelalos]

Ya lo tengo solucionado

[Munire]

de nada

[mikelalos]

Iniciado por Munire

de nada

Perdona, he escrito sin leer tu mensaje.

Gracias a todos los que me habeis aportado algo.

[Munire]

entonces era eso lo que pasaba (lo que he escrito mi ultimo mensaje)?

[mikelalos]

Iniciado por Munire

entonces era eso lo que pasaba (lo que he escrito mi ultimo mensaje)?

Aparte de eso estaba utilizando instrucciones para trabajar con ficheros de texto y el fichero es binario.