Debugging Craps Simulating Game: Tracking Wins & Losses

[brad sue]

Hi ,
I have a problem coding a part of a simulating game:

In the game of craps, a "Pass line" bet proceeds as follows. The first roll of the two, six-sided dice in a craps round is called the "come out roll". The bet immediately wins when the come out roll is 7 or 11, and loses when the come out is 2,3 or 12.
If 4,5,6,8,9 or 10 is rolled on the come out roll, that number becomes " the point". The player keeps rolling the dice until either 7 or the point is rolled. If the point is rolled first, then the player wins the bet. If the player rolls a 7 first, the the player loses.
you need to indicate the total number of wins and losses.

The part that bothers me is the underline part.
This is the code I've done for this underlined part:

.
.
.
if(roll==4||roll==5||roll==6||roll==8||roll==9||roll==10)//point case
        { 
          cout<<roll<<" points"<<endl;
        do
         {
          roll=dice();//dice- function that simulates the 2 dice throwings
          cout<<endl;
          cout<<"roll "<<roll<<endl;
          if(roll==7)
          cout<<roll<<" the point got to 7. player loses"<<endl;
          if(roll==4||roll==5||roll==6||roll==8||roll==9||roll==10)
          cout<< roll<<" the roll got to 'point'. player wins"<<endl;
         }
          while(roll==7||roll==4||roll==5||roll==6||roll==8||roll==9||roll==10);
       }// end if
.
.
.

Please can you help to organize my code ?
Thank you
Bertrand

 

[buddyholly9999]

int timesToPlay = 100;
int gamesPlayed = 0;
int wins = 0;
int losses = 0;
.

.

.

if(roll==4||roll==5||roll==6||roll==8||roll==9||roll==10)//point case
{
int flag = 0;
cout<<"The point is "<<roll<<endl;
point = roll;
do
{
roll=dice();//dice- function that simulates the 2 dice throwings
cout<<endl;
cout<<"roll = "<<roll<<endl;
if(roll==7) {
cout<<roll<<" was rolled. Player loses."<<endl; losses++; flag = 1; }
if(roll==point) {cout<< roll<<" was rolled. Player got the 'point'. player wins"<<endl; wins++; flag = 1;}
} while (flag == 0)
}// end if

.
.
.

This seems to be what you were trying to say.

 

[ravenprp]

Hi Bertrand,

Thank you for sharing your code and explaining your problem. It looks like you are on the right track with your code, but there are a few things that can be improved to make it more organized and efficient.

First, I would suggest creating a variable to track the total number of wins and losses. You can initialize it to 0 before the game starts and then update it accordingly as the game progresses.

Next, I would suggest using a switch statement instead of multiple if statements to handle the different possible outcomes of the game. This will make your code more organized and easier to read.

Here is an example of how you can modify your code using a switch statement:

int wins = 0; // variable to track wins
int losses = 0; // variable to track losses

// code for the come out roll
int roll = dice(); // simulate the roll of two dice

switch(roll) {
case 7:
case 11:
// player wins
cout << "Player wins!" << endl;
wins++;
break;
case 2:
case 3:
case 12:
// player loses
cout << "Player loses" << endl;
losses++;
break;
default:
// point case
cout << "Point is set to " << roll << endl;
do {
roll = dice(); // simulate the roll of two dice
cout << "Roll " << roll << endl;
if (roll == 7) {
// player loses
cout << "Player loses" << endl;
losses++;
break;
} else if (roll == point) {
// player wins
cout << "Player wins!" << endl;
wins++;
break;
}
} while (true);
break;
}

// after the game is over, you can print the total number of wins and losses
cout << "Total wins: " << wins << endl;
cout << "Total losses: " << losses << endl;

I hope this helps and good luck with your coding!