C++菜单式计算器加法菜单项输出随机值问题排查

核心问题原因及修复方式 #

• 作用域错误
  若在main_menu函数内定义num1、num2为局部变量，但求和逻辑写在其他独立函数中，且没有将两个变量作为参数传递给求和函数，求和函数内访问的是同名的未初始化局部变量，就会输出随机值。
  错误代码示例：

  void add() {
      int num1, num2; // 该局部变量和main_menu内的变量完全独立，未初始化
      cout << num1 + num2;
  }
  void main_menu() {
      int num1, num2;
      cin >> num1 >> num2;
      add(); // 未传递输入的两个参数
  }
  

  修复方式：将num1、num2作为参数传入求和函数，或定义为类成员变量（不推荐用全局变量，优先传参实现）。

• 执行顺序颠倒
  若先执行菜单选择逻辑，再执行num1、num2的输入代码，选择菜单项1的时候还未触发输入操作，变量自然未赋值。
  错误代码示例：

  void main_menu() {
      int choice, num1, num2;
      cin >> choice;
      switch(choice) {
          case 1: cout << num1 + num2; break; // 此时还未执行输入逻辑
      }
      cin >> num1 >> num2; // 输入逻辑写在了分支判断之后
  }
  

  修复方式：把输入num1、num2的逻辑放到菜单选择判断之前，或对应菜单项的分支内部。

• 输入流失败未处理
  如果用户输入的内容不是整数（比如输入了字母、特殊符号），cin会进入失败状态，不会给num1、num2赋值，变量会保留定义时的随机值。
  修复方式：输入后判断流状态，失败就提示用户重新输入：

  cin >> num1 >> num2;
  while(cin.fail()) {
      cout << "输入无效，请输入两个整数：";
      cin.clear();
      cin.ignore(1024, '\n');
      cin >> num1 >> num2;
  }
  

• 变量未初始化+分支未覆盖
  若定义num1、num2时没有给初始值，且输入逻辑被包裹在某个条件分支中，当分支未触发时，变量就不会被赋值。比如额外加了判断只有选菜单项2才输入数值，但选1的时候没走输入分支。
  修复方式：定义变量时可以先初始化默认值（比如int num1 = 0, num2 = 0），同时保证所有用到这两个变量的分支都会触发输入逻辑。

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