Question

If user key in letter like “a” , it will result in endless loop, the default: doesnt work.

How do i do exception handling so its will output error message instead of endless loop.

Thanks!

Below is my code:

done=false;
do
{
cout << "Please select the department: " << endl;
cout << "1. Admin " << endl;
cout << "2. HR " << endl;
cout << "3. Normal " << endl;
cout << "4. Back to Main Menu " << endl;
cout << "Selection: ";
cin >> choice;



switch (choice) {
  case 1:
      department_selection = "admin";
    done=true;
    break;
  case 2:
      department_selection = "hr";
    done=true;
    break;
  case 3:
      department_selection = "normal";
    done=true;
    break;
  case 4:
      selection = "hr_menu";
    done=true;
    break;
  default:
    cout << "Invalid selection - Please input 1 to 3 only.";
    done=false;
        }
}while(done!=true);

Answer 1

The problem isn’t your switch statement but the fact that you don’t check if input operation actually succeeded. Always use input operations in some boolean context:

int choice = 0;
while (!(cin >> choice) && (choice < 1 || choice > 4)) {
    cout << "Invalid selection - Please input 1 to 3 only.\n";
    // reset error flags
    cin.clear();
    // throw away garbage input
    cin.ignore(numeric_limits<streamsize>::max(), '\n');

    // the above two statements prevent infinite loop due to
    // bad stream state
}

// proceed to switch statement

numeric_limits template is located in <limits> header.