修复表达式评估 - NoSuchElementFound异常

时间:2014-04-29 02:58:14

标签: java parsing stack expression evaluate

我的任务是评估从文件读入的表达式,例如(4-7)/ 2 + 6< 3 *(1-9)。

<和>分别表示最小值和最大值,并且优先级高于/和*。

但是,在尝试计算表达式时,我得到一个NoSuchElementFound异常,指向我在evaluate方法结束时返回的堆栈。我假设这意味着堆栈是空的。我无法查看问题所在。这是我的评估代码。重要的是,我已经实现了自己的堆栈类(两个字符串),并且没有使用泛型。

/**
 * Evaluates the expression and updates textArea appropriately
 */
private String evaluateLine()
{
    char token;
    String[] tokens = new String[expression[0].length()];
    for(int i = 0; i < expression[0].length(); i++)
    {
         token = expression[0].charAt(i);
         tokens[i] = String.valueOf(token);
    }

    OperatorStack operators = new OperatorStack();
    IntegerStack integers = new IntegerStack();

    for(int i = 0; i < tokens.length; i++)
    {
        if(isInteger(tokens[i]))
        {
            integers.push(tokens[i]);
        }
        else if(tokens[i] == "(")
        {
            operators.push(tokens[i]);
        }
        else if(tokens[i] == ")")
        {
            while(operators.top() != "(")
            {
                integers.push(applyOp(operators.pop(), integers.pop(), integers.pop()));
                operators.pop();
            }
        }
        else if(tokens[i] == "+" || tokens[i] == "-" || tokens[i] == "*" || tokens[i] == "/" 
                || tokens[i] == "<" || tokens[i] == ">")
        {
            while(!operators.isEmpty() && hasPrecedence(tokens[i], operators.top(), operators.top()))
            {
                integers.push(applyOp(operators.pop(), integers.pop(), integers.pop()));
            }
            operators.push(tokens[i]);
        }
    }

    while(!operators.isEmpty())
    {
        integers.push(applyOp(operators.pop(), integers.pop(), integers.pop()));
    }

    return integers.pop();
}

public static boolean hasPrecedence(String op1, String op2, String op3)
{

    if(op2 == "(" || op2 == ")")
    {
        return false;
    }

    if((op1 == "<" || op1 == ">") && (op2 == "*" || op2 == "/") && (op3 == "+" || op3 == "-"))
    {
        return false;
    }
    else
    {
        return true;
    }
}

public static String applyOp(String op, String b, String a)
{
    switch (op)
    {
    case "+":
        return Integer.toString(Integer.parseInt(a) + Integer.parseInt(b));
    case "-":
        return Integer.toString(Integer.parseInt(a) - Integer.parseInt(b));
    case "*":
        return Integer.toString(Integer.parseInt(a) * Integer.parseInt(b));
    case "/":
        if(Integer.parseInt(b) == 0)
        {
            throw new
            UnsupportedOperationException("Cannot divide by 0");
        }
        return Integer.toString(Integer.parseInt(a) / Integer.parseInt(b));
    case "<":
        if(Integer.parseInt(a) < Integer.parseInt(b))
        {
            return b;
        }
        else if(Integer.parseInt(b) < Integer.parseInt(a))
        {
            return a;
        }
    case ">":
        if(Integer.parseInt(a) > Integer.parseInt(b))
        {
            return a;
        }
        else if(Integer.parseInt(b) > Integer.parseInt(a))
        {
            return b;
        }
    }

    return Integer.toString(0);

}

public boolean isInteger(String str)
{
    return str.matches("0123456789");
}

1 个答案:

答案 0 :(得分:0)

很难找到确切的问题,因为你没有包含足够的代码来编译你的例子,但有一件事我注意到你的isInteger方法不起作用。当我将输入运行到您的方法时,它为整数生成一个空堆栈。请改用:

public boolean isInteger(String str) {
    try {
        Integer.parseInt(str);
        return true;
    } catch (NumberFormatException ex) {
        return false;
    }
}

或者我认为您正在尝试使用正则表达式来确定字符串是否包含整数。为此你需要这样做:

public boolean isInteger(String str) {
    return str.matches("[0-9]+");
}

使用isInteger的这些实现之一,当我用Stack<String>集合替换堆栈时,我能够运行代码。希望有所帮助。