* Bugfix en idx_get para saltar los registros no utilizados

[z.facultad/75.06/emufs.git] / emufs / idx.c

diff --git a/emufs/idx.c b/emufs/idx.c
index 5fcec354b81614465ddd16ad098a2371093fb9f4..8b4f969a922b1c2e3df7e99ad6abdb5a6f266202 100644 (file)
--- a/emufs/idx.c
+++ b/emufs/idx.c
@@ -78,27 +78,24 @@ EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)
        strcpy(name_f_idx, emu->nombre);
        strcat(name_f_idx, EMUFS_IDX_EXT);
 
        strcpy(name_f_idx, emu->nombre);
        strcat(name_f_idx, EMUFS_IDX_EXT);
 

-       (*err) = 0;

        if ((f_idx = fopen(name_f_idx, "rb")) == NULL) {
                PERR("No se puede abrir archivo");
                *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
                return EMUFS_NOT_FOUND;
        }
        if ((f_idx = fopen(name_f_idx, "rb")) == NULL) {
                PERR("No se puede abrir archivo");
                *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
                return EMUFS_NOT_FOUND;
        }

-       while (!feof(f_idx)) {
-               /* Me aseguro de leer la cantidad de bytes correcta */
-               if (fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1) {
-                       if (feof(f_idx)) break; /* No leyó por EOF */
-                       PERR("Error al leer registros de idx");
-                       *err = 3; /* EMUFS_ERROR_FILE_READ */
-                       return EMUFS_NOT_FOUND;
-               }
-               if (reg.n_idreg >= max) {
-                       max = reg.n_idreg;
-                       found = 1;
-               }
+
+       /* Voy a la ultima entrada */
+       if (fseek(f_idx, -sizeof(EMUFS_IDX), SEEK_END) != 0) {
+               fclose(f_idx);
+               return 0;
+       }
+       
+       if (fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) == 1) {
+               found = 1;
+               max = reg.id_reg;

        }
        fclose(f_idx);
        }
        fclose(f_idx);

-
+       

        if (found) {
                return ++max;
        } else {
        if (found) {
                return ++max;
        } else {


@@ -122,16 +119,11 @@ EMUFS_BLOCK_ID emufs_idx_buscar_registro(EMUFS *emu, EMUFS_REG_ID reg_id)
                return EMUFS_NOT_FOUND;
        }
 
                return EMUFS_NOT_FOUND;
        }
 

-       while (!feof(f_idx)) {
-               if (fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1) {
-                       if (feof(f_idx)) break; /* No leyó por EOF */
-                       PERR("Error al leer registros de idx");
-                       /*   *err = 3; * EMUFS_ERROR_FILE_READ */
-                       return EMUFS_NOT_FOUND;
-               }
-               if (reg.n_idreg == reg_id) {
+       /* Me muevo a la posicion pedida */
+       if (fseek(f_idx, sizeof(EMUFS_IDX)*reg_id, SEEK_SET) == 0) {
+               if (fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) == 1) {

                        fclose(f_idx);
                        fclose(f_idx);

-                       return reg.n_location;
+                       return reg.location;

                }
        }
        fclose(f_idx);
                }
        }
        fclose(f_idx);


@@ -139,8 +131,7 @@ EMUFS_BLOCK_ID emufs_idx_buscar_registro(EMUFS *emu, EMUFS_REG_ID reg_id)
        return EMUFS_NOT_FOUND;
 }
 
        return EMUFS_NOT_FOUND;
 }
 

-/* agrega un registro al final del archivo */
-int emufs_idx_agregar(EMUFS *emu, EMUFS_REG_ID n_idreg, EMUFS_BLOCK_ID n_location)
+int emufs_idx_agregar(EMUFS *emu, EMUFS_REG_ID id_reg, EMUFS_BLOCK_ID location)

 {
        FILE *f_idx;
        EMUFS_IDX reg;
 {
        FILE *f_idx;
        EMUFS_IDX reg;


@@ -150,12 +141,13 @@ int emufs_idx_agregar(EMUFS *emu, EMUFS_REG_ID n_idreg, EMUFS_BLOCK_ID n_locatio
        strcat(name_f_idx, EMUFS_IDX_EXT);
 
        if ( (f_idx = fopen(name_f_idx,"a+"))==NULL ) return -1;
        strcat(name_f_idx, EMUFS_IDX_EXT);
 
        if ( (f_idx = fopen(name_f_idx,"a+"))==NULL ) return -1;

-               
-    /* Note: Location = Bloque para Tipo 1 y 3, Offset para Tipo 2 */
-    reg.n_idreg = n_idreg;
-       reg.n_location = n_location;
+
+       fseek(f_idx, sizeof(EMUFS_IDX)*id_reg, SEEK_SET); 
+       reg.id_reg = id_reg;
+       reg.location = location;

        fwrite(&reg,sizeof(EMUFS_IDX),1,f_idx); 
        fclose(f_idx);
        fwrite(&reg,sizeof(EMUFS_IDX),1,f_idx); 
        fclose(f_idx);

+

        return 0;
 }
 
        return 0;
 }
 


@@ -165,46 +157,18 @@ int emufs_idx_borrar(EMUFS *emu, EMUFS_REG_ID idreg)
        FILE *f_idx;
        EMUFS_IDX reg;
        char name_f_idx[255];
        FILE *f_idx;
        EMUFS_IDX reg;
        char name_f_idx[255];

-       long actual, final, cant, i, tam, a;

        
        strcpy(name_f_idx,emu->nombre);
        strcat(name_f_idx, EMUFS_IDX_EXT);
 
        
        strcpy(name_f_idx,emu->nombre);
        strcat(name_f_idx, EMUFS_IDX_EXT);
 

-       if ( (f_idx = fopen(name_f_idx,"a+"))==NULL ) return -1;
-               
-       
-       while ( !feof(f_idx) ){
-               /*busco cual tengo que borrar*/
-               if ( fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1 ) continue;
-               if ( reg.n_idreg == idreg )
-                       break;
-       }
+       if ( (f_idx = fopen(name_f_idx,"r+"))==NULL ) return 1;

 
 

-       /* me paro en el que tengo que borrar */
-       fseek(f_idx, -sizeof(EMUFS_IDX), SEEK_CUR);     
-       actual = ftell(f_idx); /* Guardo la posicion actual */
-       printf(" actual  = %ld\n",actual);
-       fseek(f_idx, 0, SEEK_END); /* me voy al final */
-       final = ftell(f_idx); /* veo cuando ocupa el archivo */
-       fseek(f_idx, actual, SEEK_SET);/* vuelvo al lugar desde donde quiero justificar */
-       /*calculo cuantos registros tengo que mover */
-       cant = (final-actual)/sizeof(EMUFS_IDX);
-       /*apunto al siguiente del que quiero borrar*/
-       /*leo todos los que quedan*/
-       for(i=0; i<cant; i++) {
-               final = actual + sizeof(EMUFS_IDX);
-               fseek(f_idx, final, SEEK_SET);
-               fread(&reg, sizeof(EMUFS_IDX), 1, f_idx);
-               fseek(f_idx, actual, SEEK_SET);
+       if (fseek(f_idx, sizeof(EMUFS_IDX)*idreg, SEEK_SET) == 0) {
+               reg.id_reg = idreg;
+               reg.location = EMUFS_NOT_FOUND;

                fwrite(&reg, sizeof(EMUFS_IDX), 1, f_idx);
                fwrite(&reg, sizeof(EMUFS_IDX), 1, f_idx);

-               actual += sizeof(EMUFS_IDX);

        }
        }

-       /* Aca deberia estar al final del archivo , no hago seek */     
-       tam = ftell(f_idx);

        fclose(f_idx);
        fclose(f_idx);

-       if ( truncate(name_f_idx, tam-sizeof(EMUFS_IDX))!= 0) 
-               printf("no pude truncar\n");
-       else printf("TRUNKE!!!!\n");

        
        return 0;
 }
        
        return 0;
 }


@@ -215,12 +179,101 @@ EMUFS_REG_ID emufs_idx_get_new_id(EMUFS* efs, int* err)
 
        id = emufs_did_get_last(efs, err);
        if (id == EMUFS_NOT_FOUND) {
 
        id = emufs_did_get_last(efs, err);
        if (id == EMUFS_NOT_FOUND) {

+               if (*err) {
+                       PERR("error al obtener ultimo id");
+                       return id;
+               }

                id = emufs_idx_buscar_mayor_id_libre(efs, err);
                if (*err) {
                        PERR("error al obtener id mayor");
                id = emufs_idx_buscar_mayor_id_libre(efs, err);
                if (*err) {
                        PERR("error al obtener id mayor");

-                       return id;
+                       return EMUFS_NOT_FOUND;

                }
        }
                }
        }

+

        return id;      
 }
 
        return id;      
 }
 

+EMUFS_REG_ID *emufs_idx_get(EMUFS *emu, EMUFS_REG_ID *cant)
+{
+       FILE *f_idx;
+       int count;
+       char name_f_idx[255];
+       EMUFS_REG_ID *tmp;
+       EMUFS_IDX reg;
+       
+       strcpy(name_f_idx,emu->nombre);
+       strcat(name_f_idx, EMUFS_IDX_EXT);
+
+       if ( (f_idx = fopen(name_f_idx, "rb"))==NULL){
+               PERR("No se pudo abrir el archvo");
+               (*cant) = EMUFS_NOT_FOUND;
+               return NULL;
+       }
+
+       tmp = NULL;
+       count = 0;
+       while (!feof(f_idx)) {
+               if (fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1) continue;
+               /* TODO : Verificar errores :-D */
+               if (reg.location != EMUFS_NOT_FOUND) {
+                       count++;
+                       tmp = realloc(tmp, count);
+                       tmp[count-1] = reg.id_reg;
+                       fprintf(stderr, "Nuevo registro de id = %lu\n", reg.id_reg);
+               }
+       }
+       fclose(f_idx);
+
+       (*cant) = count;
+       fprintf(stderr, "Mando %d registros\n", count);
+       return tmp;
+}
+
+int emufs_idx_existe_id(EMUFS *emu, int ID)
+{
+       FILE *f_idx;
+       char name_f_idx[255];
+       EMUFS_IDX reg;
+       
+       strcpy(name_f_idx,emu->nombre);
+       strcat(name_f_idx, EMUFS_IDX_EXT);
+
+       if ( (f_idx = fopen(name_f_idx, "rb")) == NULL){
+               PERR("No se pudo abrir el archivo");
+               return -1;
+       }
+       
+       if (fseek(f_idx, sizeof(EMUFS_IDX)*ID, SEEK_SET) == 0) {
+               fread(&reg, sizeof(EMUFS_IDX), 1, f_idx);
+               if (reg.location != EMUFS_NOT_FOUND) {
+                       fclose(f_idx);
+                       return 0;
+               }
+       }
+       fclose(f_idx);
+       return -1;/*no existe el id*/
+}
+
+int emufs_idx_actualizar(EMUFS *emu, int ID, EMUFS_BLOCK_ID bloque)
+{
+       FILE *f_idx;
+       char name_f_idx[255];
+       EMUFS_IDX reg;
+       
+       strcpy(name_f_idx,emu->nombre);
+       strcat(name_f_idx, EMUFS_IDX_EXT);
+
+       if ( (f_idx = fopen(name_f_idx, "r+")) == NULL){
+               PERR("No se pudo abrir el archivo");
+               return -1;
+       }
+       printf("actualice el id = %d\n",ID);
+       fseek(f_idx,0,SEEK_SET);
+       fseek(f_idx,ID*sizeof(EMUFS_IDX),SEEK_SET);
+       fread(&reg,sizeof(EMUFS_IDX),1,f_idx);
+       reg.location = bloque;
+       fseek(f_idx,-sizeof(EMUFS_IDX),SEEK_CUR);
+       fwrite(&reg,sizeof(EMUFS_IDX),1,f_idx);
+       fclose(f_idx);
+       return 0;
+}