From: Leandro Lucarella Date: Sun, 11 Apr 2004 00:37:34 +0000 (+0000) Subject: - Se cambia el prototipo de la función grabar_registro() para que acepte como X-Git-Tag: svn_import_r684~567 X-Git-Url: https://git.llucax.com/z.facultad/75.06/emufs.git/commitdiff_plain/a80376437ca7e774f41a43f33775f2c0b5b662a2?ds=inline - Se cambia el prototipo de la función grabar_registro() para que acepte como último parámetro un puntero a int para guardar un código de error. (actualice todo para que siga compilando, es transparente si no lo usan) - Implemento casi todo tipo1_grabar_registro(). Falta hacer get_id() para empezar a probarlo. --- diff --git a/emufs/emufs.h b/emufs/emufs.h index 882453c..6e3fb83 100644 --- a/emufs/emufs.h +++ b/emufs/emufs.h @@ -98,7 +98,7 @@ typedef struct _emu_fs_t { EMUFS_REG_SIZE tam_reg; /**< Tamaño de registro. 0 Si son registros variables */ void* (*leer_bloque)(struct _emu_fs_t*, EMUFS_BLOCK_ID, int*); /**< Método para leer un bloque */ void* (*leer_registro)(struct _emu_fs_t*, EMUFS_REG_ID, int*); /**< Método para leer un registro */ - EMUFS_REG_ID (*grabar_registro)(struct _emu_fs_t*, void*, EMUFS_REG_SIZE); /**< Método para grabar un registro */ + EMUFS_REG_ID (*grabar_registro)(struct _emu_fs_t*, void*, EMUFS_REG_SIZE, int*); /**< Método para grabar un registro */ int (*borrar_registro)(struct _emu_fs_t*, EMUFS_REG_ID); /**< Método para borrar un registro */ char *nombre; /**< Nombre del archivo */ } EMUFS; diff --git a/emufs/idx.c b/emufs/idx.c index 70508c4..c8babf4 100644 --- a/emufs/idx.c +++ b/emufs/idx.c @@ -180,7 +180,7 @@ int emufs_idx_borrar(EMUFS *emu, EMUFS_REG_ID n_IdReg) fseek(f_idx,final,SEEK_SET); fread(buffer,sizeof(EMUFS_IDX),cant-1,f_idx) ; for( i=0; i #include +#define PERR(msg) printf("%s:%d> %s.\n",__FILE__, __LINE__, msg); + +/*------------------ Funciones privadas ----------------------*/ + +int emufs_tipo1_header_jump(FILE*); + +size_t emufs_tipo1_header_size(void); + +int emufs_tipo1_block_jump(EMUFS*, FILE*, EMUFS_BLOCK_ID); + +void emufs_tipo1_escribir_reg_en_memoria(char* dst, EMUFS_REG_ID reg_id, + EMUFS_REG_SIZE reg_size, char* reg); + +/*------------------ Funciones públicas ----------------------*/ + int emufs_tipo1_inicializar(EMUFS* efs) { /* Asigna punteros a funciones. */ @@ -64,17 +79,16 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id, int *err) EMUFS_REG_SIZE curr_reg_size; /* tamaño del registro leído secuencialmente */ EMUFS_REG_ID curr_reg_id; /* id del registro leído secuencialmente */ - *err = 0; block_id = emufs_idx_buscar_registro(efs, reg_id); if (block_id == EMUFS_NOT_FOUND) { /* TODO Manejo de errores */ - printf("Registro no encontrado.\n"); - *err = (int) EMUFS_NOT_FOUND; + PERR("Registro no encontrado"); + *err = EMUFS_NOT_FOUND; return NULL; } - if (!(block = emufs_tipo1_leer_bloque(efs, block_id, err))) { + if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, err))) { /* TODO Manejo de errores */ - printf("no se pudo leer el bloque\n"); + PERR("no se pudo leer el bloque"); return NULL; } @@ -92,7 +106,7 @@ void* emufs_tipo1_leer_registro(EMUFS* efs, EMUFS_REG_ID reg_id, int *err) if (registro == NULL) { /* TODO Manejo de errores */ free(block); - printf("No hay memoria.\n"); + PERR("No hay memoria"); *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */ return NULL; } @@ -117,26 +131,24 @@ void* emufs_tipo1_leer_bloque(EMUFS* efs, EMUFS_BLOCK_ID block_id, int *err) strcat(name_f,".dat"); if ((file = fopen(name_f, "r")) == NULL) { + PERR("No se puede abrir archivo"); *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */ return NULL; /* FIXME ERROR */ } - fseek(file, - sizeof(EMUFS_TYPE) + /* Cabecera tipo */ - sizeof(EMUFS_BLOCK_SIZE), /* Cabecera tamaño de bloque */ - SEEK_SET); + emufs_tipo1_header_jump(file); /* salta cabeceras */ + emufs_tipo1_block_jump(efs, file, block_id); /* salta bloques */ /* FIXME: verificar que no se pase de fin de archivo*/ - fseek(file, block_id * efs->tam_bloque, SEEK_CUR); block = (char*) malloc(efs->tam_bloque); if (block == NULL) { /* TODO Manejo de errores */ - printf("No hay memoria.\n"); + PERR("No hay memoria"); *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */ return NULL; } if (fread(block, efs->tam_bloque, 1, file) != 1) { /* TODO Manejo de errores */ free(block); - printf("Error al leer bloque.\n"); + PERR("Error al leer bloque"); *err = 3; /* EMUFS_ERROR_FILE_READ */ return NULL; } @@ -144,16 +156,153 @@ void* emufs_tipo1_leer_bloque(EMUFS* efs, EMUFS_BLOCK_ID block_id, int *err) return block; } -EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg_ptr, - EMUFS_REG_SIZE reg_size) +EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS* efs, void* reg, + EMUFS_REG_SIZE reg_size, int* err) { - return -1; /* FIXME Error */ + EMUFS_REG_ID reg_id; + EMUFS_FREE fs; + EMUFS_BLOCK_ID block_id; + FILE* file; + char name_f[255]; + char* block; + + strcpy(name_f,efs->nombre); + strcat(name_f,".dat"); + + /* me devuelve el ID del bloque donde quepa un registro y el espacio libre en "fs"*/ + block_id = emufs_fsc_buscar_lugar(efs, reg_size, &fs); + /* si no hay bloques con suficiente espacio creo un bloque nuevo */ + if (block_id == EMUFS_NOT_FOUND) { + if ((file = fopen(name_f, "a+b")) == NULL) { + /* TODO Manejo de errores */ + PERR("Error al abrir archivo"); + *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */ + return EMUFS_NOT_FOUND; + } + /* crear un nuevo bloque en memoria */ + block = (char*) malloc(efs->tam_bloque); + if (block == NULL) { + /* TODO Manejo de errores */ + PERR("No hay memoria"); + *err = 2; /* EMUFS_ERROR_OUT_OF_MEMORY */ + fclose(file); + return EMUFS_NOT_FOUND; + } + /* graba el registro al principio del bloque */ + reg_id = emufs_tipo1_get_id(efs); + /* graba registro en bloque */ + emufs_tipo1_escribir_reg_en_memoria(block, reg_id, reg_size, reg); + /* graba el bloque en el archivo */ + block_id = emufs_tipo1_grabar_bloque(efs, block, block_id, err); + if (*err) { + PERR("error al grabar bloque"); + free(block); + return EMUFS_NOT_FOUND; + } + free(block); + fclose(file); + /* grabo el nuevo registro en el archivo de espacios libres */ + *err = emufs_fsc_agregar(efs, block_id, efs->tam_bloque - reg_size + - sizeof(EMUFS_REG_ID) - sizeof(EMUFS_REG_SIZE)); + if (*err) { + PERR("No se pudo agregar fsc"); + return EMUFS_NOT_FOUND; + } + + /* Encontró espacio en un bloque existente, graba registro ahí */ + } else { + /* cargo el bloque en block_id */ + if (!(block = (char*) emufs_tipo1_leer_bloque(efs, block_id, err))) { + /* TODO Manejo de errores */ + PERR("no se pudo leer el bloque"); + return EMUFS_NOT_FOUND; + } + /* inserta el registro en el bloque */ + /* tengo que buscar un ID válido para el nuevo registro */ + reg_id = emufs_tipo1_get_id(efs); + /* graba registro en bloque */ + emufs_tipo1_escribir_reg_en_memoria(block + efs->tam_bloque - fs, + reg_id, reg_size, reg); + /* graba el bloque en el archivo */ + block_id = emufs_tipo1_grabar_bloque(efs, block, block_id, err); + if (*err) { + PERR("error al grabar bloque"); + free(block); + return EMUFS_NOT_FOUND; + } + free(block); + /* actualizo el archivo de espacios libres */ + *err = emufs_fsc_actualizar(efs, block_id, fs - reg_size + - sizeof(EMUFS_REG_ID) - sizeof(EMUFS_REG_SIZE)); + if (*err) { + PERR("No se pudo actualizar fsc"); + return EMUFS_NOT_FOUND; + } + } + + /* actualizo el indice de bloques y registros */ + *err = emufs_idx_agregar(efs, block_id, reg_id); + if (*err){ + PERR("No se pudo agregar idx"); + return EMUFS_NOT_FOUND; + } + + return reg_id; } /*Graba un bloque en el archivo*/ -int emufs_tipo1_grabar_bloque(EMUFS *emu, void *ptr, EMUFS_BLOCK_ID num_bloque) +EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS *efs, void *block, + EMUFS_BLOCK_ID block_id, int* err) { - return -1; /* FIXME Error */ + FILE* file; + char name_f[255]; + + strcpy(name_f,efs->nombre); + strcat(name_f,".dat"); + + if ((file = fopen(name_f, "r+b")) == NULL) { + /* TODO Manejo de errores */ + PERR("Error al abrir archivo"); + *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */ + return EMUFS_NOT_FOUND; + } + /* Si es NOT_FOUND tengo que agregar un bloque al final del archivo */ + if (block_id == EMUFS_NOT_FOUND) { + /* me paro al final del archivo */ + if (fseek(file, 0l, SEEK_END)) { + /* TODO Manejo de errores */ + PERR("No se pudo hacer fseek()"); + fclose(file); + *err = 8; /* EMUFS_ERROR_SEEK_FILE */ + return EMUFS_NOT_FOUND; + } + /* Obtengo ID del bloque nuevo */ + block_id = (ftell(file) - emufs_tipo1_header_size()) / efs->tam_bloque; + /* Si es un ID válido, salto hasta ese bloque. */ + } else { + /* Salta el header del archivo */ + if ((*err = emufs_tipo1_header_jump(file))) { + PERR("no se pudo saltar la cabecera del archivo"); + fclose(file); + return EMUFS_NOT_FOUND; + } + /* Salta bloques */ + if ((*err = emufs_tipo1_block_jump(efs, file, block_id))) { + PERR("no se pudo saltar la cabecera del bloque"); + fclose(file); + return EMUFS_NOT_FOUND; + } + } + /* Grabo el bloque */ + if (fwrite(block, efs->tam_bloque, 1, file) != 1) { + PERR("No se pudo escribir el archivo"); + fclose(file); + *err = 6; /* EMUFS_ERROR_WRITE_FILE */ + return EMUFS_NOT_FOUND; + } + + fclose(file); + return block_id; } /*Busco en el archivo de Id`s un Id valido para un nuevo registro*/ @@ -173,3 +322,41 @@ int emufs_tipo1_borrar_registro(EMUFS *emu, EMUFS_REG_ID id_reg, { return -1; /* FIXME Error */ } + +int emufs_tipo1_header_jump(FILE* fp) +{ + if (fseek(fp, 0l, SEEK_END)) { + PERR("No se pudo hacer fseek()"); + return 8; /* EMUFS_ERROR_SEEK_FILE */ + } + return 0; /* EMUFS_OK */ +} + +int emufs_tipo1_block_jump(EMUFS* efs, FILE* fp, EMUFS_BLOCK_ID block_count) +{ + if (fseek(fp, block_count * efs->tam_bloque, SEEK_CUR)) { + PERR("No se pudo hacer fseek()"); + return 8; /* EMUFS_ERROR_SEEK_FILE */ + } + return 0; /* EMUFS_OK */ +} + +size_t emufs_tipo1_header_size(void) +{ + return sizeof(EMUFS_TYPE) + /* Cabecera de tipo de archivo */ + sizeof(EMUFS_BLOCK_SIZE); /* Cabecera de tamaño del bloque */ +} + +void emufs_tipo1_escribir_reg_en_memoria(char* dst, EMUFS_REG_ID reg_id, + EMUFS_REG_SIZE reg_size, char* reg) { + /* grabo el id en el bloque */ + memcpy(dst, ®_id, sizeof(EMUFS_REG_ID)); + /* incremento puntero de escritura */ + dst += sizeof(EMUFS_REG_ID); + /* grabo el tamaño del registro en el bloque */ + memcpy(dst, ®_size, sizeof(EMUFS_REG_SIZE)); + /* incremento puntero de escritura */ + dst += sizeof(EMUFS_REG_SIZE); + /* grabo el registro en el bloque */ + memcpy(dst, reg, reg_size); +} diff --git a/emufs/tipo1.h b/emufs/tipo1.h index 4bc6a7e..c89daf6 100644 --- a/emufs/tipo1.h +++ b/emufs/tipo1.h @@ -49,10 +49,10 @@ void* emufs_tipo1_leer_registro(EMUFS*, EMUFS_REG_ID, int*); void* emufs_tipo1_leer_bloque(EMUFS*, EMUFS_BLOCK_ID, int*); /** Graba el registro apuntado por \param ptr en el archivo */ -EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS*, void*, EMUFS_REG_SIZE); +EMUFS_REG_ID emufs_tipo1_grabar_registro(EMUFS*, void*, EMUFS_REG_SIZE, int*); /** Graba el bloque apuntado por \param ptr en el archivo */ -int emufs_tipo1_grabar_bloque(EMUFS*, void*, EMUFS_BLOCK_ID); +EMUFS_BLOCK_ID emufs_tipo1_grabar_bloque(EMUFS*, void*, EMUFS_BLOCK_ID, int*); EMUFS_REG_ID emufs_tipo1_get_id(EMUFS*); diff --git a/emufs/tipo2.c b/emufs/tipo2.c index ed8b163..70d889d 100644 --- a/emufs/tipo2.c +++ b/emufs/tipo2.c @@ -44,7 +44,7 @@ /* void *ptr // Puntero al buffer (registro) a guardar */ /* EMUFS_REG_SIZE n_RegSize // Size del reg en cuestion */ /**********************************************************************/ -EMUFS_REG_ID emufs_tipo2_grabar_registro(EMUFS *efs, void *ptr, EMUFS_REG_SIZE n_RegSize) +EMUFS_REG_ID emufs_tipo2_grabar_registro(EMUFS *efs, void *ptr, EMUFS_REG_SIZE n_RegSize, int* err) { EMUFS_REG_ID n_IdReg; EMUFS_FREE n_FreeSpace; diff --git a/emufs/tipo2.h b/emufs/tipo2.h index 3e55b7d..49b8ac0 100644 --- a/emufs/tipo2.h +++ b/emufs/tipo2.h @@ -37,7 +37,7 @@ #include #include "emufs.h" -EMUFS_REG_ID emufs_tipo2_grabar_registro(EMUFS *, void *, EMUFS_REG_SIZE); +EMUFS_REG_ID emufs_tipo2_grabar_registro(EMUFS *, void *, EMUFS_REG_SIZE, int*); int emufs_tipo2_borrar_registro(EMUFS*, EMUFS_REG_ID); EMUFS_REG_ID emufs_tipo2_get_id(EMUFS *); diff --git a/emufs/tipo2_main.c b/emufs/tipo2_main.c index c564106..ba5e615 100644 --- a/emufs/tipo2_main.c +++ b/emufs/tipo2_main.c @@ -45,6 +45,7 @@ int main(int argc, char *argv[]) char g[41]; char h[63]; char i[43]; + int err = 0; strcpy(a, "1234567890"); strcpy(b, "REGISTRO NUMERO 2. ESTE REGISTRO ES MUCHO MAS LARGO QUE EL UNO"); @@ -61,14 +62,14 @@ int main(int argc, char *argv[]) fp = emufs_crear("articulos", T2, 0, 0); /* Grabamos un par de registros */ - n1 = fp->grabar_registro(fp, a, 11); - n2 = fp->grabar_registro(fp, b, 63); - n3 = fp->grabar_registro(fp, c, 13); - n4 = fp->grabar_registro(fp, d, 8); - n5 = fp->grabar_registro(fp, e, 39); - n6 = fp->grabar_registro(fp, f, 9); - n7 = fp->grabar_registro(fp, g, 41); - n8 = fp->grabar_registro(fp, h, 63); + n1 = fp->grabar_registro(fp, a, 11, &err); + n2 = fp->grabar_registro(fp, b, 63, &err); + n3 = fp->grabar_registro(fp, c, 13, &err); + n4 = fp->grabar_registro(fp, d, 8, &err); + n5 = fp->grabar_registro(fp, e, 39, &err); + n6 = fp->grabar_registro(fp, f, 9, &err); + n7 = fp->grabar_registro(fp, g, 41, &err); + n8 = fp->grabar_registro(fp, h, 63, &err); /* Borramos un registro del medio */ printf("tipo2_main.c >> Borrando registro: %lu\n",n4); diff --git a/emufs/tipo3.c b/emufs/tipo3.c index b9dc8b7..6e1822e 100644 --- a/emufs/tipo3.c +++ b/emufs/tipo3.c @@ -116,7 +116,7 @@ void* emufs_tipo3_leer_bloque(EMUFS *emu, EMUFS_REG_ID ID, int* err) return block; } -EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE tam) +EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE tam, int* err) { EMUFS_REG_ID ID_aux; EMUFS_FREE fs; @@ -125,7 +125,6 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t FILE *file; char name_f[255]; char* bloque; - int err = 0; strcpy(name_f,emu->nombre); strcat(name_f,".dat"); @@ -163,7 +162,7 @@ EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE t } } else { /*cargo el bloque en "bloque"*/ - if (!(bloque = emufs_tipo3_leer_bloque(emu, num_bloque, &err))) { + if (!(bloque = emufs_tipo3_leer_bloque(emu, num_bloque, err))) { /* TODO Manejo de errores */ printf("no se pudo leer el bloque\n"); return -1; diff --git a/emufs/tipo3.h b/emufs/tipo3.h index 346c3c2..f17081f 100644 --- a/emufs/tipo3.h +++ b/emufs/tipo3.h @@ -54,7 +54,7 @@ void* emufs_tipo3_leer_registro(EMUFS*, EMUFS_REG_ID, int*); void* emufs_tipo3_leer_bloque(EMUFS*, EMUFS_BLOCK_ID, int*); /** Graba el registro apuntado por \param ptr en el archivo */ -EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE); +EMUFS_REG_ID emufs_tipo3_grabar_registro(EMUFS *emu, void *ptr, EMUFS_REG_SIZE, int*); /** Graba el bloque apuntado por \param ptr en el archivo */ int emufs_tipo3_grabar_bloque(EMUFS *emu, void *ptr, EMUFS_BLOCK_ID num_bloque); diff --git a/emufs/tipo3_main.c b/emufs/tipo3_main.c index 4321534..1ecd370 100644 --- a/emufs/tipo3_main.c +++ b/emufs/tipo3_main.c @@ -51,7 +51,7 @@ int main(int argc, char *argv[]) char h[100]; char i[100]; char* b_ptr; - int err; + int err = 0; if (argc != 2) { printf("Modo de uso : %s tam_bloque\n", argv[0]); @@ -74,16 +74,16 @@ int main(int argc, char *argv[]) fp = emufs_crear("articulos", T3, atoi(argv[1]), 100); - n1 = fp->grabar_registro(fp, a, 100); + n1 = fp->grabar_registro(fp, a, 100, &err); - n2 = fp->grabar_registro(fp, c, 100); + n2 = fp->grabar_registro(fp, c, 100, &err); - n3 = fp->grabar_registro(fp, d, 100); - n4 = fp->grabar_registro(fp, e, 100); - n5 = fp->grabar_registro(fp, f, 100); - n6 = fp->grabar_registro(fp, g, 100); - n7 = fp->grabar_registro(fp, h, 100); - n8 = fp->grabar_registro(fp, i, 100); + n3 = fp->grabar_registro(fp, d, 100, &err); + n4 = fp->grabar_registro(fp, e, 100, &err); + n5 = fp->grabar_registro(fp, f, 100, &err); + n6 = fp->grabar_registro(fp, g, 100, &err); + n7 = fp->grabar_registro(fp, h, 100, &err); + n8 = fp->grabar_registro(fp, i, 100, &err); ver_archivo_FS(fp);