Simple program

I'm getting back into programming now that I have some stuff to do with it so I'm trying to regain my skills, and I can't figure out the problem with this. It is not supposed to keep looping if the user doesn't input 'y' or 'n'.

Code:

#include <iostream>
using namespace std;
int main(){
char holla;
do{
		cout<<"You wanna holla at the world?\n";
		cin>>holla;
	switch (holla){
		case 'y':
			cout<<"Hello World\n";
		break;
		case 'n':
			cout<<"Why the hell not?\n";
		break;
		default:
			cout<<"not valid yo\n";
		break;
}
}
while(holla!='y'||'n');
return 0;
}

Code:

while(holla != 'y' && holla != 'n')

I changed the code and it still continues to loop regardless of input. I want it to quit after only one of them input so I don't need the &&

i think i got it right i think.

Code:

while(holla =='y'|| holla == 'n');

Originally Posted by djhabeeb

i think i got it right i think.

Code:

while(holla =='y'|| holla == 'n');

No, that's the opposite of what he wants.

That says loop while it's 'y' or 'n' and he wants it to loop while it isn't 'y' or 'n'

Look at my post, you copied something wrong.

*sigh* no I don't want it to continue while y or n are input. I want it to quit after one is input, but still loop if there is no input or invalid input.

Edit: I have fixed it, but I didn't want to have to add the return 0's. I guess it will work.

Code:

#include <iostream>
using namespace std;
int main(){
char holla;
do{
		cout<<"You wanna holla at the world?\n";
		cin>>holla;
	switch (holla){
		case 'y':
			cout<<"Hello World\n";
				return 0;
		break;
		case 'n':
			cout<<"Why the hell not?\n";
				return 0;
		break;
		default:
			cout<<"not valid yo\n";
		break;
}
}
while(holla!='y'||holla!='n');
return 0;
}

no I don't want it to continue while y or n are input. I want it to quit after one is input, but still loop if there is no input or invalid input.

That is fine, and you can do it without the return statements. That just ends your program, it will hardly be useful to only figure out control statements that works by ending the program….

Think about it:

Code:

while(holla!='y'||holla!='n')

When you type in any other char both of those will be true and loop.
But if it is one, the other will always have to be true.
If it is ‘y’ then not ‘n’ has to be true.
If it is ‘n’ then not ‘y’ has to be true.


Slys code does exactly what you want, loops until a y or n is entered and stops then.

Originally Posted by Enahs

But if it is one, the other will always have to be true.
If it is ‘y’ then not ‘n’ has to be true.
If it is ‘n’ then not ‘y’ has to be true.

Wrong, I am afraid the ||(OR) operator constitutes that either condition can be true and it will accept.

Edit: OR I am not quite sure I understand you, but I don't need operators explained to me.

Heeeeyyyy.... jack ass.

Code:

while(holla != 'y' || holla != 'n')

Why don't you spend 2 seconds actually thinking about this. Loop while a variable doesn't equal 'y' or doesn't equal 'n'... Do you know some special thing that I don't that would make a variable equal 'y' and 'n' so this loop will exit? THIS WILL ALWAYS RETURN TRUE. I gave you the answer yesterday. It's been there for you to take but you continue to argue about nothing.

...it seems you did need operators explained to you.

Wrong, I am afraid the ||(OR) operator constitutes that either condition can be true and it will accept.

Ding ding ding

OR I am not quite sure I understand you, but I don't need operators explained to me.

Yes you do.
You have a character, one single character.

If it is NOT ‘y’ the loop will continue OR if it is not ‘n’ the loop will continue.
Please tell me how it is possible to make one character to be both ‘y’ and ‘n’. That way it will evaluate false for not ‘y’ and not ‘n’ and stop the loop.

Again, Sly’s code does what you stated you want it to do.