C,BTree插入

嗨这是我的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);
}

我正在尝试向树中插入一个新节点.
但看起来我在内存管理方面缺少一些东西.
tempo是指向新节点的指针,但它与m_root无关.
我在这里很困惑.我真的很喜欢c的力量,但它会使我的逻辑变得弯曲.

我在这里错过了什么？

解决方法:

您无法仅将指针保存在速度中. Tempo是您当前在树中的位置的副本.您必须将其分配给实际变量.

我对这个问题的解决方案是在迭代之前检查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中值的副本.如果你改变速度,那么m_root保持不变.

如果你使用Node ** tempo =& m_root,那么tempo是指向m_root的指针.您可以通过速度更改m_root.

标签：c,algorithm,binary-tree,c17
来源： https://codeday.me/bug/20190827/1745638.html