* BUGFIX : en idx_borrar se estaba abriendo el archivo como "a+" en lugar

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

diff --git a/emufs/idx.c b/emufs/idx.c
index d0dd436da62f06cf3da17f4bda83fd94b6f7d3d0..6cab37cd9c1e1c3b4d59007cd2f9ee0c5931e71e 100644 (file)
--- a/emufs/idx.c
+++ b/emufs/idx.c
@@ -36,6 +36,7 @@
  */
 
 #include "idx.h"
  */
 
 #include "idx.h"

+#include "did.h"

 #include <stdlib.h>
 #include <strings.h>
 #include <unistd.h>
 #include <stdlib.h>
 #include <strings.h>
 #include <unistd.h>


@@ -66,61 +67,80 @@ int emufs_idx_crear(EMUFS *efs)
 }
 
 /* Devuelve el mayor id de registro utilizado so far en el archivo de datos, revisando el indice. */
 }
 
 /* Devuelve el mayor id de registro utilizado so far en el archivo de datos, revisando el indice. */

-EMUFS_REG_ID emufs_idx_buscar_mayor_id(EMUFS *emu)
+EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)

 {
 {

-       EMUFS_REG_ID n_IdReg, max = 0;
-       FILE *f_idx;    
+       EMUFS_REG_ID max = 0;
+       FILE* f_idx;    

        EMUFS_IDX reg;
        char name_f_idx[255]; /* TODO usar malloc para no limitar el tamaño de nombre de archivo */
        EMUFS_IDX reg;
        char name_f_idx[255]; /* TODO usar malloc para no limitar el tamaño de nombre de archivo */

-       unsigned short int b_Found = 0;
+       int found = 0;

 
 

-       strcpy(name_f_idx,emu->nombre);
+       strcpy(name_f_idx, emu->nombre);

        strcat(name_f_idx, EMUFS_IDX_EXT);
 
        strcat(name_f_idx, EMUFS_IDX_EXT);
 

-       if ( (f_idx = fopen(name_f_idx,"r")) == NULL) return -1; /*ERROR*/
-       n_IdReg = -1;
-       while ( !feof(f_idx) ){
+       (*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;
+       }
+       while (!feof(f_idx)) {

                /* Me aseguro de leer la cantidad de bytes correcta */
                /* Me aseguro de leer la cantidad de bytes correcta */

-               if (fread(&reg,sizeof(EMUFS_IDX),1,f_idx) != 1) continue;
-               if ( reg.n_IdReg >= max ) {
-                       max = reg.n_IdReg;
-                       b_Found = 1;
+               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;

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

-       
-       if (!b_Found) return (0);
-       else return(max+1);
-       return n_IdReg; 
+
+       if (found) {
+               return ++max;
+       } else {
+               return 0;
+       }

 }
 
 /* busca el registro ID en el archivo ".idx" y devuelve el nro de bloque en el que se encuentra */
 }
 
 /* busca el registro ID en el archivo ".idx" y devuelve el nro de bloque en el que se encuentra */

-EMUFS_BLOCK_ID emufs_idx_buscar_registro(EMUFS *emu, EMUFS_REG_ID n_IdReg)
+EMUFS_BLOCK_ID emufs_idx_buscar_registro(EMUFS *emu, EMUFS_REG_ID reg_id)

 {
        FILE* f_idx;
        EMUFS_IDX reg;
        char name_f_idx[255];
 {
        FILE* f_idx;
        EMUFS_IDX reg;
        char name_f_idx[255];

-       unsigned short int b_Found = 0;

        
        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,"r")) == NULL) return -1; /*ERROR*/
+       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) && !b_Found){
-               if (fread(&reg,sizeof(EMUFS_IDX),1,f_idx) != 1) continue;
-               if (reg.n_IdReg == n_IdReg) b_Found = 1;
+       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) {
+                       fclose(f_idx);
+                       return reg.n_location;
+               }

        }
        }

-       

        fclose(f_idx);
        fclose(f_idx);

-       
-       /* Sino lo encontre devuelvo uno, otherwise el offset o bloque */
-       if (!b_Found) return(-1);
-       else return(reg.n_Location);
+
+       return EMUFS_NOT_FOUND;

 }
 
 /* agrega un registro al final del archivo */
 }
 
 /* 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 n_idreg, EMUFS_BLOCK_ID n_location)

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


@@ -132,62 +152,72 @@ int emufs_idx_agregar(EMUFS *emu, EMUFS_REG_ID n_IdReg, EMUFS_BLOCK_ID n_Locatio
        if ( (f_idx = fopen(name_f_idx,"a+"))==NULL ) return -1;
                
     /* Note: Location = Bloque para Tipo 1 y 3, Offset para Tipo 2 */
        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;    
+    reg.n_idreg = n_idreg;
+       reg.n_location = n_location;

        fwrite(&reg,sizeof(EMUFS_IDX),1,f_idx); 
        fclose(f_idx);
        return 0;
 }
 
 /* Borra un registro del indice dada la eliminacion fisica de un reg */
        fwrite(&reg,sizeof(EMUFS_IDX),1,f_idx); 
        fclose(f_idx);
        return 0;
 }
 
 /* Borra un registro del indice dada la eliminacion fisica de un reg */

-int emufs_idx_borrar(EMUFS *emu, EMUFS_REG_ID n_IdReg)
+int emufs_idx_borrar(EMUFS *emu, EMUFS_REG_ID idreg)

 {
        FILE *f_idx;
 {
        FILE *f_idx;

-       EMUFS_IDX reg, buffer;
+       EMUFS_IDX reg;

        char name_f_idx[255];
        char name_f_idx[255];

-       long actual, final, cant, i, tam;
+       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;
-       
+       if ( (f_idx = fopen(name_f_idx,"r+"))==NULL ) return -1;
+

        while ( !feof(f_idx) ){
                /*busco cual tengo que borrar*/
                if ( fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1 ) continue;
        while ( !feof(f_idx) ){
                /*busco cual tengo que borrar*/
                if ( fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1 ) continue;

-               if ( reg.n_IdReg == n_IdReg )
+               if ( reg.n_idreg == idreg ) {

                        break;
                        break;

+               }

        }
        }

-       
+

        /* me paro en el que tengo que borrar */
        /* me paro en el que tengo que borrar */

-       actual = fseek(f_idx, -sizeof(EMUFS_IDX), SEEK_CUR);
-       /*actual = ftell(f_idx);*/ /* Guardo la posicion actual */
-       printf("ACTUAL = %ld\n", actual/sizeof(EMUFS_IDX));
+       fseek(f_idx, -sizeof(EMUFS_IDX), SEEK_CUR);     
+       actual = ftell(f_idx); /* Guardo la posicion actual */

        fseek(f_idx, 0, SEEK_END); /* me voy al final */
        final = ftell(f_idx); /* veo cuando ocupa el archivo */
        fseek(f_idx, 0, SEEK_END); /* me voy al final */
        final = ftell(f_idx); /* veo cuando ocupa el archivo */

-       printf("tamanio del archivo de bloques y registros = %ld\n", final/sizeof(EMUFS_IDX));
-       fseek(f_idx, actual, SEEK_SET); /* vuelvo al lugar desde donde quiero justificar */
-       
+       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);
        cant = (final-actual)/sizeof(EMUFS_IDX);

-       printf("cant = %ld\n", cant);
+       /*apunto al siguiente del que quiero borrar*/
+       /*leo todos los que quedan*/

        for(i=0; i<cant-1; i++) {
        for(i=0; i<cant-1; i++) {

-                  /* Calculo donde empieza el proximo elemento a mover */
-                  final = actual+sizeof(EMUFS_IDX); printf("final = %ld actual =  %ld\n", final/sizeof(EMUFS_IDX), actual/sizeof(EMUFS_IDX));
-                  /* Me paro en ese lugar */
-                  fseek(f_idx, final, SEEK_SET);
-                  /* y lo leo */
-                  fread(&buffer, sizeof(EMUFS_IDX), 1, f_idx);
-                  /* Ahora me paro en la nueva posicion de este item */
-                  fseek(f_idx, actual, SEEK_SET);
-                  /* y lo guardo */
-                  fwrite(&buffer, sizeof(EMUFS_IDX), 1, f_idx);
-                  /* Ahora el proximo item va en la posicion siguiente */
-                  actual += sizeof(EMUFS_IDX);printf("actual = %ld\n", actual/sizeof(EMUFS_IDX));
+               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);
+               fwrite(&reg, sizeof(EMUFS_IDX), 1, f_idx);
+               actual += sizeof(EMUFS_IDX);

        }
        }

-       fseek (f_idx,0,SEEK_END);
+       /* Aca deberia estar al final del archivo , no hago seek */     

        tam = ftell(f_idx);
        tam = ftell(f_idx);

-       printf("tamanio del archivo de bloques y registros = %ld\n", tam/sizeof(EMUFS_IDX) - 1);

        fclose(f_idx);
        fclose(f_idx);

-       truncate(name_f_idx, tam - sizeof(EMUFS_IDX));
+       truncate(name_f_idx, tam-sizeof(EMUFS_IDX));
+       

        return 0;
 }
        return 0;
 }

+
+EMUFS_REG_ID emufs_idx_get_new_id(EMUFS* efs, int* err)
+{
+       EMUFS_REG_ID id;
+
+       id = emufs_did_get_last(efs, err);
+       if (id == EMUFS_NOT_FOUND) {
+               id = emufs_idx_buscar_mayor_id_libre(efs, err);
+               if (*err) {
+                       PERR("error al obtener id mayor");
+                       return id;
+               }
+       }
+       return id;      
+}
+