Question

我已经为严格的二进制搜索树编写了这段代码。（如果二叉树中的每个非叶节点都有非空左右子树，则该树称为严格二叉树。或者，换句话说，严格二叉树中的所有节点都是零或两个度一个严格的二叉树，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，然后为其分配新分配的节点的地址。这解决了这个问题。

很少 -

检查malloc的返回值。如果失败，它将返回NULL。 <子> 1

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

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

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

启用编译器警告-Wall -Werror。如果你这样做，编译器会清楚地向你显示问题。

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

另外一件事是检查scanf的返回值。

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

正如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这是打印诊断错误消息的好选择。

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

1 个答案: