Basic Do While Loop

[AphexBravia]

Hey, I'm trying to write some basic Do while loops. Have done a couple this morning and all worked fine, but for some reason I cant get this one working properly.. I just cant figure out why I'm sure its set up exactly as my previous ones. Please someone help me I don't really like do while loops but I'm trying to understand them.

#include <iostream>
#include <string>

using namespace std;

int main(){
	string choice;

	do{
		cout<<"Please select one of the following: "<<endl;
		cout<<"1. Steak"<<endl;
		cout<<"2. Pork"<<endl;
		cout<<"3. Fish"<<endl<<endl;
		cin>>choice;
		cout<<endl;
	} while ((choice!="Steak") || (choice!="Pork") || (choice!="Fish"));
	cout<<"Great choice!";
}

Thanks

[coder777]

your condition on line 16 is always true.

try this:
} while (!((choice=="steak") || (choice=="pork") || (choice=="fish")));

or this:
} while ((choice!="steak") && (choice!="pork") && (choice!="fish"));

[rihardmarius]

if you want the code to continue asking for a dish until one of the dishes is selected you should change the || for && (or "and")

while ((choice!="steak") and (choice!="pork") and (choice!="fish"));

[AphexBravia]

Ahh Thanks coder777. I thought it was just going to be something stupid like that I just couldn't figure it out.. Looks like I'm going to have to practice the difference between my &&s and ||s.

Cheers mate.

[rihardmarius]

use "or" instead of "||" and "and" instead of "&&"

it will be easier for you

[AphexBravia]

^ Cheers mate, ill try that from now on.