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

如评论中所述，提出

if(ch == ' ') continue;

直接

char ch = input.charAt(i);

这将解决空白问题。

另一个问题是

Pstack.push(result);

执行每时间，而不仅仅是在运算符已被评估的情况下。

所以这句话应该是“其他”案件中的最后一句。

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

e.g。如果你有后缀表达式12-（应该评估为-1）， 2位于堆栈顶部，1位于堆栈之后。在当前的解决方案中，结果将是2 -1，因为以错误的顺序检索参数。

所以

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

应该更正

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

Java Postfix计算器错误

1 个答案: