表达树给出了错误的答案

时间:2011-04-20 20:06:41

标签: c++ expression-trees

修改

这是作业,所以请不要直接编码。只是提示,谢谢!

我正在开发一个项目,它将使用表达式树来派生各种事物,然后对它们执行操作。现在我不太担心派生部分,我只想让操作部分失效。

我正在使用的表达式树代码适用于整数但是一旦输入“x”或任何其他变量,我的答案就错了。我的程序使用后缀表达式字符串...下面是一个正确和错误的例子。

5 6 +返回11.正确

5x 6x +返回11.错误需要为11x

这是我的代码:

// This is the expression tree code I'm using
#ifndef EXPRNODE_H
#define EXPRNODE_H

#include <cstdlib>  // for NULL
using namespace std;

//====================================== class ExprNode
class ExprNode {
 public:
    ExprNode(char oper, ExprNode* left, ExprNode* right);
    ExprNode(int val);
    int eval() const; // Evaluate expr tree. Return result.

 private:
    char      _op;    // one of +, -, *, /, #
    int       _value; // integer value used for constants.
    ExprNode* _left;  // left subtree
    ExprNode* _right; // right subtree
};
#endif

//============================================= ExprNode constructor
// Constructs node for a binary operator.
ExprNode::ExprNode(char oper, ExprNode* left, ExprNode* right) {
    _op    = oper;
    _left  = left;
    _right = right;
}

//============================================== ExprNode constructor
// Constructs a node for an integer constant
ExprNode::ExprNode(int v) {
    _op    = '#';
    _value = v;
    _left  = NULL;
    _right = NULL;
}

//===================================================== ExprNode::eval
int ExprNode::eval() const {
    // Recursively evaluate expression tree and return result.
    int result;
    switch (_op) {
        case '+': 
                result = _left->eval() + _right->eval();
                break;
        case '-': 
                result = _left->eval() - _right->eval();
                break;
        case '*':
                result = _left->eval() * _right->eval();
                break;
        case '/':
                result = _left->eval() / _right->eval();
                break;
        case '#': 
                result = _value;  // an integer constant
                break;
     }
     return result;
}

bool isOperator (char operand)
{
    return operand == '+' || operand == '-' || operand == '*' || operand == '/' || operand == '^';
}

bool isNumber (char potentialNumber)
{
    return potentialNumber >= '0' && potentialNumber <= '9';
}

bool isX (char letter)
{
    return letter == 'x' || letter == 'X';
}

我不会包含从中缀到postfix的代码,因为它是不必要的(我认为)....接下来是表达式树和计算的代码

// the expression string is the postfix expression I returned previously
void expressionTree(string expression)
{
    string tempNum = "";
    string tempNum2 = "";
    int count = 1;
    int tempNumInt;
    int tempNum2Int;

    // creates a blank total value and blank numbers
    ExprNode* totalVal = new ExprNode('+', new ExprNode(0), new ExprNode(0));
    ExprNode* tNum;
    ExprNode* tNum2;

    // loop through the postfix expression
    for (unsigned int iterator = 0; iterator < expression.length(); iterator++)
    {
        if (isOperator(expression[iterator]))
        {
                    // Don't need to worry about at the moment
            if (expression[iterator] == '^')
            {
                // go to derivative later
            }
            else
            {
                if (count % 2 != 0)
                {
                    // we'll do different derivatives here.... for now just add, subtract, multiply, divide
                    totalVal = new ExprNode(expression[iterator], tNum, tNum2);
                }
                else if (count % 2 == 0 && expression[iterator] == '+' || expression[iterator] == '*')
                {
                    totalVal = new ExprNode(expression[iterator], tNum, totalVal);
                }
                else if (count % 2 == 0 && expression[iterator] == '-' || expression[iterator] == '/')
                {
                    totalVal = new ExprNode(expression[iterator], totalVal, tNum);
                }
            }
            count++;
        }
        if (isNumber(expression[iterator]) && count % 2 != 0)
        {
            tempNum += expression[iterator];
        }
        else if (isNumber(expression[iterator]) && count % 2 == 0)
        {
            tempNum2 += expression[iterator];
        }
        if (expression[iterator] == ' ' && count % 2 != 0)
        {
            tempNumInt = atoi (tempNum.c_str());
            tNum = new ExprNode(tempNumInt);
            tempNum = "";
            count++;
        }
        else if (expression[iterator] == ' ' && count % 2 == 0)
        {
            tempNum2Int = atoi (tempNum2.c_str());
            tNum2 = new ExprNode(tempNum2Int);
            tempNum2 = "";
            count++;
        }
        else if (expression[iterator] == ' ')
        {
            count++;
        }
    }
    cout << totalVal->eval() << endl;
}

我会尝试解释任何不清楚的事情。提前谢谢。

1 个答案:

答案 0 :(得分:1)

我没有指出确切的错误,但给你一个建议:int ExprNode :: eval()const不应该返回'int'。这还不足以处理变量结果,如“11x”(这不能用简单的int表示)。你必须创建自己的结构来存储结果的整数部分和变量部分(最后一个是可选的)。