How could I manage number range output

Prime b

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

9 Replies

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

Vulpes14y ago
Accepted 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 b14y ago
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 b14y ago
OMG NOT AND ITS OR!!!!!!!!!!!!!!!!!!!!!!!!! HAHAHA!!!
Vulpes14y ago
Doh, just spotted the problem!

This line:

if (playerbet < minBet && playerbet > maxBet)

should be:

if (playerbet < minBet || playerbet > maxBet)

Must have been half-asleep earlier :)
Prime b14y ago
It shows up the box when I click it
Vulpes14y ago
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 b14y ago
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 < 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 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
Vulpes14y ago
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 b14y ago
Vulpes? You are the only help!