C++ C++；，树插入

嗨，这是我的SearchTree类的代码。 Node*是一个包含m_info type int、m_left（按信息显示较小节点）和m_right（按信息显示较大节点）的结构

我正在尝试将新节点插入到树中。 但看起来我在内存管理方面遗漏了一些东西。 tempo是指向新节点的指针，但它与m_root无关。 我在这里感到困惑。我很喜欢C++的力量，但是它颠覆了我的逻辑。

---

我在这里遗漏了什么？

你继续前进

节奏

，直到它等于

nullptr

。在这一点上，你已经离开了这棵树，你手里只有一个指向虚无的指针。请注意，特别是程序无法确定上次访问哪个节点导致

tempo

变为

null

---

您需要做的是提前一步停止：虽然

tempo

仍指向节点，但下一步将使其指向

null

。现在，您手中仍然有一个有效的树节点，可以将新分配的节点附加到它。

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

我解决这个问题的方法是在迭代之前检查child是否为null ptr

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

的指针。您可以通过

tempo

更改

m_root

好的，我可以保留另一个变量，节奏的父项。@R0xx0rZzz是的，这是一种方法。如果你觉得答案是你的问题，请考虑接受答案。很高兴听到你理解了这个问题！但是，看看当m_root为null时我应该怎么做？我不想有一个if语句来检查每个insert@R0xx0rZzz：如果，我个人会选择

，但是你的另一个选择是为树指定一个虚拟根。好的。谢谢，我还不知道智能指针。我觉得不错。但我想要一个更优雅的解决方案。如果可能的话，我添加了一个更“优雅”的解决方案。是的，它很有效。太好了，这正是我想要的。现在我只需要理解它，在第一个解决方案中，
tempo
是节点的副本。这意味着，如果更改
tempo
的值，则不会更改节点。在第二种解决方案中，tempo是指向节点地址的指针。这样你就可以更改地址了。啊。令人惊叹的。非常感谢。这种解决方案在软件开发中是一种好的实践吗？
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);
}