Question

#include <stdio.h>
#include <stdlib.h>

typedef struct nod{
    int data;
    struct nod *left,*right;
}NOD;

NOD * generate(NOD * root)
{
    NOD *r,*p;
    int d=-1,value,line,position,i,f,v;
    if(root==NULL)
    {
        do{
            printf("Would you like to create the root node?\n\n1 - yes\n0 - no\n");
            scanf("%d",&d);
            switch(d)
            {
                case 1:
                    printf("Value=");
                    scanf("%d",&value);
                    root=add_root(value);
                    break;
                case 0:
                    return NULL;
                    break;
                default:
                    printf("Command unrecognized!\n");
                    break;
            }
        } while(d==-1);
        if(root!=NULL)
                printf("Root node successfully created!\n");
        else
            printf("Error: could not create root node!\n");
        d=-1;
        do{
            printf("Continue adding nodes?\n\n1 - yes\n0 - no\n");
            scanf("%d",&d);
            switch(d)
            {
                case 1:
                    printf("Insert the line and the position of the node you wish to add (root node has line=0, position=0)\nLine=");
                    scanf("%d",&line);
                    printf("Position ( less or equal with 2^$line-1 )=");
                    scanf("%d",&position);
                    printf("Value=");
                    scanf("%d",&value);
                    r=p=root;
                    for(i=line-1;i=0;i--)  
                    {
                        f=power(2,i);
                        if(position & f == f)   // if the i-(st,nd,rd,th) bit of "position" is 1, then (position & f == f) is true and *r will go right
                        {
                            p=r;
                            r=r->right;
                            v=1;
                        }
                        else
                        {
                            p=r;
                            r=r->left;
                            v=0;
                        }
                    }
                    if(v==0)
                        p=add_left(&r,value);
                    if(v==1)
                        p=add_right(&r,value);

                    break;
                case 0:
                    return root;
                    break;
                default:
                    printf("Command unrecognized!\n");
                    break;
            }
        } while(d==-1);
    }
    else
    {
        ...
    }

NOD * add_left(NOD **p,int value)
{
    NOD * r;
    r=malloc(sizeof(NOD));
    r->data=value;
    r->left=NULL;
    r->right=NULL;
    (*p)->left=r;
    return r;
}

NOD * add_right(NOD **p,int value)
{
    NOD * r;
    r=malloc(sizeof(NOD));
    r->data=value;
    r->left=NULL;
    r->right=NULL;
    (*p)->right=r;
    return r;
}

NOD * add_root(int value)
{
    NOD * x;
    x=malloc(sizeof(NOD));
    x->data=value;
    x->left=NULL;
    x->right=NULL;
    return x;
}

}
int main() {
    NOD *root=NULL;
    root=generate(root);
    return 0;
}

我尝试创建一个创建二叉树的程序，但我一直收到SIGSEGV Segmentation故障，我不明白为什么。你能告诉我我做错了吗？

if(position & f == f)   // if the i-(st,nd,rd,th) bit of "*position*" is 1,
                        // then (position & f == f) is true and *r will go right

您如何看待这部分？

line 是树的级别（root有行= 0）
位置是节点从左到右的位置（根位置= 0）

Answer 1

你已经加入了一个有问题的部分：

if(position & f == f)

==的优先级高于&，因此会被解析为

if(position & (f == f))

与if (position & 1)相同，而不是您想要的。

此外，您的循环条件错误

for(i=line-1;i=0;i--)

测试应该是i >= 0，否则循环永远不会被执行（i = 0是一个评估为0的赋值。）

如果这些是固定的并且循环被执行，在第一次迭代r是空指针之后，那么下一次循环迭代会导致r = r->right;中的崩溃，或者，如果循环只迭代一次， add_left(&r,value);（或add_right）在尝试访问其left（resp。right）指针的倒数第二行上取消引用nullpointer：

NOD * add_left(NOD **p,int value)
{
    NOD * r;
    r=malloc(sizeof(NOD));
    r->data=value;
    r->left=NULL;
    r->right=NULL;
    (*p)->left=r;          // *p == NULL here
    return r;
}

Answer 2

在这一行

for(i=line-1;i=0;i--)

我认为你的意思是

for(i=line-1;i!=0;i--)

否则循环将不会执行（i=0计算为false）。这导致v具有未定义的值（因为它从未初始化）。虽然这很可能不会导致段错误，但是当你覆盖根的左/右子树时会导致内存泄漏。

此外，默认switch分支不会更改d的扫描值，这意味着您继续使用NULL根（检查root != NULL的错误分支））不退出功能）

C - 分割故障中的二叉树

2 个答案: