我怎样才能做到这一点？有没有办法优化它？

Question

Closed. This question needs to be more focused. It is not currently accepting answers.

---

想要改进这个问题吗？通过编辑这篇文章来更新问题，使其仅关注一个问题。

2 年前已关闭。

Answer 1

在 switch 语句外部定义 top1。您定义的全局 top1 没有您期望的值，因为您在每个 case 语句内定义了一个新的 top1。您的编译器应该对此发出警告（实际上，我认为在这种情况下这应该是一个错误）。

此外，由于 num1 到 num4 都是浮点数，因此您的 top1 和 top2 也应该是 float< /code>s，而不是 ints

有效地，从全局变量中删除 int top1，然后在 main 中：

float top1 = 0;
switch ( operatr1 ) {
    case '+':
        top1 = num1 + num2;
        break;
    // ... continue with other cases here
}

然后，对于第二个 switch语句，top1 将可见并具有正确的值。

Answer 2

您可以使用一个函数：

int Evaluate(const char operator,
             const int  num1,
             const int  num2)
{
    bool is_valid_operator = true;
    int result = 0;
    switch (operator)
    {
        case '+':
            result = num1 + num2;
            break;
        case '-':
            result = num1 - num2;
            break;
        case '/':
            //  Note: integer division.
            result = num1 / num2;
            break;
        case '*':
            result = num1 * num2;
            break;
        default:
            result = 0;
            is_valid_operator = false;
            break;
    }
    if (is_valid_operator)
    {
        std::cout << "Result of " << num1 << " " << operator << " " << num2 << "\n";
        std::cout << result << "\n";
    }
    return result;
}

您的 main 将被简化为：

//...
int top1 = Evaluate(operatr1, num1, num2);
int top2 = Evaluate(operatr2, top1, num3);