X-Git-Url: https://git.llucax.com/z.facultad/75.06/emufs.git/blobdiff_plain/07dca6914e0894f05543795c7c453168e0d572cd..720b9cc2195c640096b5461399fe142fa987c5e4:/emufs/idx.c

diff --git a/emufs/idx.c b/emufs/idx.c
index 6cf562e..1ae3190 100644
--- a/emufs/idx.c
+++ b/emufs/idx.c
@@ -78,6 +78,7 @@ 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);
 
+	(*err) = 0;
 	if ((f_idx = fopen(name_f_idx, "rb")) == NULL) {
 		PERR("No se puede abrir archivo");
 		*err = 4; /* EMUFS_ERROR_CANT_OPEN_FILE */
@@ -86,6 +87,7 @@ EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)
 	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;
@@ -105,30 +107,36 @@ EMUFS_REG_ID emufs_idx_buscar_mayor_id_libre(EMUFS* emu, int* err)
 }
 
 /* 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];
-	unsigned short int b_Found = 0;
 	
 	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);
-	
-	/* 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 */
@@ -145,66 +153,56 @@ int emufs_idx_agregar(EMUFS *emu, EMUFS_REG_ID n_idreg, EMUFS_BLOCK_ID n_locatio
 		
     /* Note: Location = Bloque para Tipo 1 y 3, Offset para Tipo 2 */
     reg.n_idreg = n_idreg;
-		reg.n_location = n_location;
+	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 */
-int emufs_idx_borrar(EMUFS *emu, EMUFS_REG_ID n_IdReg)
+int emufs_idx_borrar(EMUFS *emu, EMUFS_REG_ID idreg)
 {
 	FILE *f_idx;
-	EMUFS_IDX reg, *buffer;
+	EMUFS_IDX reg;
 	char name_f_idx[255];
-	long actual, final, cant, i, tam, a;
+	long actual, final, cant, i, tam;
 	
 	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;
-		if ( reg.n_idreg == n_IdReg )
+		if ( reg.n_idreg == 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 */
-	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);
-	buffer = (EMUFS_IDX*)malloc((cant)*sizeof(EMUFS_IDX));
-	printf("sizeof buffer = %ld\n",(cant-1)*sizeof(EMUFS_IDX));
 	/*apunto al siguiente del que quiero borrar*/
-	final = actual + sizeof(EMUFS_IDX);
 	/*leo todos los que quedan*/
-	fseek(f_idx,final,SEEK_SET);
-	fread(buffer,sizeof(EMUFS_IDX),cant-1,f_idx) ;
-/*	for( i=0; i<cant-1; i ++)
-		printf("buffer[%ld] = %ld , %ld\n",i,*(buffer+i),*(buffer+2*i));*/
-	/*apunto al lugar donde deben ser guardados*/
-	fseek(f_idx,actual,SEEK_SET);
-	printf(" actual  = %ld\n",actual);
-	/*los guardo*/
-	fwrite(buffer,sizeof(EMUFS_IDX),cant-1,f_idx);
-	
-
-	fseek (f_idx, -sizeof(EMUFS_IDX),SEEK_END);
+	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);
+		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);
-	if ( truncate(name_f_idx, actual+(cant-1)*sizeof(EMUFS_IDX))!= 0) 
-		printf("no pude truncar\n");
-	else printf("TRUNKE!!!!\n");
+	truncate(name_f_idx, tam-sizeof(EMUFS_IDX));
 	
-	free(buffer);
 	return 0;
 }
 
@@ -212,18 +210,14 @@ EMUFS_REG_ID emufs_idx_get_new_id(EMUFS* efs, int* err)
 {
 	EMUFS_REG_ID id;
 
+	(*err) = 0;
 	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 id;
+			return EMUFS_NOT_FOUND;
 		}
 	}
 	return id;	
 }
-