Question

所以我将一个模板化的二叉树字典实现为一个继承自抽象Dictionary类的类，我的add函数存在问题，我无法弄清楚。

基本上，我的树有节点，它既有键又有值，指向父节点，左子节点和右子节点。节点的代码是

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

（我知道使用shared_ptr比原始指针好，但这个项目不需要）该树还有一个成员指针bNode* root。

为了测试我的树，我向它添加元素并在每个树之后调用display。到目前为止，显示空列表的工作正常，显示包含一个元素和两个元素的列表也是如此。当我添加第三个，然后和调用显示时，它会中断......

以下是添加功能的代码

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“正确显示”和“向左显示”的行是我尝试修复错误，因为它不应该运行，除非该函数即将显示另一个节点。在显示前2个节点后，这一切都按预期工作;但是，在第三个节点，它将显示该元素，然后打印“显示左＆lt; *第三个节点的键＆gt; ...”，然后崩溃并给我一个分段错误。

据我所知，这意味着由于某种原因，代码运行时好像第三个添加的节点（应该是叶子节点）实际上有子节点，但是当它尝试读取成员变量时崩溃不存在的节点。

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

编辑：这是一个展示问题的代码片段;

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），但编译器正在尝试运行，就好像存在一样。

Answer 1

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

二叉树类创建随机节点，打破

1 个答案: