C ++，BTree插入

Question

嗨，这是我SearchTree类中的代码。 Node *是一种结构，其中m_info类型为int，m_left（按信息表示较小的节点）和m_right（按信息表示较大的节点）

void SearchTree::insert(const int &x) {
  Node* tempo = m_root;
  while (tempo != nullptr) {
    if (tempo->m_info >= x) {
      tempo = tempo->m_left;
    } else {
      tempo = tempo->m_right;
    }
  }
  tempo = new Node(x);
}

我正在尝试在树中插入一个新节点。 但是看起来我在内存管理中缺少一些东西。 速度是指向新节点的指针，但是它与m_root没有关系。 我在这里很困惑。我真的很喜欢c ++的强大功能，但它改变了我的逻辑。

我在这里想念什么？

Answer 1

您继续前进tempo，直到它等于nullptr。此时，您已经离开了树，您手中只有空虚的指针。请注意，特别是该程序无法确定您最后一次访问的导致tempo变成null的节点。

相反，您需要做的是提前一个步骤 ：虽然tempo仍指向一个节点，但是下一步将使其指向null。现在您手中的树仍然有效，可以将新分配的节点附加到树上。

Answer 2

不能仅以速度保存指针。速度是您在树中当前位置的副本。您必须将其分配给实际变量。

我对这个问题的解决方案是在迭代之前检查child是否为nullptr

void SearchTree::insert(const int &x) {
  if (!m_root) {
      m_root = new Node(x);
      return;
  }
  Node* tempo = m_root;
  while (true) {
    if (tempo->m_info >= x) {
      if (!tempo->m_left) {
        tempo->m_left = new Node(x);
        return;
      }
      tempo = tempo->m_left;
    } else {
      if (!tempo->m_right) {
        tempo->m_right = new Node(x);
        return;
      }
      tempo = tempo->m_right;
    }
  }
}

此外，您应该使用智能指针而不是原始指针。

另一种解决方案是使用指针指向指针。我没有测试，但是您可以尝试

void SearchTree::insert(const int &x) {
  Node** tempo = &m_root;
  while (*tempo) {
    if ((*tempo)->m_info >= x) {
      tempo = &(*tempo)->m_left;
    } else {
      tempo = &(*tempo)->m_right;
    }
  }
  *tempo = new Node(x);
}

在此图像中您可以看到。如果您使用Node* tempo = m_root，则tempo将包含m_root中值的副本。如果更改tempo，则m_root保持不变。

如果您使用Node** tempo = &m_root，则tempo指向m_root的指针。您可以通过m_root来更改tempo。