C++非void函数实现迷你计算器输出异常求助

你的代码输出异常值的核心原因有三个：

1. 未接收函数返回值：ArithmeticOperation是返回计算结果的非void函数，但你在main里调用时仅执行函数，未将返回值赋值给result变量。
2. 变量未初始化：main中的result是局部变量，未初始化时会保留内存中的随机垃圾值，直接输出就会出现类似4.68151e-310的异常值。
3. 多余的传值参数：函数里的result参数是传值传递，你在函数内修改的只是该参数的副本，对main里的result完全无影响，这个参数本身就是多余的。

修复步骤 #

• 移除ArithmeticOperation函数中的result参数，直接在函数内部计算并返回结果。
• 在main函数中，将ArithmeticOperation的返回值赋值给result。
• （可选）添加除数为0的判断避免运行时错误；将运算符类型从string改为char，防止输入多字符的无效运算符。

修复后的代码 #

#include <iomanip>
#include <iostream>

using namespace std;

double ArithmeticOperation(float x, float y, char opt)
{
    if (opt == '+') {
        return x + y;
    } else if (opt == '-') {
        return x - y;
    } else if (opt == '*') {
        return x * y;
    } else if (opt == '/') {
        if (y == 0) {
            cerr << "Error: Division by zero!" << endl;
            return 0; // 可根据需求调整错误处理逻辑
        }
        return x / y;
    } else {
        cerr << "Error: Invalid operator!" << endl;
        return 0;
    }
}

int main()
{
    char opt;
    float x, y;
    double result;

    cout << "Enter an Arithmetic Operator (+,-,*,/): ";
    cin >> opt;

    cout << "Input first number: ";
    cin >> x;

    cout << "Input second number: ";
    cin >> y;

    result = ArithmeticOperation(x, y, opt);
    cout << "The answer is: " << result << endl;

    return 0;
}

关键修改说明 #

• 移除函数中多余的result参数，直接返回计算结果，贴合非void函数的设计逻辑。
• main函数通过result = ArithmeticOperation(...)接收返回值，确保输出正确计算结果。
• 新增无效运算符和除数为0的错误处理，提升程序健壮性。
• 将opt改为char类型，匹配运算符的单一字符特性，避免输入多字符的异常情况。

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