【问题标题】:Where is the flaw in my binary tree insertion logic?我的二叉树插入逻辑的缺陷在哪里?
【发布时间】:2017-01-30 03:13:38
【问题描述】:

我的解决方案是这样的

/*
Node is defined as 

typedef struct node
{
   int data;
   node * left;
   node * right;
}node;

*/


node * insert (node * root, int value)
{
   bool inTreeAlready = false;
   node * cur = root;
   while(cur != NULL)
   {
       if(cur->data < value)
           cur = cur->right;
       else if(cur->data > value)
           cur = cur->left;
       else
       {
           inTreeAlready = true;
           break;
       }
   }
   if(!inTreeAlready)
   {
       cur = new node; 
       cur->data = value;
       cur->left = NULL;
       cur->right = NULL;
   }   
   return root;
}

问题提示说明您应该在插入后返回树的根。

这显然是错误的,因为输出是

Wrong Answer!
Some possible errors:
1. You returned a NULL value from the function. 
2. There is a problem with your logic
3. You are printing some value from the function

这不是很具有描述性。

我已经仔细检查了我的逻辑,但不知道交易是什么。

【问题讨论】:

  • 我看到您创建了一个新节点,但我没有看到您实际上将它链接到树。从root 无法访问它 - 它只是泄露了。如果您从一棵空树 (root == NULL) 开始,那么您显然也会以一棵空树结束(root 仍然是 NULL) - 所以您甚至永远不会进入第一个节点。
  • 奇怪的是这段代码打印了一些东西,因为它没有任何printf 调用。

标签: c++ algorithm tree


【解决方案1】:

您没有将新节点添加到树中。这是修改后的版本。

node * insert (node * root, int value)
{
   bool inTreeAlready = false;
   node * cur = root;
   node *parent;
   bool right;
   while(cur != NULL)
   {
       parent = cur;
       if(cur->data < value)
       {
           cur = cur->right;
           right = true;
       }
       else if(cur->data > value)
       {
           cur = cur->left;
           right = false;
       }
       else
       {
           inTreeAlready = true;
           break;
       }
   }
   if(!inTreeAlready)
   {
       cur = new node; 
       cur->data = value;
       cur->left = NULL;
       cur->right = NULL;
       if(root == NULL) root = cur;
       else if(right) parent->right = cur;
       else parent->left = cur;
   }   
   return root;
 }

【讨论】:

  • 想想我本可以在 C# 中用 5-7 行代码完成这一切,哈哈。一定喜欢 C++。
  • @user7127000 究竟在“5-7 行”中做什么?无论使用哪种语言,您对代码的问题是您完全没有将新节点与树链接起来。如果没有代码,任何语言都不会神奇地使节点成为树的一部分。
  • @user7127000 我可以将其设为一行(当然,其中包含几个分号)。 ;-) 只是在开玩笑。我同意使用一种语言完成一项任务似乎很复杂,但使用另一种语言可能会非常简单。
  • 顺便说一下,这个解决方案在其中一个测试用例上出现了分段错误
  • @user7127000 再简单看一下代码,如果 root 为 NULL 就会有问题。所以我修改了代码。希望这能解决您遇到的问题。
猜你喜欢
  • 2017-06-20
  • 1970-01-01
  • 2016-01-17
  • 2017-07-16
  • 2015-02-13
  • 2019-08-26
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多