How to stop numeric updown control from incrementing and display a message?

Question

0.00/5 (No votes)

See more:

I have a Numeric Updown Control and a label it is connected to a label. What I wanted to do is when the label reaches 0 it should display a message like "You don't have enough points" and the NUD shouldn't be able to increment or decrement.. I got this part working but the message is popping twice. I know it's just a minor bug but I'm already trying to figure this out for hours.

What I have tried:

Below is my code to display a message when the label reaches 0 but the display message is popping twice which is not right.

VB

Private Sub numCategory_ValueChanged(sender As Object, e As EventArgs) Handles numCategory.ValueChanged


       If num <> 0 Then

           lblpoints.Text = (num - numCategory.Value).ToString

       Else

           numCategory.Value = 0
           MsgBox("No more points to allocate")


       End If

   End Sub

Posted 12-May-20 17:00pm

lelouch_vi 2

Updated 12-May-20 20:34pm

Add a Solution

3 solutions

Solution 1

The message comes twice because you change the value:

numCategory.Value = 0

So comment out this line, or use some construction with a boolean variable to skip the message.

Posted 12-May-20 19:47pm

RickZeeland

Comments

Maciej Los 13-May-20 2:34am

Good point!

lelouch_vi 2 13-May-20 6:58am

Hi RickZeeland,

Yes I think that's because of that line which causes the ValuChange event to be fired twice, But without setting the NUD to zero and the user clicks up the label misbehaves. I guess the best solution for this is to set the NUD to false when the label reaches 0. Thanks anyway.

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Maciej Los · Accepted Answer · 2020-05-12T20:35:00

As RickZeeland pointed it out... below line is redundant and raises numCategory_ValueChanged event again!
VB

numCategory.Value = 0

As to the code... I'd change the body of event this way:

VB

num = numCategory.Value 'set value of NUD to "num" variable
If num = 0 Then MsgBox("No more points to allocate") 'display message
numCategory.Enabled = (num > 0) 'enable/disable NUD
lblpoints.Text = num.ToString()'display current value of NUD

BillWoodruff · Accepted Answer · 2020-05-12T19:49:00

Solution 2

in the case zero is reached, either:

1. hide the numCategory control

or

2. set the Enabled Property of the numCategory control to 'false

Posted 12-May-20 19:49pm

BillWoodruff

Comments

Maciej Los 13-May-20 2:14am

5ed!