How could I manage number range output

Question

Updated form   http://www.4shared.com/rar/X2DWbFqF/LetItRide.html?refurl=d1url
If you go to the form I have three buttons, 1st is 5-100 range, 2nd is 25-500 range, and 3rd is 50-1000 range.

For example when I click 5-100 range the bet amount shouldn't be above or below that range.
I tried to do that:
        private void lowestBetButton_Click(object sender, EventArgs e)
            minBet = 5;
            maxBet = 100;
            if (playerbet < minBet && playerbet > maxBet)
            {
                MessageBox.Show("Bet must be between 5$ and 100$", "Bet range error");
            }
else
{
                middleBetButton.Enabled = false;
                topBetButton.Enabled = false;
                chipBet500Button.Visible = false;
                dealButton.Enabled = true;

                EnableChips(true);
}

But it doesnt work...

Vulpes · Answer

Although it's not always possible to do so, I think it's better to prevent users from carrying out an invalid action by disabling the relevant button rather than allowing them to do it and then popping up a MessageBox to say there was an error. So I'd disable the deal button until the user has made a bet. However, there doesn't appear to be anything you can do to stop the user making a bet which is too small or large for the current range apart from popping up a MessageBox to flag the error.

Prime b · Answer

Hmm , its in the wrong location(as the if statement should be under different button). Now when I click that button it display MessageBox right away, without allowing to bet. I am thinking when lowestBetButton is enabled then the range must be between 5-100. Which means I have to put something in the deal button. So if the player bets 4 dollars when the range is between 5-100 and click Deal button then the range error MessageBox pops up. You think that's good idea? I updated few things in the form, so you can take a look http://www.4shared.com/rar/AQvIkAQr/LetItRide.html

Prime b · Answer

OMG NOT AND ITS OR!!!!!!!!!!!!!!!!!!!!!!!!! HAHAHA!!!

Vulpes · Answer

Doh, just spotted the problem! This line: if (playerbet && playerbet > maxBet) should be: if (playerbet || playerbet > maxBet) Must have been half-asleep earlier :)

Prime b · Answer

It shows up the box when I click it

Vulpes · Answer

I think before we try anything else I'd make sure that the eventhandler is actually wired up. You can do this by placing this line as the first line of lowestBetButton_Click: MessageBox.Show("Yes, I'm wired up!");

Prime b · Answer

lowestBetButtonshould set the bet range from 5-100. I did this private void lowestBetButton_Click(object sender, EventArgs e) minBet = 5; maxBet = 100; if (playerbet maxBet) { MessageBox.Show("Bet must be between 5$ and 100$", "Bet range error"); } else { middleBetButton.Enabled = false; topBetButton.Enabled = false; chipBet500Button.Visible = false; dealButton.Enabled = true; EnableChips(true); } But it doesn't work, and playerbet is used to assign the money on the label. The box should appear when the range is below 5 and over 100, but it doesn't appear. I hope that was clear, if not tell me

Vulpes · Answer

You'll need to be more specific about what the problem is. If you're saying that if playerbet is (say) 10 but the messagebox is still appearing even though it's between 5 and 100, then the likelihood is that playerbet is not what you think it is.

Prime b · Answer

Vulpes? You are the only help!