0

我想评估一个二叉表达式树。这是我到目前为止收集的代码。当我评估它时,我正在破坏树的节点。但问题是,我认为,在递归过程中,它会寻找它没有的数据。它只是继续返回 0。

void calc(bnode *&b)
{
    bnode *c;
    int m;
    switch (b->data.ch)
    {
        case '+':m=b->lchild->data.in+b->rchild->data.in;
                 break;
        case '-':m=b->rchild->data.in-b->lchild->data.in;
                 break;
        case '*':m=b->lchild->data.in*b->rchild->data.in;
                 break;
        case '/':m=b->lchild->data.in/b->rchild->data.in;
                 break;
        case '%':m=b->lchild->data.in%b->rchild->data.in;
                 break;
    }

     c=new(bnode);
     c->lchild=NULL;
     c->rchild=NULL;
     c->tag=1;
     c->data.in=m;
     b=c;
}

int eval(bnode *b)
{
    if (b->tag==1)
    return b->data.in;
    else
    {
        if (b->lchild->tag==0)
           eval(b->lchild);
        if (b->rchild->tag==0)
           eval(b->rchild);
        if (b->lchild->tag==1&&b->rchild->tag==1)
           calc(b);
    }
}

我使用的结构是

union un
{
    int in;
    char ch;
};

struct bnode{
    bnode *lchild;
    un data;
    int tag;
    bnode *rchild;
};
4

1 回答 1

2

功能eval()坏了。在“else”分支中,您永远不会返回任何值,这是未定义的行为。

[编辑] eval 函数应该改成这样。

int eval(bnode *b)
{
    if (b->lchild && b->lchild->tag == 0)
        eval(b->lchild);
    if (b->rchild && b->rchild->tag == 0)
        eval(b->rchild);
    if (b->lchild && b->rchild && b->lchild->tag == 1 && b->rchild->tag == 1)
        calc(b);

    if (b->tag == 1)
        return b->data.in;
    else
        throw "Evaluation error";
}

你也有内存泄漏,因为bnode对象永远不会被删除。

于 2013-09-01T17:30:38.077 回答