为什么用于表达式计算的C++哨兵循环一直返回错误计算结果？

问题根因 #

原代码的运算逻辑顺序错误，最后一个输入的数字没有参与计算：

• 代码默认将读取的运算符作用于前一个读取的数字，但当读取到哨兵值=时，会直接跳出循环，=前面的最后一个数字还没来得及参与运算
• 初始逻辑也不符合表达式计算的常规逻辑：第一个输入的数字应该作为计算初始值，后续的运算符+数字组合才是对初始值的累加/累减操作

修复后代码 #

// this program finds the value of the expression-
// defined by the sequence of operands and operators
// the user enters a series of numbers and + or - then,
// " = " when finished

#include <cstdlib>
#include <iostream>
using namespace std;

int main()
{
    float total = 0;
    float num;
    char op;
    
    cout << "Enter a number followed by an operator and '=' to get the result:\n"; 
    // 第一个数字直接赋值给初始总值
    cin >> total;
    cin >> op;
    
    while (op != '=')
    {
        // 先读取运算符后面的数字
        cin >> num;
        // 用当前运算符对总值做运算
        if (op == '+')
        {
            total = total + num;
        }
        else if (op == '-')
        {
            total = total - num;
        }
        // 读取下一个运算符
        cin >> op;
    }
    cout << "\nThe total value of the expression is " << total << endl;
    return 0;
}

核心修改点 #

• 把第一个读取的数字直接赋值给total作为初始值，替代原来的0初始值
• 调整循环内的执行顺序：先读取运算符对应的操作数，做完运算后再读取下一个运算符，保证所有操作数都能参与计算
• 把两个独立的if判断改成else if，避免不必要的判断逻辑

内容的提问来源于stack exchange，提问作者nim123