二进制树类创建随机节点,断裂



因此,我正在实现一个模板的二进制树词典作为一个从抽象词典类继承的类,并且我对添加函数遇到了问题,我无法找到。

基本上,我的树的节点既有钥匙和价值,又要给其父母,左子和右子孩子的指针。节点的代码为

struct bNode {
    K key;
    V value;
    bNode* left;
    bNode* right;
    bNode* parent;
};

(我知道使用shared_ptr比原始指针更好,但是这个项目并不需要)该树还具有成员指针bNode* root

要测试我的树,我在其中添加了元素,并在每一个后呼叫显示。到目前为止,显示空列表正常工作,显示一个元素和两个元素的列表也是如此。但是,当我添加第三个并呼叫显示时,它会破裂...

这是添加函数的代码

void add(K key, V value) override {
    // if list is empty, create new root node
    if (!root){
        root = new bNode;
        root->key = key;
        root->value = value;
        return;
    }
    bNode* node = root;
    // travel down tree, checking each node on the way
    while (true){
        // if node exists with passed-in key, change value to passed-in value
        if (node->key == key){
            node->value = value;
            return;
        }
        // if key < or > node and node has a left / right child, travel left / right branch
        // if node doesn't have a left / right child, create one and pass in key
        if (key < node->key){
            if (node->left){
                node = node->left;
            } else {
                node->left = new bNode;
                node->left->parent = node;
                node->left->key = key;
                node->left->value = value;
                return;
            }
        } else {
            if (node->right){
                node = node->right;
            } else {
                node->right = new bNode;
                node->right->parent = node;
                node->right->key = key;
                node->right->value = value;
                return;
            }
        }
    }
}

此处显示

void display() const override {
    if (!root){
        return;
    }
    displayHelper(root);
    cout << endl;
}

这是DisplayHelper

void displayHelper(bNode* node) const {
    cout << "(" << node->key << ", " << node->value << ") ";
    if (node->left){
        cout << "displaying left... <" << node->key << "> ";
        displayHelper(node->left);
    }
    if (node->right){
        cout << "displaying right... <" << node->key << "> ";
        displayHelper(node->right);
    }
}

cout"显示右显示"one_answers"显示左显示"的行是我试图进行错误的尝试,因为除非函数即将显示另一个节点,否则它不应运行。在显示前两个节点后,这一切都按预期工作;但是,在第三个节点上,它将显示该元素,然后打印"显示左&lt;*3rd Node的键> ...",然后崩溃并给我一个分割故障。

据我所知,这意味着,由于某种原因,代码正在运行,好像第三个节点(应该是叶子节点)实际上有孩子,但是当它试图读取成员变量的情况时崩溃不存在节点。

我无法弄清楚是什么原因导致错误,因为这都是非常简单的代码!任何帮助将不胜感激!

编辑:这是代码的示例片段,证明了问题;

BSTDictionary<string, int> t;
t.display();
t.add("e", 5);
t.display();
t.add("b", 20);
t.display();
t.add("c", 12);
t.display();
return 0;

运行时,输出看起来像

(E,5)

(e,5)显示左...(e)(b,20)

(e,5)显示左...(e)(b,20)显示右...(b)(c,12)显示左...(c)

流程返回-1073741819(0xc0000005)执行时间:1.993 S

不应该有包含(C,12)的节点的左子女,但是编译器正试图像存在一样运行。

您需要确保对 nullptr对象中的所有叶子节点指针初始化,除非您立即将它们分配给有意义的东西。

最新更新