Finalize part: bug check

OK, I've made a large number of changes to the version which was current when you first started this thread, so I hope that any subsequent changes you've made won't affect it:

1. I fixed the format exception which was due to you adding a line resetting anteLabel.Text to empty at the end of the EnableChips method.

2. After a deal is completed and result MessageBox has been closed, the ante and raise labels are all cleared but not the cards themsleves, just like the online game.

However, the player then has three choices (all other buttons are disabled):

a) Pressing one of the bet range buttons, when the cards then disappear and the player can make his bet. His money is carried forward from the previous deal.

b) Pressing the Rebet button, when the cards are redealt and the player automatically bets his previous ante again. His money is carried forward from the previous deal.

b) Pressing the Restart button, when the whole game (money and everything) is reset to the state it was in when the Start button was pressed in the first place.

3. When a bet range is chosen, the other two ranges are made invisible and the chosen range is disabled so it can't be pressed again but remains visible so the player knows what the current range is.

4. A bet range cannot now be chosen if there is insufficient money to cover the minimum bet.

5. When there's less than $5 money left (i.e. no bet can be made), a MessageBox pops up to tell the player the game is over. When he presses OK the application ends.

6. Sorry I just haven't had time to look at the moving cards code - there is a bug on a Rebet as you say when only two player cards are dealt - but I'll try and come back to it later tonight if I can.

7. There shouldn't be any problem with the decks no matter how many deals there are as the cards are always reshuffled.

8. I've tried to highlight the changes but I'd keep an eye open in case I've missed something:

// LetEmRide class

private bool handCleared = false; // **ADDED**

public void ClearHand()

{

cardPlayer1.Visible = false;

cardPlayer2.Visible = false;

cardPlayer3.Visible = false;

cardDealer1.Visible = false;

cardDealer2.Visible = false;

handCleared = true; // **ADDED**

}

public void ShowHand(Hand hand)

{

cardPlayer1.Card = hand.CardsInHand[0];

cardPlayer1.DisplayCard();

cardPlayer2.Card = hand.CardsInHand[1];

cardPlayer2.DisplayCard();

cardPlayer3.Card = hand.CardsInHand[2];

cardPlayer3.DisplayCard();

cardDealer1.Card = hand.CardsInHand[3];

cardDealer1.Card.ShowFace = false;

cardDealer1.DisplayCard();

cardDealer2.Card = hand.CardsInHand[4];

cardDealer2.Card.ShowFace = false;

cardDealer2.DisplayCard();

newDeckLabel.Size = cardDealer1.Size;

recycleDeckLabel.Size = cardDealer1.Size;

cards[0] = cardDealer1;

cards[1] = cardPlayer2;

cards[2] = cardPlayer3;

cards[3] = cardDealer1;

cards[4] = cardDealer2;

for (int index = 0; index < 5; index++)

{

cardsLocation[index] = cards[index].Location;

cards[index].Location = newDeckLabel.Location;

}

handCleared = false; // **ADDED**

timerOne.Enabled = true;

timerTick = 50;

}

/* **REMOVED AS NOT NEEDED**

///

/// Show's current ante bet, in the anteLabel.

///

public void ShowBetAmount(int amount)

{

anteLabel.Text = "";

}

public void RevealCards()

{

if (cardDealer2.Card.ShowFace)

{

FlipLeftCard();

playerbet = Convert.ToInt32(anteLabel.Text); // **ADDED TO SAVE ANTE IN CASE REBET BUTTON PRESSED ON NEXT DEAL**

game.FlipLeft();

}

else

{

FlipRightCard();

}

public void EnableChips(bool enable)

{

chipBet1Button.Enabled = enable;

chipBet5Button.Enabled = enable;

chipBet25Button.Enabled = enable;

chipBet100Button.Enabled = enable;

chipBet500Button.Enabled = enable;

//anteLabel.Text = ""; // **REMOVED**

}

public void EnableRangeBet(bool enable) //**CHANGED FROM PRIVATE TO PUBLIC**

{

lowestBetButton.Enabled = enable;

middleBetButton.Enabled = enable;

topBetButton.Enabled = enable;

}

private void startButton_Click(object sender, EventArgs e)

{

this.Cursor = Cursors.WaitCursor;

game.StartGame(); // **REMOVED PARAMETER**

startButton.Visible = false;

this.Cursor = Cursors.Default;

//dealButton.Enabled = false; // **REMOVED**

EnableChips(false);

ActionButtons(false);

EnableRangeBet(true);

dealButton.Enabled = false;

restartButton.Enabled = false;

//continueButton.Enabled = false; // **REMOVED**

//raiseButton.Enabled = false; // **REMOVED**

//rebetButton.Enabled = false; // **REMOVED**

}

private void lowestBetButton_Click(object sender, EventArgs e)

{

minBet = 5;

maxBet = 100;

playerbet = 0; // **ADDED**

hasRaised = false; // **ADDED**

lowestBetButton.Enabled = false; // **ADDED**

middleBetButton.Visible = false; // **CHANGED**

topBetButton.Visible = false; // **CHANGED**

chipBet500Button.Visible = false;

dealButton.Enabled = true;

EnableChips(true);

restartButton.Enabled = false; // **ADDED**

rebetButton.Enabled = false; // **ADDED**

if (!handCleared) ClearHand(); // **ADDED**

}

private void middleBetButton_Click(object sender, EventArgs e)

{

int money = Convert.ToInt32(playerMoneyLabel.Text); // **ADDED**

if (money < 25) // **ADDED**

{

MessageBox.Show("Sorry, you don't have sufficient money for this bet range");

return;

}

minBet = 25;

maxBet = 500;

playerbet = 0; // **ADDED**

hasRaised = false; // **ADDED**

middleBetButton.Enabled = false; // **ADDED**

lowestBetButton.Visible = false; // **CHANGED**

topBetButton.Visible = false; // **CHANGED**

dealButton.Enabled = true;

EnableChips(true);

restartButton.Enabled = false; // **ADDED**

rebetButton.Enabled = false; // **ADDED**

if (!handCleared) ClearHand(); // **ADDED**

}

private void topBetButton_Click(object sender, EventArgs e)

{

int money = Convert.ToInt32(playerMoneyLabel.Text); // **ADDED**

if (money < 50) // **ADDED**

{

MessageBox.Show("Sorry, you don't have sufficient money for this bet range");

return;

}

minBet = 50;

maxBet = 1000;

playerbet = 0; // **ADDED**

hasRaised = false; // **ADDED**

topBetButton.Enabled = false; // **ADDED**

lowestBetButton.Visible = false; // **CHANGED**

middleBetButton.Visible = false; // **CHANGED**

dealButton.Enabled = true;

EnableChips(true);

restartButton.Enabled = false; // **ADDED**

rebetButton.Enabled = false; // **ADDED**

if (!handCleared) ClearHand(); // **ADDED**

}

private void restartButton_Click(object sender, EventArgs e)

{

playerbet = 0;

//anteLabel.Text = ""; // **REMOVED**

//raiseLabel.Text = ""; // **REMOVED**

//raise2Label.Text = ""; // **REMOVED**

//game.ClearBet(); // **REMOVED**

hasRaised = false;

this.Cursor = Cursors.WaitCursor;

game.StartGame(); // **REMOVERED PARAMETER**

this.Cursor = Cursors.Default;

ClearHand(); // Clear cards labels

dealButton.Enabled = false; // DISABLE DEAL BUTTON UNTIL BET RANGE IS CHOOSEN

chipBet500Button.Visible = true;

EnableChips(false);

ActionButtons(false);

EnableRangeBet(true);

restartButton.Enabled = false;

}

private void rebetButton_Click(object sender, EventArgs e)

{

//playerbet = Convert.ToInt32(anteLabel.Text); // **REMOVED**

// game.ClearBet(); // **REMOVED**

hasRaised = false;

if (game.Bet(playerbet))

{

timerTwo.Enabled = true;

timerTick2 = 50;

//anteLabel.Text = ""; // **REMOVED**

//raiseLabel.Text = ""; // **REMOVED**

//raise2Label.Text = ""; // **REMOVED**

ClearHand();

restartButton.Enabled = false;

ActionButtons(true);

EnableRangeBet(false);

rebetButton.Enabled = false;

anteLabel.Text = Convert.ToString(playerbet);

game.Deal();

}

}

// Game class

public void StartGame() // **REMOVED PARAMETER AND RECODED METHOD**

{

deck = new GraphicalDeck();

int money = gen.Next(100, 1001);

player = new Player(money);

gui.DisplayPlayerMoney(player.Money);

gui.EnableChips(false);

}

public void PayOutMoney()

{

int payOut = 0;

int betAmount = player.GetBetAmount();

string description;

switch (player.Hand.DetermineHandByType(out description))

{

case HandType.NOTHING:

break;

case HandType.TENS_OR_BETTER:

payOut = betAmount * 1;

break;

case HandType.TWO_PAIRS:

payOut = betAmount * 2;

break;

case HandType.THREE_OF_A_KIND:

payOut = betAmount * 3;

break;

case HandType.STRAIGHT:

payOut = betAmount * 5;

break;

case HandType.FLUSH:

payOut = betAmount * 8;

break;

case HandType.FULL_HOUSE:

payOut = betAmount * 11;

break;

case HandType.FOUR_OF_A_KIND:

payOut = betAmount * 50;

break;

case HandType.STRAIGHT_FLUSH:

payOut = betAmount * 200;

break;

case HandType.ROYAL_FLUSH:

payOut = betAmount * 1000;

break;

}

if (payOut > 0)

{

description += " " + payOut.ToString("C0");

}

System.Windows.Forms.MessageBox.Show(description, "Result");

if (payOut > 0)

{

player.Money += payOut + betAmount; // also get ante back

gui.DisplayPlayerMoney(player.Money);

}

// **CHANGED**

if (player.Money < 5)

{

System.Windows.Forms.MessageBox.Show("You don't have enough money left to make a bet, game over!", "Game Over");

gui.Close();

}

else

{

ClearBet();

gui.Controls["continueButton"].Enabled = false;

gui.Controls["raiseButton"].Enabled = false;

gui.Controls["rebetButton"].Enabled = true;

gui.Controls["restartButton"].Enabled = true;

gui.Controls["anteLabel"].Text = "";

gui.Controls["raiseLabel"].Text = "";

gui.Controls["raise2Label"].Text = "";

gui.Controls["lowestBetButton"].Visible = true;

gui.Controls["middleBetButton"].Visible = true;

gui.Controls["topBetButton"].Visible = true;

gui.EnableRangeBet(true);

}

// **CHANGED**

}

4 Replies

Know the answer? Post it — somebody with the same question will find it here.

Vulpes14y ago
Accepted answer
OK, I've made a large number of changes to the version which was current when you first started this thread, so I hope that any subsequent changes you've made won't affect it:

1. I fixed the format exception which was due to you adding a line resetting anteLabel.Text to empty at the end of the EnableChips method.

2. After a deal is completed and result MessageBox has been closed, the ante and raise labels are all cleared but not the cards themsleves, just like the online game.

However, the player then has three choices (all other buttons are disabled):

a) Pressing one of the bet range buttons, when the cards then disappear and the player can make his bet. His money is carried forward from the previous deal.

b) Pressing the Rebet button, when the cards are redealt and the player automatically bets his previous ante again. His money is carried forward from the previous deal.

b) Pressing the Restart button, when the whole game (money and everything) is reset to the state it was in when the Start button was pressed in the first place.

3. When a bet range is chosen, the other two ranges are made invisible and the chosen range is disabled so it can't be pressed again but remains visible so the player knows what the current range is.

4. A bet range cannot now be chosen if there is insufficient money to cover the minimum bet.

5. When there's less than $5 money left (i.e. no bet can be made), a MessageBox pops up to tell the player the game is over. When he presses OK the application ends.

6. Sorry I just haven't had time to look at the moving cards code - there is a bug on a Rebet as you say when only two player cards are dealt - but I'll try and come back to it later tonight if I can.

7. There shouldn't be any problem with the decks no matter how many deals there are as the cards are always reshuffled.

8. I've tried to highlight the changes but I'd keep an eye open in case I've missed something:

// LetEmRide class

private bool handCleared = false; // **ADDED**

public void ClearHand()
{
cardPlayer1.Visible = false;
cardPlayer2.Visible = false;
cardPlayer3.Visible = false;

cardDealer1.Visible = false;
cardDealer2.Visible = false;

handCleared = true; // **ADDED**
}

public void ShowHand(Hand hand)
{
cardPlayer1.Card = hand.CardsInHand[0];
cardPlayer1.DisplayCard();
cardPlayer2.Card = hand.CardsInHand[1];
cardPlayer2.DisplayCard();
cardPlayer3.Card = hand.CardsInHand[2];
cardPlayer3.DisplayCard();

cardDealer1.Card = hand.CardsInHand[3];
cardDealer1.Card.ShowFace = false;
cardDealer1.DisplayCard();
cardDealer2.Card = hand.CardsInHand[4];
cardDealer2.Card.ShowFace = false;
cardDealer2.DisplayCard();
newDeckLabel.Size = cardDealer1.Size;
recycleDeckLabel.Size = cardDealer1.Size;

cards[0] = cardDealer1;
cards[1] = cardPlayer2;
cards[2] = cardPlayer3;
cards[3] = cardDealer1;
cards[4] = cardDealer2;
for (int index = 0; index < 5; index++)
{
cardsLocation[index] = cards[index].Location;
cards[index].Location = newDeckLabel.Location;
}

handCleared = false; // **ADDED**
timerOne.Enabled = true;
timerTick = 50;
}

  /* **REMOVED AS NOT NEEDED**
///
/// Show's current ante bet, in the anteLabel.
///
///
public void ShowBetAmount(int amount)
{
anteLabel.Text = "";
}
*/

  public void RevealCards()
{
if (cardDealer2.Card.ShowFace)
{
FlipLeftCard();
playerbet = Convert.ToInt32(anteLabel.Text); // **ADDED TO SAVE ANTE IN CASE REBET BUTTON PRESSED ON NEXT DEAL**
game.FlipLeft();
}
else
{
FlipRightCard();
}
}

public void EnableChips(bool enable)
{
chipBet1Button.Enabled = enable;
chipBet5Button.Enabled = enable;
chipBet25Button.Enabled = enable;
chipBet100Button.Enabled = enable;
chipBet500Button.Enabled = enable;
//anteLabel.Text = ""; // **REMOVED**
}

public void EnableRangeBet(bool enable) //**CHANGED FROM PRIVATE TO PUBLIC**
{
lowestBetButton.Enabled = enable;
middleBetButton.Enabled = enable;
topBetButton.Enabled = enable;
}

private void startButton_Click(object sender, EventArgs e)
{
this.Cursor = Cursors.WaitCursor;
game.StartGame(); // **REMOVED PARAMETER**
startButton.Visible = false;
this.Cursor = Cursors.Default;
//dealButton.Enabled = false; // **REMOVED**

EnableChips(false);
ActionButtons(false);
EnableRangeBet(true);
dealButton.Enabled = false;
restartButton.Enabled = false;

//continueButton.Enabled = false; // **REMOVED**
//raiseButton.Enabled = false; // **REMOVED**
//rebetButton.Enabled = false; // **REMOVED**
}

private void lowestBetButton_Click(object sender, EventArgs e)
{
minBet = 5;
maxBet = 100;
playerbet = 0; // **ADDED**
hasRaised = false; // **ADDED**
lowestBetButton.Enabled = false; // **ADDED**
middleBetButton.Visible = false; // **CHANGED**
topBetButton.Visible = false; // **CHANGED**
chipBet500Button.Visible = false;
dealButton.Enabled = true;
EnableChips(true);
restartButton.Enabled = false; // **ADDED**
rebetButton.Enabled = false; // **ADDED**
if (!handCleared) ClearHand(); // **ADDED**
}

private void middleBetButton_Click(object sender, EventArgs e)
{
int money = Convert.ToInt32(playerMoneyLabel.Text); // **ADDED**
if (money < 25) // **ADDED**
{
MessageBox.Show("Sorry, you don't have sufficient money for this bet range");
return;
}
minBet = 25;
maxBet = 500;
playerbet = 0; // **ADDED**
hasRaised = false; // **ADDED**
middleBetButton.Enabled = false; // **ADDED**
lowestBetButton.Visible = false; // **CHANGED**
topBetButton.Visible = false; // **CHANGED**
dealButton.Enabled = true;

EnableChips(true);
restartButton.Enabled = false; // **ADDED**
rebetButton.Enabled = false; // **ADDED**
if (!handCleared) ClearHand(); // **ADDED**
}

private void topBetButton_Click(object sender, EventArgs e)
{
int money = Convert.ToInt32(playerMoneyLabel.Text); // **ADDED**
if (money < 50) // **ADDED**
{
MessageBox.Show("Sorry, you don't have sufficient money for this bet range");
return;
}
minBet = 50;
maxBet = 1000;
playerbet = 0; // **ADDED**
hasRaised = false; // **ADDED**
topBetButton.Enabled = false; // **ADDED**
lowestBetButton.Visible = false; // **CHANGED**
middleBetButton.Visible = false; // **CHANGED**
dealButton.Enabled = true;

EnableChips(true);
restartButton.Enabled = false; // **ADDED**
rebetButton.Enabled = false; // **ADDED**
if (!handCleared) ClearHand(); // **ADDED**
}

private void restartButton_Click(object sender, EventArgs e)
{
playerbet = 0;
//anteLabel.Text = ""; // **REMOVED**
//raiseLabel.Text = ""; // **REMOVED**
//raise2Label.Text = ""; // **REMOVED**
//game.ClearBet(); // **REMOVED**
hasRaised = false;
this.Cursor = Cursors.WaitCursor;
game.StartGame(); // **REMOVERED PARAMETER**
this.Cursor = Cursors.Default;
ClearHand(); // Clear cards labels
dealButton.Enabled = false; // DISABLE DEAL BUTTON UNTIL BET RANGE IS CHOOSEN
chipBet500Button.Visible = true;
EnableChips(false);
ActionButtons(false);
EnableRangeBet(true);
restartButton.Enabled = false;
}

private void rebetButton_Click(object sender, EventArgs e)
{
//playerbet = Convert.ToInt32(anteLabel.Text); // **REMOVED**
// game.ClearBet(); // **REMOVED**
hasRaised = false;

if (game.Bet(playerbet))
{
timerTwo.Enabled = true;
timerTick2 = 50;
//anteLabel.Text = ""; // **REMOVED**
//raiseLabel.Text = ""; // **REMOVED**
//raise2Label.Text = ""; // **REMOVED**
ClearHand();
restartButton.Enabled = false;
ActionButtons(true);
EnableRangeBet(false);
rebetButton.Enabled = false;
anteLabel.Text = Convert.ToString(playerbet);
game.Deal();
}
}

// Game class

public void StartGame() // **REMOVED PARAMETER AND RECODED METHOD**
{
deck = new GraphicalDeck();
int money = gen.Next(100, 1001);
player = new Player(money);
gui.DisplayPlayerMoney(player.Money);
gui.EnableChips(false);
}

public void PayOutMoney()
{
int payOut = 0;
int betAmount = player.GetBetAmount();

string description;
switch (player.Hand.DetermineHandByType(out description))
{
case HandType.NOTHING:
break;

case HandType.TENS_OR_BETTER:
payOut = betAmount * 1;
break;
case HandType.TWO_PAIRS:
payOut = betAmount * 2;
break;

case HandType.THREE_OF_A_KIND:
payOut = betAmount * 3;
break;

case HandType.STRAIGHT:
payOut = betAmount * 5;
break;

case HandType.FLUSH:
payOut = betAmount * 8;
break;

case HandType.FULL_HOUSE:
payOut = betAmount * 11;
break;

case HandType.FOUR_OF_A_KIND:
payOut = betAmount * 50;
break;

case HandType.STRAIGHT_FLUSH:
payOut = betAmount * 200;
break;
case HandType.ROYAL_FLUSH:

payOut = betAmount * 1000;
break;
}
if (payOut > 0)
{
description += " " + payOut.ToString("C0");
}
System.Windows.Forms.MessageBox.Show(description, "Result");
if (payOut > 0)
{
player.Money += payOut + betAmount; // also get ante back
gui.DisplayPlayerMoney(player.Money);
}

// **CHANGED**
if (player.Money < 5)
{
System.Windows.Forms.MessageBox.Show("You don't have enough money left to make a bet, game over!", "Game Over");
gui.Close();
}
else
{
ClearBet();
gui.Controls["continueButton"].Enabled = false;
gui.Controls["raiseButton"].Enabled = false;
gui.Controls["rebetButton"].Enabled = true;
gui.Controls["restartButton"].Enabled = true;
gui.Controls["anteLabel"].Text = "";
gui.Controls["raiseLabel"].Text = "";
gui.Controls["raise2Label"].Text = "";
gui.Controls["lowestBetButton"].Visible = true;
gui.Controls["middleBetButton"].Visible = true;
gui.Controls["topBetButton"].Visible = true;
gui.EnableRangeBet(true);
}
// **CHANGED**
}
Prime b14y ago
It's due today at 11:59 EST , which means i have 7 hours, its almost 9 in UK.
I am just giving general idea of the time
Prime b14y ago
Vulpes everything works like a magic! This is great. Except one thing, my animation is bugged . First round works good, but when you click rebet all my animation get messed up. Such as one of the two hidden cards for some reason is open , and one of the three cards is missing.
But other than that, it works clearly excellent. I definitely need to buy a gift for the work you have done!
Prime b14y ago
Also vulpes, what would happen when we would go through the whole deck? Would it throw an error, or it would have new deck ready?