Jumping in C++ Chapter 5 Question 2

Hello again. Thanks for the help on Question 1 in Jumping in C++ but now my program won’t work for question 2. This was my program.

Code:

---

---

Code:

---

#include <iostream>
#include <string>


usingnamespacestd;


int main()


{
    string program;
   

while(true)
    {
cout<<"Choose a following program:\n";
cout<<"Minecraft.\nSkype.\nSkyrim.\n";
        cin>>program;
       

if(program=="Minecraft"||"Skype"||"Skyrim")
        {
            cout<<"You chose: "<<program<<".\n";
            break;
        }
       

        else
        {
            continue;
        }
    }
}




---

But whenever I don’t type in a opinion from the menu it doesn’t reprint the menu, it says "You chose:” and then whatever the user input was.


Can you please tell me why it won;t work and how to fix it?


Thanks.

Line 22 does not compare program with the three stings "Minecraft", "Skype", and "Skyrim".

The expression ``program=="Minecraft"||"Skype"||"Skyrim"'' is actually equivalent to (additional brackets and text added for clarity) to ``(program=="Minecraft") || ("Skype" != NULL || ("Skyrim" != NULL)''

String literals like "Skype" are never equal to the NULL pointer, so your condition is always true.

If you want to test if program is equal to one of "Minecraft", "Skype", or "Skyrim", the test you need to do is ``(program == "Minecraft") || (program == "Skype") || (program == "Skyrim")'' (again, I've added brackets for clarity onlu- they're not technically required unless explaining problems like this to newbies).

Thank you so much!!!