标签:c algorithm binary-tree c17
嗨这是我的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
本站声明: 1. iCode9 技术分享网(下文简称本站)提供的所有内容,仅供技术学习、探讨和分享; 2. 关于本站的所有留言、评论、转载及引用,纯属内容发起人的个人观点,与本站观点和立场无关; 3. 关于本站的所有言论和文字,纯属内容发起人的个人观点,与本站观点和立场无关; 4. 本站文章均是网友提供,不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属;如您发现该文章侵犯了您的权益,可联系我们第一时间进行删除; 5. 本站为非盈利性的个人网站,所有内容不会用来进行牟利,也不会利用任何形式的广告来间接获益,纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。