guardar estructuras en ficheros

Iniciado por pedroedlp, 22 Abril 2014, 03:50 AM

0 Miembros y 1 Visitante están viendo este tema.

pedroedlp

Hola, no entiendo porque en mi programa no se me guardan las estructuras cuando pongo esto:

fwrite(&p, sizeof(p), 1, pf);

siendo p una variable de una estructura, les dejo el codigo del programa a medio hacer:

#include <stdio.h>

typedef struct producto{
    int numSerie;
    char nombre[20];
    char descripcion[100];
    int precio;
}producto;

void imprimirMenu(void);
void nuevoProducto(char nombreArchivo[]);
//borrarProducto();
//modificarProducto();
//verProducto(char nombreArchivo[]);

int main(){

    char nombreArchivo[20]="productos.dat";

    int r=0;

    do{

    r=0;
    int opcion;

    system("cls");
    imprimirMenu();

    fflush(stdin);
    scanf("%d", &opcion);

    switch (opcion) {

        case 1: nuevoProducto(nombreArchivo);
                break;
        case 4: verArchivo(nombreArchivo);
                break;



    }

    printf("\n\nOtra? ");
    scanf("%d", &r);



    }while(r==1);


    printf("\n\n");

    return 0;
}


void imprimirMenu(void){

    system("cls");
    printf("----Data Productos----");
    printf("\n\n-Menu:");
    printf("\n1.Nuevo Producto");
    printf("\n2.Modificar Producto");
    printf("\n3.Borrar Producto");
    printf("\n4.Ver Producto");
    printf("\n\nOpcion: ");

}


void nuevoProducto(char nombreArchivo[]){

    FILE *pf;

    if((pf=fopen(nombreArchivo, "ab"))==NULL){
        printf("\nError al abrir el archivo");
        exit (-1);
    }

    producto p;

    system("cls");

    printf("Numero: ");
    fflush(stdin);
    scanf("%d", &p.numSerie);
    printf("\nNombre: ");
    fflush(stdin);
    gets(p.nombre);
    printf("\nDescripcion: ");
    fflush(stdin);
    gets(p.descripcion);
    printf("\nPrecio: ");
    fflush(stdin);
    scanf("%d", &p.precio);

    fwrite(&p, sizeof(p), 1, pf);

    fclose(pf);

}

eferion

A mí si me funciona.

Por cierto, fflush está pensado para buffers de salida, no de entrada... cuidado con eso.

Echa un vistazo al hilo que trata sobre el tema: http://foro.elhacker.net/programacion_cc/lo_que_no_hay_que_hacer_en_cc_nivel_basico-t277729.0.html

pedroedlp

Cita de: eferion en 22 Abril 2014, 09:34 AM
A mí si me funciona.

Por cierto, fflush está pensado para buffers de salida, no de entrada... cuidado con eso.

Echa un vistazo al hilo que trata sobre el tema: http://foro.elhacker.net/programacion_cc/lo_que_no_hay_que_hacer_en_cc_nivel_basico-t277729.0.html

como que te funciona? yo compilo, creo un archivo, abro el archivo creado y me sale todo raro, con codigos null y no se que

rir3760

Cita de: pedroedlp en 22 Abril 2014, 23:54 PMcompilo, creo un archivo, abro el archivo creado y me sale todo raro, con codigos null y no se que
Eso sucede porque estas utilizando fwrite para escribir los datos en el archivo y esa funcion copia los bytes que los constituyen tal cual (como estan en memoria).

Si quieres que se almacenen de forma legible en el archivo debes utilizar las funciones fprintf, fputs, fgetc, etc. Ejemplos sobre el uso de ellas los puedes revisar cortesía del motor de búsqueda de los foros.

Un saludo
C retains the basic philosophy that programmers know what they are doing; it only requires that they state their intentions explicitly.
--
Kernighan & Ritchie, The C programming language

pedroedlp

Cita de: rir3760 en 23 Abril 2014, 02:56 AM
Eso sucede porque estas utilizando fwrite para escribir los datos en el archivo y esa funcion copia los bytes que los constituyen tal cual (como estan en memoria).

Si quieres que se almacenen de forma legible en el archivo debes utilizar las funciones fprintf, fputs, fgetc, etc. Ejemplos sobre el uso de ellas los puedes revisar cortesía del motor de búsqueda de los foros.

Un saludo

y asi no me deberia andar?

#include <stdio.h>

struct moto{

    int precio, km;
    char *marca;
    char *descripcion;

};


int main(){

    FILE *pf;

    pf=fopen("datos.dat", "w+b");

    struct moto moto1;

    struct moto moto2;

    moto1.precio=1500;
    moto1.km=0;
    moto1.marca="Yamaha";
    moto1.descripcion="La moto es muy linda y rapida";

    fwrite(&moto1, sizeof(moto1), 1, pf);

    fread(&moto2, sizeof(moto2), 1, pf);

    printf("%d\n", moto2.precio);
    printf("%d\n", moto2.km);
    puts(moto2.marca);
    puts(moto2.descripcion);


    fclose(pf);

    return 0;
}

rir3760

Cita de: pedroedlp en 23 Abril 2014, 03:41 AMy asi no me deberia andar?
Habría que saber a que te refieres con "andar".

Si te refieres a almacenar los datos como texto en el archivo eso no sucederá ya que continuas usando la función fwrite.

----

Si te refieres a que el programa correctamente almacene los datos de "moto1" y los lea inmediatamente en "moto2" eso tampoco ya que hay dos errores importantes: los campos "marca" y "descripcion" son punteros y no deberías almacenarlos en el archivo y antes de la lectura debes colocarte al inicio del archivo utilizando la función rewind.

----

Lo mejor por sencillo es utilizar arrays para los campos "marca" y "descripcion". Para imprimir en el archivo utilizas fprintf, mas o menos asi:
struct moto{
   int precio;
   int km;
   char marca[100];
   char descripcion[100];
};

/* ... */

pf = fopen("datos.dat", "w+");
fprintf(pf, "%d\n%d\n", moto1.precio, moto1.km);
fprintf(pf, "%s\n", moto1.marca);
fprintf(pf, "%s\n", moto1.descripcion);


Y para leerlos como texto:
rewind(pf);
fscanf(pf, "%d %d%*c", &moto2.precio, &moto2.km);
fgets(moto2.marca, 100, pf);
fgets(moto2.descripcion, 100, pf);


Un saludo
C retains the basic philosophy that programmers know what they are doing; it only requires that they state their intentions explicitly.
--
Kernighan & Ritchie, The C programming language

pedroedlp