Piloteado de mini bug que pudiera existir en insertar ordenado, siempre se devuelve...

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

diff --git a/emufs/idx.c b/emufs/idx.c
index ae717e7993519e0d7d0afdad24bee1c5a8ea0107..ec869663f027d6725a3a1f6718a7e7dd72c5991a 100644 (file)
--- a/emufs/idx.c
+++ b/emufs/idx.c
@@ -37,6 +37,8 @@
 
 #include "idx.h"
 #include "did.h"
 
 #include "idx.h"
 #include "did.h"

+#include "error.h"
+#include "common.h"

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


@@ -50,7 +52,6 @@ FILE* emufs_idx_abrir(EMUFS* efs, const char* mode)
        filename = (char*) malloc(sizeof(char) * (strlen(efs->nombre)
                                + strlen(EMUFS_IDX_EXT) + 1));
        if (filename == NULL) {
        filename = (char*) malloc(sizeof(char) * (strlen(efs->nombre)
                                + strlen(EMUFS_IDX_EXT) + 1));
        if (filename == NULL) {

-               /* TODO Manejo de errores */

                return NULL;
        }
        strcpy(filename, efs->nombre);
                return NULL;
        }
        strcpy(filename, efs->nombre);


@@ -78,27 +79,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");
        if ((f_idx = fopen(name_f_idx, "rb")) == NULL) {
                PERR("No se puede abrir archivo");

-               *err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
+               *err = EMUFS_ERROR_CANT_OPEN_FILE;

                return EMUFS_NOT_FOUND;
        }
                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.id_reg >= max) {
-                       max = reg.id_reg;
-                       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 {


@@ -118,18 +116,13 @@ EMUFS_BLOCK_ID emufs_idx_buscar_registro(EMUFS *emu, EMUFS_REG_ID reg_id)
        
        if ((f_idx = fopen(name_f_idx, "rb")) == NULL) {
                PERR("No se puede abrir archivo");
        
        if ((f_idx = fopen(name_f_idx, "rb")) == NULL) {
                PERR("No se puede abrir archivo");

-               /*    *err = 4; * EMUFS_ERROR_CANT_OPEN_FILE */
+               /* *err = EMUFS_ERROR_CANT_OPEN_FILE; */

                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.id_reg == 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);
                        return reg.location;
                }
                        fclose(f_idx);
                        return reg.location;
                }


@@ -139,7 +132,6 @@ 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 id_reg, EMUFS_BLOCK_ID location)
 {
        FILE *f_idx;
 int emufs_idx_agregar(EMUFS *emu, EMUFS_REG_ID id_reg, EMUFS_BLOCK_ID location)
 {
        FILE *f_idx;


@@ -149,13 +141,14 @@ int emufs_idx_agregar(EMUFS *emu, EMUFS_REG_ID id_reg, EMUFS_BLOCK_ID location)
        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;
-               
-    /* Note: Location = Bloque para Tipo 1 y 3, Offset para Tipo 2 */
+       if ( (f_idx = fopen(name_f_idx,"r+"))==NULL ) return -1;
+
+       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);
        reg.id_reg = id_reg;
        reg.location = location;
        fwrite(&reg,sizeof(EMUFS_IDX),1,f_idx); 
        fclose(f_idx);

+

        return 0;
 }
 
        return 0;
 }
 


@@ -165,43 +158,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;

        
        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;
+       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;
-               if ( reg.id_reg == idreg ) {
-                       break;
-               }
-       }
-
-       /* me paro en el que tengo que borrar */
-       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, 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);

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

        
        return 0;
 }
        
        return 0;
 }


@@ -210,53 +178,111 @@ EMUFS_REG_ID emufs_idx_get_new_id(EMUFS* efs, int* err)
 {
        EMUFS_REG_ID id;
 
 {
        EMUFS_REG_ID id;
 

-       (*err) = 0;

        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");
                        return EMUFS_NOT_FOUND;
                }
        }
                id = emufs_idx_buscar_mayor_id_libre(efs, err);
                if (*err) {
                        PERR("error al obtener id mayor");
                        return EMUFS_NOT_FOUND;
                }
        }

+

        return id;      
 }
 
        return id;      
 }
 

-unsigned int emufs_idx_get_count(EMUFS *emu)
+EMUFS_REG_ID *emufs_idx_get(EMUFS *emu, EMUFS_REG_ID *cant)

 {
 {

-       FILE *fp;
+       FILE *f_idx;
+       int count;

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

-       long tam;
+       EMUFS_REG_ID *tmp;
+       EMUFS_IDX reg;

        
        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);
 

-       fp = fopen(name_f_idx, "r");
-       if (fp == NULL) return 0;
+       if ( (f_idx = fopen(name_f_idx, "rb"))==NULL){
+               PERR("No se pudo abrir el archvo");
+               (*cant) = EMUFS_NOT_FOUND;
+               return NULL;
+       }

 
 

-       fseek(fp, 0, SEEK_END);
-       tam = ftell(fp);
-       fclose(fp);
+       tmp = NULL;
+       count = 0;
+       while (!feof(f_idx)) {
+               if (fread(&reg, sizeof(EMUFS_IDX), 1, f_idx) != 1) continue;
+               if (reg.location != EMUFS_NOT_FOUND) {
+                       count++;
+                       tmp = realloc(tmp, count*sizeof(EMUFS_REG_ID));
+                       tmp[count-1] = reg.id_reg;
+               }
+       }
+       fclose(f_idx);

 
 

-       return tam/sizeof(EMUFS_IDX);
+       (*cant) = count;
+       return tmp;

 }
 
 }
 

-unsigned long emufs_idx_get_id_att(EMUFS *emu, unsigned int pos)
+/** @todo FIXME borrar cuando se cambie emufs_gui/registros.c que es el unico
+ * que lo usa.
+ * @deprecated Usar emufs_idx_buscar_registro.
+ */
+int emufs_idx_existe_id(EMUFS *emu, int ID)

 {
 {

-       FILE *fp;
+       FILE *f_idx;

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

-       EMUFS_IDX id;
+       EMUFS_IDX reg;

        
        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);
 

-       fp = fopen(name_f_idx, "r");
-       if (fp == NULL) return EMUFS_NOT_FOUND;
+       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*/
+}

 
 

-       fseek(fp, pos*sizeof(EMUFS_IDX), SEEK_SET);
-       fread(&id, sizeof(EMUFS_IDX), 1, fp);
-       fclose(fp);
+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);

 
 

-       return id.id_reg;
+       if ( (f_idx = fopen(name_f_idx, "r+")) == NULL){
+               PERR("No se pudo abrir el archivo");
+               return -1;
+       }
+       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;

 }
 
 }
 

+long emufs_idx_get_file_size(EMUFS* efs, int* err)
+{
+       char name[255];
+       strcpy(name, efs->nombre);
+       strcat(name, EMUFS_IDX_EXT);
+       return emufs_common_get_file_size(name, err);
+}