Using functions or modules help

[TamaThps]

I have to write a program that lets the user play the game of Craps. As of right now it is incomplete and I have run into some errors. It's my first time using functions instead of the program all being in the int main().

I am using visual studio on windows XP.

In my code I've defined a local variable in a function but when I run the program and it gets to that function it says I have an undeclared variable.

The code for the program is as follows:

// #include preprocessor directives
#include <iostream>
#include <cstdlib>        // for using rand() function
#include <ctime>            // for using time() function
#include <string>
 
// namespace directives
using namespace std ;
 
void CrapsIntro() ;            // brief description of Craps game 
int  CrapsRoll() ;            // outcome of two six-sided dice rolls
void myTimeDelay(int) ;            // time delay between rolls
void playLineFunction(int);
void dontPlayLineFunction();
// functionA prototype
// functionB prototype
// other function prototypes
// . . .
 
const int DELAY = INT_MAX / 2 ;    // manual delay between rolls
 
 
int main ()
{
    // variable declaration
    int myRoll;
    char bettingMode; //used for choosing a betting mode
    char playAgain; //Y or N to play again
    int playLoop; //used for making the loop to play the game again
    // variable initializations
 
    // Display game instructions
    CrapsIntro() ;
    playLoop = 1;
 
while (playLoop > 0)
{
    cout << "Which betting mode would you like to play, Pass Line, or Don't Pass Line?" << endl;
    cout << "Type P for Pass Line or D for Don't Pass Line" << endl; // Ask user to play Pass Line OR Don't Pass Line
    cin >> bettingMode;
    if (bettingMode == 80)// ASCII code for P
    {
        cout << "You chose to play Pass Line" << endl;
        myRoll = CrapsRoll();
        playLineFunction(myRoll);
    }
    if (bettingMode == 68)// ASCII code for D
    {    
        cout << "You chose to play Don't Pass Line" << endl;
        myRoll = CrapsRoll();
    }
    if (bettingMode != 80 && bettingMode != 68) //If the user inputs a character other than P or D 
    {                                           //it ends the program for an invalid play mode
        cout << "You have chosen an invalid game mode" << endl;
        return 1;
    }
    cout << "Do you wish to play again? Type Y or N" << endl; // Ask user if they want to play another game
    cin >> playAgain;
    if (playAgain == 78) //ASCII code for Y
    {    
        cout << "Thank you for playing" << endl;
        playLoop--; // Decrement playLoop so that it exits the loop
    }
 
}
    return 0;
}
 
// Play Line Function
// This function tests the conditions and rules
// in order to play the Play Line mode of Craps
void playLineFunction(int myRoll)
{
    int POINT;
 
    if (myRoll == 7 || myRoll == 11)
    {
        cout << "You win!" << endl;
    }
    if (myRoll == 2 || myRoll == 3 || myRoll == 12)
    {
        cout << "You lose! Crapped out" << endl;
    }
    else 
    {
        myRoll = POINT;
        cout << "You've established a point of " << POINT << endl;
    }
    while (myRoll != 7)
    {
        myRoll = CrapsRoll();
        if (myRoll = POINT)
        {
            cout << "You win!" << endl;
 
        }
    }
    cout << "You Lose! Seven out" << endl;
}
 
 
// functionB definition
// Provide function description comment block, preconditions, and 
// postconditions
 
// . . .
 
// The purpose of this function is to give an intorduction to the user 
// about the program
void CrapsIntro()
{
    cout << "This is my Craps game. You will need to choose a playing mode of Craps, " << endl ;
    cout << "and then proceed following the basic rules of Craps to win or lose." << endl << endl;
}
 
// Provide function description comment block, preconditions, and 
// postconditions
void myTimeDelay(int delay)
{
    int count = 0 ;
    while (delay > count)
    {
        // do nothing delay period
        count++;
    }
}
 
// Provide function description comment block, preconditions, and 
// postconditions
int CrapsRoll()
{
    int randomNumber ;            // a random number
    int dieOne ;                // a six-sided die roll value
    int dieTwo ;                // a six-sided die roll value
 
    myTimeDelay(DELAY) ;
 
    cout << "Rolling my two dice ..." << endl ;
 
    // die one
    srand(int(time(0))) ;                // seeding random number generator 
    randomNumber = rand() ;            // generate random number
    dieOne = (randomNumber % 6) + 1 ;    // a number between 1 and 6
 
    myTimeDelay(DELAY) ;
 
    // die two
    srand(int(time(0))) ;                // seeding random number generator 
    randomNumber = rand() ;            // generate random number
    dieTwo = (randomNumber % 6) + 1 ;    // a number between 1 and 6
 
    cout << "You rolled a " << dieOne + dieTwo << endl ;
 
    return dieOne + dieTwo ;
}
 

The program compiles correctly with no errors, but when it is run, I get the error that POINT is an undeclared variable. POINT is declared inside the void function playLineFunction (line 74) that I initialized in the main function.

I don't understand why I am getting this error. Any assistance is appreciated.

[boxfish]

Maybe it says POINT is uninitialized, not undeclared?

myRoll = POINT;
cout << "You've established a point of " << POINT << endl;

You have never given POINT any value, so it will be completely random. For me it was 2088810195.
I didn't take the time to understand the logic of your code, but it would make more sense to me to write

POINT = myRoll;
cout << "You've established a point of " << POINT << endl;

but that's just my wild guess.
Hope this helps anyway.

[TamaThps]

Sorry you were correct the error I get was an initializing error and not a declaration error.

I set it now to:
int POINT = 0;
And I had the other 2 reversed like you said. Now POINT = myRoll

I no longer get the error but I've run into trouble with the nested If statement within the while statement. Here are the changes I made (I only made changes to the playLineFunction)

void playLineFunction(int myRoll)
{
    int POINT = 0;
 
    if (myRoll == 7 || myRoll == 11)
    {
        cout << "You win!" << endl;
    }
    if (myRoll == 2 || myRoll == 3 || myRoll == 12)
    {
        cout << "You lose! Crapped out" << endl;
    }
    else 
    {
        POINT = myRoll;
        cout << "You've established a point of " << POINT << endl;
    }
    while (myRoll != 7)
    {
        myRoll = CrapsRoll();
        if (myRoll = POINT)
        {
            cout << "You win!" << endl;
 
        }
    }
    cout << "You Lose! Seven out" << endl;
}
 

Ideally when the program runs it sets POINT equal to myRoll. While myRoll does not equal to 7 (7 means you've lost after you've established a point) myRoll calls the CrapsRoll function and gets a new value. Then it goes to the nested if statement saying you win if myRoll is equal to POINT.
The problem is that the program is saying "You win!" even when myRoll doesn't equal POINT also, it does not exit the while loop when myRoll is 7. So as of now it's in an infinite loop. I think this has to do with line 20 where it calls the CrapsRoll function.
I know I've set POINT = myRoll which would cause an infinteloop of "You win!" but it should be getting a new value for myRoll from the CrapsRoll function which would cause it loop until it rolls a 7 or it rolls the value of POINT.

I think that explains my new problem. Thanks for the help before boxfish

[boxfish]

also, it does not exit the while loop when myRoll is 7. So as of now it's in an infinite loop.

You wrote
if (myRoll = POINT)
You assign POINT to myRoll instead of comparing them. Use
if (myRoll == POINT)
Hope this helps.