Question

I m 定义双双平级搜索树类:

template <class T>
class BSTNode
{
  public:
    BSTNode(){
      right = left = 0;
    }

    BSTNode(const T& el, BSTNode * l = 0, BSTNode * r = 0)
    {
    val = el; left = l; right = r;
    }   

    T val;
    BSTNode * right;
    BSTNode * left;
};

template <class T>
class BST
{
  public:
    BST(){ root = 0;}           
    void Insert(const T& el);
  private:
    BSTNode<T> * root;
};

and I implement the Insert() function like this:

template <class T>
void BST<T>::Insert(const T& el)
{
  bool bEqual = false;
  BSTNode<T> * p = root;

  while(p)
  {
    if(el == p->val)
    {
      bEqual = true;
      return;
    }
    if(el < p->val)
    {
      // go left
      p = p->left;
    }
    if(el > p->val)
    {
      // go right
      p = p->right;
    }
  }

  if(!p)
  {
    std::cout << "creating new node " << el << "
";
    p = new BSTNode<T>(el);
  }  
}

为什么<代码>root点变量停留在0,而不是新物体的地址?

Answer 1

您从来不做root = ?; 载于您的代码;还有p = 新的BSTNode<T>el); 泄漏记忆。

我的猜测是,你希望<代码>p>成为点人,这样你就可以改变原点。

 BSTNode<T> *& p = root; // watch out, it won t solve anything

But, in such case, p is not-reassignable. You may want to check if pointer you assign to p is null and just insert the new value in correct place (eg. p->left = new BSTNode<T>(el);) and reassign p only when given pointer is not null.

我指的是:

template <class T>
void BST<T>::Insert(const T& el)
{
  bool bEqual = false;
  BSTNode<T> * p = root;

  if (p == 0)
  {
     root = new BSTNode<T>(el);
     return;
  }

  while(true)
  {
    if(el == p->val)
    {
      bEqual = true;
      return;
    }
    if(el < p->val)
    {
      if (p->left == 0)
      {
        p->left = new BSTNode<T>(el);
        return;
      }
      p = p->left;
    }
    if(el > p->val)
    {
      if (p->right == 0)
      {
        p->right = new BSTNode<T>(el);
        return;
      }
      p = p->right;
    }
  }
}

Answer 2

由于在建造物体时

the statement

  p= root

最初将p定为无效。

你们制造了一个新物体,把其地址转到一页而不是根基......

这只是一个根本的复制件,而不是一个提及点。

友情链接