Thread: ending problem

hmm i didnt even see that.. thanks for pointing that out...
okay, i just took out the return 0; but, the thing is. It is still doing it. I'm now a bit lost..

The way I see it coded, it looks like it's supposed to close when they win or lose. If you want it to ask for a new game or continue then you have to the gamestatus == WON if statement into the loop and change the condition of the loop.

Code:

int rollDice( void )
{
    int die1, die2, workSum;
    
    die1 = 1 + rand() %6;
    die2 = 1 + rand() %6;
    workSum = die1 + die2;
    cout << "player rolled " << die1 << " + " << die2
         << " = \n" << workSum;
    
    system("PAUSE"); 
    return workSum;
    return EXIT_SUCCESS;
}

That code will never be executed. Get rid of it.

Wait, did you just change that return 0 to return EXIT_SUCCESS?

Here, I added a newgame loop for you. Study what I did so you can understand it.

Code:

#include <cstdlib>
#include <iostream>
#include <time.h>

int rollDice( void );       // function prototype

using namespace std;

int main(int argc, char *argv[])
{
   enum Status { CONTINUE, WON, LOST };
   int sum, myPoint;
   char newGame;
   Status gameStatus;
 
   srand( time( NULL ) );
   
   do {
      sum = rollDice();
      switch ( sum ) {
        case 7:
        case 11:
             gameStatus = WON;
             break;
        case 2:
        case 3:
        case 12:
             gameStatus = LOST;
             break;
        default:
             gameStatus = CONTINUE;
             myPoint = sum;
             cout << "point is " << myPoint << endl;
             break;
      }
        
       while ( gameStatus == CONTINUE ) { //keep rolling
        sum = rollDice();     
        if ( sum == myPoint ) 
           gameStatus = WON;
        else if ( sum == 7 ) 
           gameStatus = LOST;
        }
        
       if ( gameStatus == WON ) 
        cout << "Player wins!" << endl;  
       else if ( gameStatus == LOST )
        cout << "Player loses..." << endl;
        
       cout << "New Game? (y/n)" << endl; 
       cin >> newGame; 
      } while (newGame == 'y' || newGame == 'Y');
            
   return 0;
   
}
          
int rollDice( void )
{
    int die1, die2, workSum;
    
    die1 = 1 + rand() %6;
    die2 = 1 + rand() %6;
    workSum = die1 + die2;
    cout << "player rolled " << die1 << " + " << die2
         << " = " << workSum << endl;
    
    system("PAUSE"); 
    return workSum;
    return 0;
}

Code:

    return workSum;
    return 0;

Two return statements! Again!

Code:

while (newGame == 'y' || newGame == 'Y');

In cases like this, tolower() is your friend:

Code:

while(tolower(newGame) == 'y');

Originally Posted by dwks

Code:

while (newGame == 'y' || newGame == 'Y');

In cases like this, tolower() is your friend:

Code:

while(tolower(newGame) == 'y');

tolower(), I'll have to remember that one. Thanks.