C++求和程序输出异常求助：问题分析与高效代码优化

问题描述 #

我编写了一个接收两个用户输入并计算数值之和的C++程序，但实际输出与预期不符，出现了异常大的数值。

原始代码 #

#include <iostream>
#include <string>

using namespace std;

int main()
{
    int nm1, nm2;
    cout<<"Your numbers please";

    cin >> nm1, nm2 ;  
    cout << nm1 + nm2;

    return 0 ;
}

程序运行结果 #

PS E:\programmes\C programming> cd "e:\programmes\C programming" ; if ($?Your numbers please1 2
4201052
PS E:\programmes\C programming> cd "e:\programmes\C programming" ; if ($?) { g++ c++intros2.cpp -o c++intros2 } ; if ($?) { .\c++intros2 }
Your numbers please1
4201052

要求：分析代码问题，提供更高效的实现方案，不要参考分步提示输入的代码（示例如下）：

#include <iostream>
#include <string>

using namespace std;

int main()
{
    int nm1, nm2;
    cout<<"Your numbers please";

    cout << "Your 1st number please";
    cin >> nm1 ;
    cout << "Your 2nd number please";
    cin >> nm2 ;
    cout << nm1 + nm2;

    return 0 ;
}

问题根源 #

代码的核心错误在这一行：cin >> nm1, nm2 ;
这里误用了逗号运算符。C++的逗号运算符会依次执行左右两个表达式，但仅返回右侧表达式的结果。也就是说，这行代码只会执行cin >> nm1读取第一个输入值，而nm2完全没有被赋值——它是未初始化的局部变量，内存中存储的是随机垃圾值。当计算nm1 + nm2时，垃圾值与输入的nm1相加，就会出现异常大的随机数值。

高效实现方案 #

下面的代码支持一次性输入两个数值（用空格、回车等分隔符均可），同时加入输入合法性校验，避免非法输入导致的错误：

#include <iostream>

int main()
{
    int num1, num2;
    std::cout << "请输入两个整数（用空格分隔）：";
    
    // 同时读取两个输入，并校验输入是否合法
    if (std::cin >> num1 >> num2) {
        std::cout << "两数之和为：" << num1 + num2 << std::endl;
    } else {
        std::cerr << "输入无效，请输入合法整数！" << std::endl;
        return 1; // 返回非0值表示程序异常退出
    }
    
    return 0;
}

优化说明 #

• 修正输入读取方式：用std::cin >> num1 >> num2连续读取两个整数，这是C++标准的多输入读取方式，会自动跳过输入中的空白字符（空格、换行、制表符等）。
• 移除不必要的头文件：代码未用到字符串相关功能，无需引入<string>。
• 避免using namespace std：虽然简化代码，但大型项目中易引发命名冲突，直接使用std::前缀更规范。
• 增加输入校验：通过判断std::cin的状态，确保输入为合法整数，避免程序在非法输入时出现未定义行为。

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