Finalize part : hopfully last

Prime b

Ok, so we(more like you ) fixed the major bugs right, but there are few left

When the last two cards revelead it should distribute the money, it works correct when players wins ( addes the money to the player) but when players loses or wins it should clear up the labels and the only buttons that should be available is chips( if right amount of money is available,If you bet last time 300 lost and now have only 200 left you cant rebet 300), restart, and rebet button.

I believe that's the last bug Vulpes that is holding me from submitting my project.

8 Replies

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

Vulpes14y ago
Accepted answer
It may be that I'm misunderstanding the rules but it seems to me that following a win by either the player or by the house, only the Rebet and Restart buttons should be enabled.

If the player clicks Rebet and has enough money left, the cards should be re-dealt with an ante equal to whatever it was on the previous deal.

If the play clicks Restart, then the player will be able to choose the range again, choose the ante again and press the deal button.

In both cases, the player's money should be carried forward from the previous deal and not reset.

Another thing I've noticed from playing the online game is that, when a player wins, he gets his bet back as well. So we need to allow for that.

Anyway, this is what I've assumed and it seems to be working quite well after making the folllowing changes:

// LetEmRide class

public LetEmRide()
{
InitializeComponent();
game = new Game(this);
InitializeCardLabels();
this.Text = "LET IT RIDE"; // added this line
}

private void dealButton_Click(object sender, EventArgs e)
{
if (playerbet == 0)
{
MessageBox.Show("You have to place bet first", "Bet required");
}
else
{
if (playerbet < minBet || playerbet > maxBet)
{
MessageBox.Show("Bet must be between " + minBet + " and " + maxBet, "Bet range error");
}
else
{
game.Deal();

int betAmount = Convert.ToInt32(anteLabel.Text);
ActionButtons(true);
EnableRangeBet(false);
EnableChips(false);
dealButton.Enabled = false;
rebetButton.Enabled = false; // added this line
}
}
}

// recoded these methods more or less from scratch

private void restartButton_Click(object sender, EventArgs e)
{
playerbet = 0;
anteLabel.Text = "";
raiseLabel.Text = "";
raise2Label.Text = "";
game.ClearBet();
hasRaised = false;
this.Cursor = Cursors.WaitCursor;
game.StartGame(true);
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);
game.ClearBet();
hasRaised = false;

if (game.Bet(playerbet))
{
anteLabel.Text = "";
raiseLabel.Text = "";
raise2Label.Text = "";
ClearHand();
restartButton.Enabled = false;
ActionButtons(true);
rebetButton.Enabled = false;
anteLabel.Text = Convert.ToString(playerbet);
game.Deal();
}
}

// Game class

public void StartGame(bool restart) // now takes a bool parameter
{
deck = new GraphicalDeck();
if (!restart)
{
int money = gen.Next(100, 1001);
player = new Player(money);
}
player.Hand = new Hand();
gui.DisplayPlayerMoney(player.Money);
gui.ShowBetAmount(player.Money);
gui.EnableChips(false);
}

public void ClearBet() // ADDED THIS METHOD
{
player.ClearBet();
}

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

string description;
switch (player.Hand.DetermineHandByType(out description))
{
case HandType.NOTHING:
// payOut = -betAmount;
//pay the dealer

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;
}

//Add (or subtract) the bet amount from the players cash.
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);
}

gui.Controls["continueButton"].Enabled = false;
gui.Controls["raiseButton"].Enabled = false;
gui.Controls["rebetButton"].Enabled = true;
gui.Controls["restartButton"].Enabled = true;
}

public void FlipLeft()
{
//both cartds flipped, you need to go figure out if this was a winner or not.

//diplsay the results...

//Tell the gui to clear itself and get ready for a new hand

//things like enable the betting buttons again.
PayOutMoney();
//gui.EnableChips(true); // take this line out
}

// Player class

public void ClearBet() //ADDED THIS METHOD
{
bet = 0;
}

One thing I don't like is that once a bet range has been chosen, the button for that range is enabled so the player can see what the range is. A better idea may be to introduce a label to show the chosen range and then disable all the range buttons. I haven't done that for now but see what you think.
Prime b14y ago
Ok vulpes, I was wonder where you went, at first I thought maybe you tired of helping me, but now I understand.

So I am adding comments to the code so it follows the requirements. I added the animation, so the cards flying now, pretty cool. I created separate thread. You should download the latest version.
Thank you again!
Vulpes14y ago
Sorry, I had to go out and have only just got back, much later than I anticipated :(

Yes, you're right that the StartGame call in the startButton_Click method needs to be altered - I had in fact altered it but overlooked it when preparing my previous post.

I don't understand why you're getting the format exception unless anteLabel.Text has been cleared in the latest version of the program - I was working from an earlier version as I couldn't download the latest version at the time.

Anyway, I'll have another look at it tomorrow, in the light of your comments and the code you've added for moving the cards, to see if I can iron out the remaining bugs :)
Prime b14y ago
I am adding moving cards, so when we click deal, it will looks like the cards are being thrown
Prime b14y ago
I updated the form, and uploaded http://www.4shared.com/rar/r0_7mFGH/LetItRide.html

Just in case you need it
Prime b14y ago
I'll go make some food, I got back from college and checked the forums right away, and you already replied! So i haven't had any food today!
Prime b14y ago
Also the label thingy for the range I think is pretty cool, I am agree with you on that.
And Restart button is like whole new game, so it should give the player new amount of money.
Also when player has 0 money left, what happens after? It should same GAME OVER!!!!! lol
Prime b14y ago
I cant test, it because I get an error.

First in this click method I had to add true as a paramter for the game.StartGame
Well then I realize I had to put false in there, so i added false instead of true
private void startButton_Click(object sender, EventArgs e)
        {
            this.Cursor = Cursors.WaitCursor;
            game.StartGame(true);   <<=== added true, otherwise it would give me 0 arguments passed. <<<=== changed to false
            startButton.Visible = false;
            this.Cursor = Cursors.Default;
            dealButton.Enabled = false; // DISABLE DEAL BUTTON UNTIL BET RANGE IS CHOOSEN

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

            continueButton.Enabled = false;
            raiseButton.Enabled = false;
            rebetButton.Enabled = false;

But the main error I am getting is once you revealed your cards, and click Rebet, it gives me FormatException was unhandled at

private void rebetButton_Click(object sender, EventArgs e)
        {
            playerbet = Convert.ToInt32(anteLabel.Text); <<===format exception.
            game.ClearBet();
            hasRaised = false;

            if (game.Bet(playerbet))
            {
                anteLabel.Text = "";
                raiseLabel.Text = "";
                raise2Label.Text = "";
                ClearHand();
                restartButton.Enabled = false;
                ActionButtons(true);
                rebetButton.Enabled = false;
                anteLabel.Text = Convert.ToString(playerbet);
                game.Deal();

And the last thing is when you click deal ,your bet amount disappears(anteLabel)