Java 后缀计算器错误

Question

我正在使用堆栈在 java 中创建一个后缀计算器。我已经写好了主要部分，但是我遇到了一些问题。对于初学者，我必须考虑空格，我不确定如何使用我当前的设置来做到这一点。我不确定这是否会完全修复程序，但这将是一个开始。任何帮助将不胜感激。

 import java.util.Scanner;
 import java.util.Stack;
 public class Postfix 
{
public static void main(String[]args)
{
    Scanner sc = new Scanner(System.in);
    System.out.println("Enter a Postfix expression");
    String input = sc.nextLine();
    Stack<Integer> Pstack = new Stack<Integer>();
    int result = 0;
    for(int i=0; i<input.length();i++)
    {
        char ch = input.charAt(i);

        if(ch>='0' && ch<='9')
        {
            Pstack.push((int)(ch-'0'));
        }
        else
        {
            int o1 = Pstack.pop();
            int o2 = Pstack.pop();

            switch(ch)
            {
            case '+':result=o1+o2;
            break;
            case '-':result=o1-o2;
            break;
            case '/':result=o1/o2;
            break;
            case '*':result=o1*o2;
            }
        }
        Pstack.push(result);
    }
    result = Pstack.pop();
    System.out.println("result: "+ result);
   }
}

Answer 1

如cmets中所述，放

if(ch == ' ') continue; 

紧接着

char ch = input.charAt(i);

这将解决空格问题。

另一个问题是

Pstack.push(result);

每次都会执行，而不仅仅是在对运算符进行评估的情况下。

所以这个语句应该是“else”情况下的最后一个语句。

您还必须更正从堆栈中弹出参数的方式。

例如如果你有后缀表达式 12- （应该计算为 -1）， 2 在堆栈顶部，1 在后面。 在当前解决方案中，结果将是 2 -1，因为以错误的顺序检索参数。

所以

int o1 = Pstack.pop();
int o2 = Pstack.pop();

应该改成

int o2 = Pstack.pop();
int o1 = Pstack.pop();