Question

我正在进行轮换，并尝试根据我调用的方法向左侧或右侧旋转已经到我的二叉搜索树的课程。我的代码中有一个错误，我认为我已经将它与旋转方法隔离，因为每次我成功进行旋转后尝试打印，它都会进入无限循环。在我的方法中是否有某种类型的指针或调用混合？我觉得我的逻辑是对的，但似乎无法弄清楚错误的位置（或者即使它在这种方法中）。

这是我的左旋法：

bool BinarySearchTree::leftRotate(string number)
{
Course *x, *y;

if (treeSearch(number) == NULL){
    return false;
}
else{
    x = treeSearch(number);
}

if (x->getRight() == NULL){
    return false;
}
else{
    y = x->getRight();
    x->setRight(y->getLeft());
}

if (y->getLeft() != NULL){
    y->getLeft()->setParent(x);
}
y->getParent()->setParent(x);

if(x->getParent() == NULL){
    root = y;
}
else if( x->getParent()->getRight() == x){
    x->getParent()->setLeft(y);
}
else{
    x->getParent()->setRight(y);
}
    y->setLeft(x);
    x->setParent(y);
return true; 
}

提前致谢。

Answer 1

似乎好像

y->getParent()->setParent(x);

应该是

y->setParent(x->getParent());

但是在检查x-＆gt; getParent为NULL后执行此操作。

像

这样的东西

// Remove this line: y->getParent()->setParent(x);

if(x->getParent() == NULL){
    root = y;
}
else if( x->getParent()->getRight() == x){
    y->setParent(x->getParent());   // Insert this line
    x->getParent()->setLeft(y);
}

编辑：在再次查看代码之后，我认为即使x-＆gt; getParent（）返回NULL，也应调用y-＆gt; setParent（x-＆gt; getParent（））。原因是如果y成为新的根，y也应该将Parent设置为NULL。

所以下面的代码可能是一个更好的答案：

// Remove this line: y->getParent()->setParent(x);

y->setParent(x->getParent());   // Insert this line

if(x->getParent() == NULL){
    root = y;
}
else if( x->getParent()->getRight() == x){
    x->getParent()->setLeft(y);
}

二进制搜索树左旋转

1 个答案: