[mnm2317]

Ok, I need a loop (while or if statement) to make sure 'T' or 'F' is the only input accepted inputted characters (case does not matter). Here's what I have:

if ( (p!='t') || (p!='T') || (p!='f') || (p!='F') ){
	cout << "Error: Must be 1 or 2";
}

As it acts right now, no matter what I input for 'p' it always displays the 'Error' message.

[PlasticineGuy]

Use && instead of ||.

By the way, an if statement is not a loop.

This post has been edited by PlasticineGuy: 05 March 2010 - 08:47 PM

[mnm2317]

This is a final line in my homework, and it determines if what's been inputted is logically correct. As you can read the user needs to input 'f' and 't' in that order in order to get the 'consistent' message. However if 'f' and 'f' are input, it gives the 'consistent' message, which is wrong. Also, I have test it, and if 't' and 't' are entered, it gives the 'inconsistent' message, so it is not simply giving the first message, so somewhere my logic is off I believe.

if( (p=='f') || (p=='F') && (q=='t') || (q=='T') ){
	cout << "This is consistent.";
}
else{
	cout << "This is inconsistent.";
}

[PlasticineGuy]

Don't double post.

[mnm2317]

Wow so simply but worked like a charm. Weird too, because I thought I tried that. Oh well.

Now here's a related question: Why won't it continue to loop if I input something other than T or F? Like when I input Y, it gives me the error message but still moves through the code. How do I keep it looping until I get T or F?

[NickDMax]

*** Merged ***

[PlasticineGuy]

Use a while loop and continually read into "p" until the same condition is true.

[mnm2317]

Ahh ok, the while loop worked perfectly! I think I'm starting to get my coding footing back....just a little.

Ok, I can see the call for the merging. However I don't want this part to get lost:

if( (p=='f') || (p=='F') && (q=='t') || (q=='T') ){
	cout << "This is consistent.";
}
else{
	cout << "This is inconsistent.";
}

Since fixing the other if statement by turning it into a while loop, this still does not work right. I can still get a 'consistent' message with inputs of 'F' and 'F'. But I should only get that message if 'F' and 'T' are inputted. Where am I going wrong?

[PlasticineGuy]

http://www.cpprefere...ator_precedence
&& takes precedence over ||.

[mnm2317]

Hey I was just on that cite, lol. Ok that makes sense, but now how to I get the code to read it the way I'm trying to write it?

[PlasticineGuy]

Use brackets to force the correct precedence.

if((x == 5 || x == 9) && (y == 1 || y == 19)) {
    //...
}

This post has been edited by PlasticineGuy: 05 March 2010 - 09:14 PM

[mnm2317]

Awesome! Thanks Guy, you're my personal tutor today, thx!

[PlasticineGuy]

It's always a good idea to use parentheses anyway to avoid possible bugs like this.

[mnm2317]

Yea, I'm going to work on doing that. I never really used them in the past, but I've never done this complicated of comparing like I did here.