function return

[CASHOUT]

If I call a Play(); function from main, and that function returns a 1 for a win a and a 0 for a lose. How can I assign those results to a variable. The main program needs to recognize the result in order to print the proper message to the screen.

I'm assuming that I need to assign the returns from the Play function to a variable and assign that variable to the function call within main to be able to check the result and display a message.

result = Play();

[Salem]

Yes, that should work.

[CASHOUT]

Yes, that should work.

result = Play(answer);


int Play(int answer)
{	
	int numGuesses = 0;	
	int guess = 0;
	int result = 0;
	do
	{	
		guess = GetGuess();   
		
			CompareGuess(guess, answer);
				if(guess == answer)
				{
					result = 1;
				}
					return result;
		
					numGuesses = numGuesses +1; 
	}
	while( numGuesses < MAXGUESSES);
		
	if (guess != answer)
				{
					result = 0;
				}
					return result; 
}

Does this look right??

[Matticus]

Does it work?

With improved indentation, and some additional notes:

result = Play(answer);

int Play(int answer)
{   
    int numGuesses = 0; 
    int guess = 0;
    int result = 0;                   // Variable not needed (read on)
    do
    {   
        guess = GetGuess();
        CompareGuess(guess, answer);  // Are you comparing the guess with
                                      //   the answer here?

        if(guess == answer)           // ... or here?
        {
            result = 1;               // You can just return 1 here...
        }

        return result;                // ...instead of using this variable
         
        numGuesses = numGuesses +1; 
    } while( numGuesses < MAXGUESSES);
         
    if (guess != answer)              // if the number of guesses have already
    {                                 //   been used up with no correct result,
        result = 0;                   //   you don't need this extra check here.
    }                                 //   Just return 0 after the loop.

    return result;                    // Then you wouldn't need this line, either.
}

[Salem]

> Does this look right??
There's this marvellous tool called a "compiler" sitting right there in your toolbox.

Posting "will this work" is a pointless endeavour.

However, trying things, analysing things and only then posting when you get stuck (along with a more specific question) will get you much further in the long run.

[iMalc]

Because of line 19, there is no way to ever execute lines 21 & 22.
Perhaps you means to put the return inside the preceeding curly braces?

[CASHOUT]

Because of line 19, there is no way to ever execute lines 21 & 22.
Perhaps you means to put the return inside the preceeding curly braces?

Thanks for the help. Everything works now. I appreciate the help.