URGENT - Need Help With A Switch (C++)

[dadasmithywinkle]

Ok, I'm using a switch but when running it, it shows all of the options, regardless of what's chosen.

 

Code:

 

 do{

    cout << "1. Show information on all letters." << endl;

    cout << "2. Show information on one letter." << endl;

    cout << "3. Quit the program." << endl;

    cout << "Enter your choice here: " << endl;

    cin >> num;

    switch(num){

    case 1: printInfoAllLetters(ar);

    case 2:

      cout << "Which letter are you interested in?: ";

      cin >> letter;

      printInfoOneLetter(letter,ar);

    case 3: cout << "Thank you for using the system. Good bye." << endl;

    default: cout <<"Invalid choice. Please try again." << endl;

    }

  }while(num <1 || num>3);

 

  return 0;

}

 

Output:

 

2. Show information on one letter.

3. Quit the program.

Enter your choice here: 

1

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

B 0  - 

Which letter are you interested in?: S

S 0  - 

Thank you for using the system. Good bye.

Invalid choice. Please try again.

[-username-]$ ./a.out

1. Show information on all letters.

2. Show information on one letter.

3. Quit the program.

Enter your choice here: 

2

Which letter are you interested in?: s

s 8  - xxxxxxxxxxxxxxxxxxxxxxxxxxxxx

Thank you for using the system. Good bye.

Invalid choice. Please try again.

[-username-]$ ./a.out

1. Show information on all letters.

2. Show information on one letter.

3. Quit the program.

Enter your choice here: 

3

Thank you for using the system. Good bye.

Invalid choice. Please try again.

 

 

 

So that's what happening. Any help?

[Nineshadow]

Add break after each case.

case 1: printInfoAllLetters(ar); break;    case 2:      cout << "Which letter are you interested in?: ";      cin >> letter;      printInfoOneLetter(letter,ar);    break;    case 3: cout << "Thank you for using the system. Good bye." << endl; break;    default: cout <<"Invalid choice. Please try again." << endl;

[dadasmithywinkle]

 

Add break after each case.

case 1: printInfoAllLetters(ar); break;    case 2:      cout << "Which letter are you interested in?: ";      cin >> letter;      printInfoOneLetter(letter,ar);    break;    case 3: cout << "Thank you for using the system. Good bye." << endl; break;    default: cout <<"Invalid choice. Please try again." << endl;

I'll try this. Thanks!