UNDO: Cambio constructor por metodo estatico.

Cambio el constructor de "abrir" por un metodo estatico Open. Esto
tiene la ventaja que si el archivo a abrir no es un arbol B no se crea
un arbol inconsistente y se elimina posible error de uso.

diff --git a/src/btree.cpp b/src/btree.cpp
index 2701b3a270aa3864346edf05525ef9c035471573..ae68316e256a790dee91e9a888856b044310fe51 100644 (file)
--- a/src/btree.cpp
+++ b/src/btree.cpp
@@ -36,54 +36,41 @@ BTree::BTree (const std::string &name, unsigned int block_size, int tt, int kt,
        delete [] node;
 }
 
-BTree* BTree::Open (const std::string &name)
+BTree::BTree (const std::string &name)
 {
        /* Leo los bloques recuperables */
-       FILE *fp;
-       BTree *tree = new BTree ();
-
-       std::string del = name + ".del";
+       std::string del = filename + ".del";
 
        fp = fopen (del.c_str (), "wb");
        if (fp != NULL) {
                uint i;
 
                while (fread (&i, 1, sizeof (uint), fp)) {
-                       tree->deleted_nodes.push_back (i);
+                       deleted_nodes.push_back (i);
                }
 
                fclose (fp);
        }
 
-       del = name + ".blockdel";
+       del = filename + ".blockdel";
 
        fp = fopen (del.c_str (), "wb");
        if (fp != NULL) {
                uint i;
 
                while (fread (&i, 1, sizeof (uint), fp)) {
-                       tree->deleted_block_data.push_back (i);
+                       deleted_block_data.push_back (i);
                }
 
                fclose (fp);
        }
 
-       tree->filename = name;
-       tree->fp = fopen (name.c_str(), "rb+");
+       fp = fopen (name.c_str(), "rb+");
        if (!fp) {
                /* TODO : mandar una exception ? */
-               delete tree;
-               return NULL;
-       }
-       tree->ReadFileHeader ();
-
-       if (strcmp ("DILUMA", tree->header.magic)) {
-               /* Arbol no tiene magic!! */
-               delete tree;
-               return NULL;
+               return;
        }
-
-       return tree;
+       ReadFileHeader ();
 }
 
 BTree::~BTree ()

diff --git a/src/btree.h b/src/btree.h
index 526b428dac7953ecfa459338f38607b16481796e..9e492139317a15351162c43f5c3637fc42a767cf 100644 (file)
--- a/src/btree.h
+++ b/src/btree.h
@@ -251,8 +251,7 @@ struct BTreeFindResult {
 class BTree {
        public:
                BTree (const std::string &filename, unsigned int block_size, int t_t = TYPE_IDENTIFICACION, int k_t = KEY_FIXED, bool create_new_file = false);
-
-               static BTree *Open (const std::string &filename);
+               BTree (const std::string &filename);
 
                ~BTree ();
 
@@ -334,9 +333,6 @@ class BTree {
 
 
                /* DEBUG */
-       private:
-               BTree () {}
-
        public:
                void PrintNode (uint num);
 };