使用全局指针在C++中实现红黑树

Implementation of Red black tree in C++ using global pointer

本文关键字：实现 C++ 全局 指针      更新时间：2023-10-16

我在更新以下代码中的全局指针时遇到问题，

#include <iostream>
using namespace std;
struct RB{
    RB()=default;
    RB(int clr):color(clr) { }
    int color;
    RB *p,*left,*right;
    int key;
};
RB *Tnil=new RB(0);
RB *T=Tnil;
void insert(RB *T,RB *z)
{
    RB *y=Tnil;
    RB *x=T;
    while(x!=Tnil)
    {
        y=x;
        if(z->key<y->key)
          x=x->left;
        else
         x=x->right;
    }
    z->p=y;
    if(y==Tnil)
      T=z;
    else if(z->key<y->key)
      y->left==z;
    else
      y->right=z;
    z->right=Tnil;
    z->left=Tnil;
    z->color=1;
}
void print(RB *T)
{
    if(T==Tnil)
      return;
    print(T->left);
    cout<<T->key;
    print(T->right);
}
int main()
{
  for(int i=1;i<10;++i)
  {
    RB *x=new RB;
    x->key=i;
    insert(T,x);
  }
   print(T);
}

问题是，当我期望insert函数中的比较y==Tnil为true时，它的计算结果为false。函数结束后，T再次变为等于Tnil，因此没有插入任何内容。有什么帮助吗？


您想要更新全局T。
因此，您应该传递全局T的引用以插入：

更换

空心插入件（RB*T，RB*z）

带有

空心插入件（RB*&T，RB*z）

（否则，只更新全局指针T的副本）

同样正如ComicansMS在您的示例中所提到的

y->left==z

应该被取代

y->left=z


最佳，

Jack

您的命名搞砸了。

您有两个名为T的变量，一个在全局范围内，另一个作为insert的参数。因此，insert中的赋值T=z;实际上并不作用于全局变量T，而是作用于参数，因此在函数之外没有副作用。

一般来说，尽量避免使用像T、z和x这样的单字母变量名。它们使您的代码难以阅读，并且可以很容易地隐藏像这样的错误。此外，请避免从函数内部进行非本地化更新。从函数中更新全局变量只会带来这样的麻烦。更好的方法是让insert返回一个指向新顶级节点的指针。

相关文章：