Question

我试图在C ++中删除二叉树，但我遇到的问题是树的大小似乎没有变化。这是我使用的尺寸函数：

int BST::size(Node *& cur_root)
{
    if (cur_root == NULL) {
        return 0;
    } else { 
        return(size(cur_root->m_left) + 1 + size(cur_root->m_right));
    }
}

这就是我尝试使用它的功能：

void BST::deletetree(Node *& cur_root)
{
    cout << "tree size: " << size() << endl;
    if (cur_root!=NULL)
    {
        deletetree(cur_root->m_left);
        deletetree(cur_root->m_right);
        delete cur_root;
        if(cur_root->m_left != NULL) {
            cur_root->m_left = NULL;
        }
        if(cur_root->m_right != NULL) {
            cur_root->m_right = NULL;
        }
        cur_root=NULL;
    }
}

对于三个（1,2,3）的树大小，我的输出是：

tree size: 3
tree size: 3
tree size: 3
tree size: 3
tree size: 3
tree size: 3
tree size: 3

有谁知道为什么我的尺寸不会随着每个删除的节点而下降？

编辑：我删除了if语句，但问题仍然存在

void BST::deletetree(Node *& cur_root)
{
    cout << "tree size: " << size() << endl;
    if (cur_root!=NULL)
    {
        deletetree(cur_root->m_left);
        deletetree(cur_root->m_right);
        delete cur_root;
        cur_root=NULL;
    }
}

编辑2：这是我的完整代码。 BST.h：

#ifndef BST_H
#define BST_H

#include <iostream>
using namespace std;

class BST
{
    public:
        BST();
        bool insert(string str) {return insert(str, m_root);}
        int size() {return size(m_root);}
        void deletetree() {return deletetree(m_root);}
    private:
        class Node
        {
            public:
                Node(string value, Node *left = NULL, Node *right = NULL)
                {m_value = value; m_left = left; m_right = right;}
                string m_value;
                Node *m_left;
                Node *m_right;
        };
        Node *m_root;
        bool insert(string str, Node *& cur_root);
        int size(Node *& cur_root);
        void deletetree(Node *& cur_root);
};

#endif

BST.cpp：

#include <iostream>
using namespace std;
#include <algorithm>
#include <vector>
#include <queue> 
#include <math.h> 
#include "bst.h"

BST::BST()
{
    m_root = NULL;
}

bool BST::insert(string str, Node *& cur_root)
{
    /*if (find(str) == true) {
        return false;
    }*/
    if (cur_root == NULL) {
        cur_root = new Node(str);
        return true;
    } else {
        if (cur_root->m_value < str) {
            return insert(str, cur_root->m_right);
        } else {
            return insert(str, cur_root->m_left);
        }
    }
}

int BST::size(Node *& cur_root)
{
    if (cur_root == NULL) {
        return 0;
    } else { 
        return(size(cur_root->m_left) + 1 + size(cur_root->m_right)); 
    }
}

void BST::deletetree(Node *& cur_root)
{
    cout << "tree size: " << size() << endl;
    if (cur_root!=NULL)
    {
        deletetree(cur_root->m_left);
        deletetree(cur_root->m_right);
        delete cur_root;
        cur_root=NULL;
    }
}

main.cpp中：

#include <iostream>
using namespace std;
#include "bst.h"

int main()
{
    BST tree;
    tree.insert("1");
    tree.insert("2");
    tree.insert("3");
    tree.deletetree();
}

忽略插入时的注释部分。当我有一个find函数时，这是我将要实现的东西。

Answer 1

让我们看一下deletetree函数中的这一行：

delete cur_root;
if(cur_root->m_left != NULL) {
    cur_root->m_left = NULL;
}
if(cur_root->m_right != NULL) {
    cur_root->m_right = NULL;
}

第一行破坏cur_root指向的对象，其余行取消引用指针以访问现在 被破坏的 对象。

取消引用指向被破坏对象的指针会导致undefined behavior，这使得所有关于行为的猜测都无用。

对此的简单解决方案是不执行if检查，因为根本不需要它们。您所需要的只是delete cur_root后跟cur_root = nullptr。

现在，当我们看到您的完整代码时，我们可以看到您的插入函数不会创建树。它会创建一个列表（按照您显示的顺序插入数据）。

当您插入"1"时，它将成为树的根。然后当您插入"2"时，它变为m_root->m_right。然后当您插入"3"时，它变为m_root->m_right->m_right。我没有介绍你的代码，知道它是否会导致size的{{1}}出现错误，但其中一个会这样做。您应该使用调试器来找出导致您明显问题的函数。

然后您应该考虑将节点插入树中的顺序。或者，如果您的树应该自我平衡并在插入时重新排序节点。

试图删除二叉树C ++

1 个答案: