Question

我有这个插入方法，应该将节点插入到包含人名和年龄的BST中。树本身按年龄排序，每个节点包含该年龄段的人员链接列表。

我对此树的插入方法并未正确地将这些节点相互比较。输入如

insert 1 50 john
insert 1 30 elvis
insert 1 90 david
insert 1 50 james
insert 1 95 abel
insert 1 80 esther
insert 1 95 vivian
insert 1 95 barbara
insert 1 50 james

我应该只看到插入失败一次，重复插入James。相反，我的代码似乎制作了两个50岁的节点，并且在插入vivian

时失败了

Input Command: insert 1 50 john
Input Command: insert 1 30 elvis
Input Command: insert 1 90 david
Input Command: insert 1 50 james
Input Command: insert 1 95 abel
Input Command: insert 1 80 esther
Input Command: insert 1 95 vivian
    --- Failed.
Input Command: insert 1 95 barbara
        --- Failed.
Input Command: insert 1 50 james

我不确定为什么会这样。它似乎甚至没有在正确的时间进行比较。

这里的任何一种方式都是我的代码

    bool insert(const int &a, const string &n) {
        BinaryNode* t = new BinaryNode;
        BinaryNode* parent;
        t->it = t->nameList.begin();
        t->nameList.insert(t->it ,n);
        t->age = a;
        t->left = NULL;
        t->right = NULL;
        parent = NULL;

        if(isEmpty()){ 
            root = t;
            return true;
        }
        else {
            BinaryNode* curr;
            curr = root;
            while(curr) {
                parent = curr;
                if(t->age > curr->age) 
                    curr = curr->right;
                else 
                    curr = curr->left;
            }

            if(t->age == parent->age) {
                for(list<string>::iterator ti = parent->nameList.begin(); ti != parent->nameList.end(); ti++) {
                    string temp = *ti;
                    cout << temp << endl;
                    if(n.compare(temp)) 
                        return false;
                    else if(ti == parent->nameList.end())
                        parent->nameList.insert(ti,n);
                }
                return true;
            }

            if(t->age < parent->age) {
                parent->left = t;
                return true;
            }
            else {
                parent->right = t;
                return true;
            }
        }
    }

Answer 1

您的代码中的问题是您假设在BST中，如果插入重复值，它肯定会是原始值的子项，这是不正确的。例如，考虑这种情况。

因此，您要做的是在遍历树而不是if(t->age == parent->age)时检查重复的年龄。您可以按如下方式更新while循环 -

while(curr){
    parent = curr;
    if(t->age == curr->age){
        //Check for duplicate name here
    }
    else if(t->age > cur->age)
        curr = curr->right;
    else
        curr = curr->left;
}

此外，您应该在失败的情况下释放新节点t的内存。

Answer 2

我想这个

if(n.compare(temp)) 
  return false;
else if(ti == parent->nameList.end())
  parent->nameList.insert(ti,n);

应该是

if(!n.compare(temp)) 
  return false;
else if(ti == parent->nameList.end())
  parent->nameList.insert(ti,n);

通过在每次检测到相同年龄时写if (n.compare(temp))，当名称不同时，它会失败。

插入方法无法正确比较

2 个答案: