Finalize part : hopfully last

Question

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 availableIf 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.

Vulpes · 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 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 b · Answer

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!

Vulpes · Answer

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 b · Answer

I am adding moving cards so when we click deal it will looks like the cards are being thrown

Prime b · Answer

I updated the form and uploaded http://www.4shared.com/rar/r0_7mFGH/LetItRide.html Just in case you need it

Prime b · Answer

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 b · Answer

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 b · Answer

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