ending a loop correctly

[suds]

I need some assistance on how to make this loop end with this
"Sorry no good" or "See you later"

It has several function calls in the middle (that I have taken out) to save
some space.


Every thing I try it doesn't end the loop but goes right back into it...

int main()
{
		// declare variables
	char something' ';
	string something = "";
	int days = 0;
	double something = 0.0;
	
	cout << "Do you want to do this?: Y/N " << endl;
	cin >> doThis;	

		{	   	      
 	 if  (doThis!= 'Y' && doThis != 'N')           
		 cout << "Sorry no good" << endl;
    }
	
	while (doThis == 'Y') 
	{
	// declare variables
	char something = ' ';
	string something = "";
	double something = 0.0;


					
		 function calls;		
	
	
	 
	cout << "Do you want to something else?: Y/N " << endl; 
	cin >> doThis;
	{	   	      
 	 if  (doThis!= 'Y' && doThis != 'N')           
		 cout << "Sorry no good" << endl;
    }
} // end while
          cout << "See you later" << endl;
    
 
 system("pause");

	}  //end of main 

[Thumper]

Try

if(doThis != 'Y' && doThis != 'N')
{
    cout << "Sorry no good" << endl;
    break;
}

that will end the while loop, if that's what you were aiming for.

[suds]

that did work but now it goes to that and doesn't allow the loop to continue if the
cout << "Do you want to do this?: Y/N " << endl; equals = Y

I want the loop to continue if Y, to go to "sorry no good" if !=Y && !=N
and "see you later" if N.

Make sense?

That's where I'm having the issues

[suds]

I think I got it but now I think I have un-neccessary code...
but I did this at the end

if(doThis == 'Y')
continue;

else if (doThis != 'Y' && doThis != 'N')
{
    cout << "Sorry no good" << endl;
    
else if    (doThis == 'N' )
cout << "later" << endl;
    break;
}

[rocketboy9000]

Your brackets are wrong, it should be:

if(doThis == 'Y')//one statement, no brackets
    continue;
else if (doThis != 'Y' && doThis != 'N')//one statement, no brackets
    cout << "Sorry no good" << endl;  
else if (doThis == 'N' ){//two statements, use brackets!!
    cout << "later" << endl;
    break;
}

Some people even use brackets for one statement, because it saves them work later on when they add more.