fix destructor.

Author: zhangyuwei

binaryTree.cpp

@@ -118,8 +118,8 @@ namespace SuperTAD::binary
 
     BasePruner::~BasePruner()
     {
-        _data = NULL;
-        _tree = NULL;
+        _data = nullptr;
+        _tree = nullptr;
     }
 
     Pruner1::Pruner1(Tree &tree, int k) : BasePruner(tree)

data.cpp

@@ -74,13 +74,22 @@ namespace SuperTAD
         for (int s=0; s<_N_; s++) {
             delete [] _logVolTable[s];
             delete [] _edgeCountTable[s];
-            if (!_initByPointer)
+            _logVolTable[s] = nullptr;
+            _edgeCountTable[s] = nullptr;
+            if (!_initByPointer){
                 delete [] _contactTable[s];
+                _contactTable[s] = nullptr;
+            }
+
         }
         delete [] _logVolTable;
         delete [] _edgeCountTable;
-        if (!_initByPointer)
+        _logVolTable = nullptr;
+        _edgeCountTable = nullptr;
+        if (!_initByPointer){
             delete [] _contactTable;
+            _contactTable = nullptr;
+        }
     }
 
 

multiTree.cpp

@@ -23,9 +23,13 @@ namespace SuperTAD::multi {
 
     Tree::~Tree()
     {
-        for (auto & i : _nodeList)
+        for (auto & i : _nodeList){
             delete i;
+            i = nullptr;
+        }
+        _nodeList.clear();
         delete _root;
+        _root = nullptr;
     }
 
 

multiTree.h

@@ -29,9 +29,10 @@ namespace SuperTAD::multi {
         }
 
         ~TreeNode() {
-            for (auto i : _children)
-                delete i;
-            _children.clear();
+            if (! _children.empty()){
+                for (auto i : _children)
+                    i = nullptr;
+            }
         }
 
         TreeNode& operator=(const TreeNode &copy) {