如何正確測試后綴表達式是否有效？

Question

我被分配編寫一個程序，該程序使用堆棧評估后綴表達式。

我編寫了該程序，它似乎在大多數情況下都有效，但是它在確定表達式是否有效方面存在問題。

以下是我已完成的基本步驟：

1. 要求用戶輸入表達式（存儲為字符串）
2. 遍歷每個char並確定char是操作數，運算符，空格還是無效字符
3. if（char ==操作數）壓入堆棧
4. if（char ==運算符）彈出兩次並執行算術運算，然后將結果壓入堆棧
5. 在循環外，if（！stack.empty（））結果== stack.top，stack.pop
6. 否則無效的表達

因此，如果堆棧已經為空，則上面的方法效果很好，但是如果堆棧上有更多的操作數，則結果只會打印出來。 這顯然是不正確的，因為如果堆棧上還有多個操作數，則它應該是無效的表達式。

我在想我應該做while(!stack.empty()) result = stack.top, stack.pop()但是，這仍然會有同樣的問題。

有人可以告訴我如何正確測試嗎？

碼：

int main() 
{
    string expression;
    char response;
    int result = -1;        //result of expression. Initialized to -1
    Stack stack;

    printMenu();

    do {
        cout << "Would you like to enter an expression? (y / n)" << endl;
        cin >> response;
        response = toupper(response);

        switch(response) 
        {
            case 'Y':
                //needed due to new line
                cin.ignore();
                doWork(stack, expression, result);
                break;
            case 'N':
                cout << "Exiting program." << endl;
                break;
            default:
                cout << "Invalid response. Try again." << endl;
        }

    } while(response != 'N');

    return EXIT_SUCCESS;
}

doWork（不用擔心，它將被重命名）功能：

void doWork(Stack stack, string expression, int result)
{
    cout << "Enter a PostFix expression: ";
    getline(cin, expression);

    for(int i = 0; i < expression.size(); i++)
    {
        if(expression[i] == ' ') {
            //do nothing
        } else if(isInteger(expression[i])) {
           stack.push(convertChar2Int(expression[i]));
        } else if(isOperator(expression[i])) {
           // pop last 2 ints from stack and do arithmetic on them 
           int a = stack.top();
           stack.pop();
           int b = stack.top();
           stack.pop();
           // push result onto stack 
           stack.push(calculate(a, b, expression[i]));
        } else {
           //cerr : enter different expression
           cout << expression[i] << " is an invalid character." << endl;
        }
    }

    //the result should be the top of stack
    // THIS IS WHERE MY ISSUE IS
    if(!stack.empty()) {
        result = stack.top();
        stack.pop();
    } else {
        cout << "Invalid expression." << endl;
    }

    cout << "Result: " << result << endl;
}

Answer 1

要驗證您的表達，您需要測試多個條件。

• 在表達式驗證期間，堆棧永遠不應為空。 也就是說，在將參數彈出給任何運算符時，您不應獲取stack.empty() 。
• 完成對表達式的求值后，堆棧上應該恰好有一個元素。 您可以通過以下過程確定這一點（假設您的Stack沒有返回當前堆棧深度的方法）：
  1. 首先檢查堆棧是否為空。 如果是這樣：錯誤。
  2. 接下來，彈出堆棧頂部，作為可能的結果 。 放在一邊。
  3. 現在，再次檢查堆棧是否為空。 如果不為空 ：錯誤。
  4. 最后，如果到這里沒有錯誤，則將潛在結果作為最終結果返回。

那應該做。