web-dev-qa-db-ja.com

このエラーを解決する方法：ケースラベルへのジャンプが初期化を超えています

計算機コードに次のエラーがあり、修正方法がわかりません。アドバイスがあれば参考にしてください。

エラー：エラー：ケースラベルにジャンプ[-fpermissive] |エラー：「int sum」の初期化を超えます|エラー： 'exit'はこのスコープで宣言されませんでした|

コード：

#include <iostream>
#include <cmath>
using namespace std;         
void display_menu(); 
int get_menu_choice();
void get_two_numbers(int &a, int &b);
int add(int a, int b);
int subtract(int a, int b);


int main()
 {
 int choice;

  do
   {
    display_menu();
    choice = get_menu_choice();
    int x, y;
    switch (choice)
    {
        case 1: get_two_numbers(x, y);
                int sum = add(x, y);
                cout << x << " + " << y << " = " <<  sum << endl;
                break;
        case 2: get_two_numbers(x, y);
                int diff = subtract(x, y);
                cout << x << " - " << y << " = " <<  diff << endl;
                break;
        default:;
    }

     } while (choice != 3);

     cout << "Good bye...now." << endl;

     return 0;
       }


 void display_menu()
  {
   cout << endl;
   cout << "Simple Calculator Menu" << endl;
   cout << "----------------------" << endl;
   cout << " 1. Addition (+) " << endl;
   cout << " 2. Subtraction (-) " << endl;
   cout << " 3. Quit to exit the program" << endl;
   cout << endl;
  }

 int get_menu_choice()
  {
   int choice;
   cout << "Enter your selection (1, 2, or 3): ";
   cin >> choice;

  while(((choice < 1) || (choice > 3)) && (!cin.fail()))
   {
    cout << "Try again (1, 2, or 3): ";
    cin >> choice;
    }
  if (cin.fail())
    {
      cout << "Error: exiting now ... " << endl;
      exit(1);
     }
   return choice;
    }

 void get_two_numbers(int &a, int &b)
  {
    cout << "Enter two integer numbers: ";
    cin >> a >> b;
  }


 int add(int a, int b)
  {
   return (a + b);
  }

 int subtract(int a, int b)
  {
    return (a - b);
  }
c++variables
31
user3555274

囲みスコープを作成せずに、caseステートメント内で新しい変数を宣言しています。

switch (choice)
{
    case 1: get_two_numbers(x, y);
            //* vv here vv *
            int sum = add(x, y);
            //* ^^ here ^^ */
            cout << x << " + " << y << " = " <<  sum << endl;
            break;
    case 2: get_two_numbers(x, y);
            //* vv here vv */
            int diff = subtract(x, y);
            //* ^^ here ^^ */
            cout << x << " - " << y << " = " <<  diff << endl;
            break;
    default:;
}

修正方法は次のとおりです。

switch (choice)
{
    case 1:
        {
            get_two_numbers(x, y);
            int sum = add(x, y);
            cout << x << " + " << y << " = " <<  sum << endl;
        }
        break;
    case 2:
        {
            get_two_numbers(x, y);
            int diff = subtract(x, y);
            cout << x << " - " << y << " = " <<  diff << endl;
        }
        break;
    default:
        break;
}

もちろん、括弧とインデントの正確なフォーマットはあなた次第です。

56
kfsone

スイッチの「ケース」はスコープを作成しないため、エラーが示すように、選択肢が1でない場合、「sum」の初期化を飛び越えています。

スイッチの外側でsumとdiffを宣言するか、ケースごとに{}を使用してブロックを作成する必要があります。

20
Andrew McGuinness

ケース内ではなく、switchステートメントの外で変数を宣言する必要があります。特に、sumとdiffはコードに問題があります。これらの変数がswitchステートメントの外部で初期化されると、値を設定できます。

4
Nassim