二叉搜索树的搜索代码中的错误在哪里？

Question

我为严格的二叉搜索树编写了这段代码。 （如果二叉树中的每个非叶子节点都有非空的左子树和右子树，则该树称为严格二叉树。或者，换句话说，严格二叉树中的所有节点的度数均为 0 或 2 ，从不一阶。具有 N 个叶子的严格二叉树总是包含 2N – 1 个节点。）不幸的是，它不能正确打印值。我不知道有什么问题。我在互联网上看过其他网站，代码几乎相同，但我仍然无法在我的代码中找到错误。

#include <stdio.h>
#include <stdlib.h>
struct node
{
    struct node* prev;
    int data;
    struct node* next;
};
void Binary_Search_Tree(struct node *newnode,struct node *p);
void print(struct node* p);

main()
{
    struct node *root,*nnode,*temp;
    int c,d;
    root=malloc(sizeof(struct node));

    printf("Enter the root data\t");
    scanf("%d",&d);
    root->data=d;
    root->prev=NULL;
    root->next=NULL;

    while(1)
    {
        printf("Enter your choice\n1 insert element.\n2 print tree.\n3 Exit");
        scanf("%d",&c);
        switch(c)
        {
            case 1:
                nnode=malloc(sizeof(struct node));
                printf("Enter the node data\t");
                scanf("%d",&d);

                nnode->data=d;
                nnode->prev=NULL;
                nnode->next=NULL;
                temp=root;

                Binary_Search_Tree(nnode,temp);
                break;
            case 2:
                temp=root;
                print(temp);
                break;
            case 3:
                free(root);
                free(nnode);
                free(temp);
                temp=nnode=root=NULL;
                exit(1);
                break;
        }
    }
    return 0;
}

void Binary_Search_Tree(struct node *newnode,struct node *p)
{

    if(newnode->data<p->data)
    {
        if(p->prev=NULL)
        {
            p->prev=newnode;
        }
        else if(p->prev!=NULL)
        {
            p=p->prev;
            Binary_Search_Tree(newnode,p);
        }
    }
    else if(newnode->data>p->data)
    {
        if(p->next=NULL)
        {
            p->next=newnode;
        }
        else if(p->next!=NULL)
        {
            p=p->next;
            Binary_Search_Tree(newnode,p);
        }
    }
}

void print(struct node* p)
{
    if(p!=NULL)
    {
        print(p->prev);
        printf("%d\n",p->data);
        print(p->next);
    }
}

Answer 1

主要问题是您使用赋值代替相等。

if( p->next=NULL )

与您预期的完全不同。会是

if ( p->next == NULL )
             ^^^

对于p->prev，支票将是p->prev == NULL。

所以让我们分析一下您犯错误的第一种情况。

if( p->next = NULL )

首先给p->next赋值NULL，然后我们知道赋值语句的结果就是赋值。所以条件是

 if( NULL ) 

所以永远不会输入if 语句。 else 也是如此，因为那时 p->next = NULL。所以它不会添加新节点。树保持不变。

它并没有止步于此。由于您丢失了新分配节点的地址 - 您这里有内存泄漏。

然后解决办法

if( p->next == NULL )

好吧，当我们达到叶级别时，它将等于NULL，然后您将新分配的节点的地址分配给它。这样就解决了问题。

几件事 -

1. 检查malloc的返回值。如果失败，它将返回NULL。 ₁

   root=malloc(sizeof(struct node));
   if( root == NULL ){
      perror("Malloc failure");
      exit(EXIT_FAILURE);
   }
   

2. 使用完后释放动态分配的内存。

   void freeTree(struct node *root){
      if( root ){
         freeTree(root->prev);
         freeTree(root->next);
         free(root);
      }
   }
   

3. 启用编译器警告-Wall -Werror。如果你这样做了，那个编译器就会清楚地告诉你问题。

   error: suggest parentheses around assignment used as truth value [-Werror=parentheses]
            if(p->next=NULL)
            ^~
   

4. 另外，检查scanf 的返回值。

   if( scanf("%d",&d) != 1 ){
     // Input failed 
     exit(EXIT_FAILURE); // or handle error.
   }
   

5. 正如 Jonathan Leffler 在评论中提到的那样，如果您以适当的间距编写语句，可读性将会提高。 else if(newnode->data>p->data) 很难阅读。与 else if (newnode->data > p->data) 相比更具可读性。当您编写语句if(p->next = NULL) 时，您的眼睛会很容易看到该错误。显而易见的是if(p->next == NULL)。

---

释放树有点棘手，因为您必须始终执行后序遍历。您需要先释放孩子，然后再释放父母。否则会出现内存泄漏。

1。之前我提到fprintf(stderr,...) Basile Starynkevitch 使用perror，这是打印诊断错误消息的好选择。