Why are my or operators not working as intended? - c++

Question

My if statements are getting hit no matter the input. I can't wrap my head around why. Here is the code:
void Novice::selection()
{
char selection, shift;
cout << "Please select a section to run: A - Home Row, B - Bottom Row, C - Top Row, D - Pointer Fingers, E - Right Pinky;" << endl;
cin >> selection;
selection = toupper(selection);
if (selection != 'A' || 'B' || 'C' || 'D' || 'E') {
cout << "Invalid Input. Please select again" << endl;
cin >> selection;
}
if (selection == 'A' || 'B' || 'C') {
cout << "you're here" << endl;
}
If input is 'A', the first if statement triggers, if I then put in A again second if statement triggers as well. Any help would be appreciated.

Answer 1

This is not how logical operators work in c++. To compare against multiple values, you need to do:
if (selection == 'A' || selection == 'B' || selection == 'C') {
// ...
}
Note that your first if is incorrect, even if you use the fix above. If you check whether a value is not equal to several other values, this will always be true. That condition probably needs to be something like:
if (selection != 'A' && selection != 'B' &&
selection != 'C' && selection != 'D' && selection != 'E') {
// ...
}
Alternatively, for the first if condition, you can use a switch statement, like this:
switch ( selection )
{
case 'A':
case 'B':
case 'C':
case 'D':
case 'E':
break;
default:
cout << "Invalid Input. Please select again" << endl;
cin >> selection;
}