Thread: Repeat Code

You're calling a function in the wrong way, you don't write it's return type when calling them. And don't call main().
What you wan't to do is wrap your code inside a loop that doesn't end until a == b.

You are not supposed to recursively call int main. Instead make a while loop like this

Code:

while (b != a)
{
     check too high/low
}
win();

He means the (a < b) and (a > b) if-statements that prints out whether the number is too high or too low.

I *think* I get what you're saying, but now the prompt is throwing error #E2120--Cannot call "main" from within the program in function main().

Also, it's saying function win() is undefined.....

You're not supposed to call main(). You will have to find another way around it e.g. a loop. This means you will have to put your first loop inside another loop that doesn't exist until the user says that he doesn't want to play anymore.
win() is not declared, you're calling the function before the compiler knows how it looks like so you have to put a prototype of it at the top.

Code:

int win (void);

I realize you've probably said this two or three times already, and I apologize, but I just don't get it. I understand that i need to write a while loop. The part i don't understand is how to get back to function main without calling it. I appreciate your patience in this.

GAAH! I changed a few things, and it worked, except now i'm getting a new error...the prompt tells me there is a syntax error on line 9:

int main()

is there??

Well, written explanations are usually harder to grasp than a visual demonstration so I fixed your program to a working state and put out some comments and I hope it will help.

Code:

#include <iostream>
#include <cstdlib>
#include <ctime>
#include <ctype>

using namespace std;

bool win (void); //Function prototype.

int main () //Remember to use good variable names if applicable!
{
	int number; //Originally "a". Also a local variable this time, don't use globals.
	int guess; //Originally "b".
	int guesscount; //Originally "x".

	srand(time(0)); //Give rand() a seed so we actually get a random number.
	do { //The main loop.
		guesscount = 1;
		number = rand()%999 + 1; //rand()%999 gives a number between 0 and 998 instead of 1 and 999.
		cout << "See if you can crack the code!" << endl;

		do { //The other loop.
			cout << "Guess #" << guesscount++ << ": " << flush; //Notice the ++ after guesscount.
			cin >> guess;
			cin.ignore();

			if (guess > number)
				cout << "Too high, try again..." << endl;
			else if (guess < number)
				cout << "Too low, try again..." << endl;
		} while (guess != number);
	} while (win()); //If the user wants to continue win() returns true and the loop continues.
}

bool win (void)
{
	char answer; //Originally "y".

	cout << "Correct!" << endl //I am only making one call to cout here.
	     << "Try to break another code? Y or N: " << flush; //If I'm not ending a cout with endl then I end it with flush.
	cin >> answer;

	if (tolower(answer) == 'y')
		return true;
	//"else"
		return false;
}