Textbox should get only numeric with only one decimal

Question

2.00/5 (2 votes)

See more:

i used below coding. but its accepting alphabets also., how to prevent this.,
[i given keypreview=false and tried with keypreview=true]

C#

private void txtorate_KeyPress(object sender,KeyPressEventArgs e)
        {

            if(!char.IsControl(e.KeyChar)&&(!char.IsDigit(e.KeyChar))&& e.KeyChar!='.')
            {
                e.Handled = true;
            }

            if (e.KeyChar == '.' && (sender as TextBox).Text.IndexOf('.') > -1)
            {
                e.Handled = true;
            }
        }

Posted 2-Oct-12 0:47am

Umapathi K

Add a Solution

Comments

Matt T Heffron 2-Oct-12 13:30pm

I don't see a problem with this at all.
It allows digits, the decimal point, and control characters like Ctrl-C (Copy), Ctrl-V (Paste), etc. All other characters are ignored.
What is the problem you are seeing?

By the way, instead of the hard-coded '.', you might want to use CultureInfo.CurrentCulture.NumberFormat.NumberDecimalSeparator. This is a string, not char, so checking could be more difficult. :(

Sergey Alexandrovich Kryukov 2-Oct-12 21:23pm

Good point, about the culture...
And yes, this is not a question at all. Again...
--SA

3 solutions

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)

OriginalGriff · Answer 1 · 2012-10-02T00:49:00

Solution 1

Why not use a NumericUpDown[^] instead? That is pretty much what it was designed for...

Posted 2-Oct-12 0:49am

OriginalGriff

Comments

Sergey Alexandrovich Kryukov 2-Oct-12 21:25pm

Well, about the vote of 3... I'm really surprised that some moron did not vote 1. This is what usually happens when we answer like this.
But in fact, this is the best solution, by far. My 5.
--SA

**JF2015** · Answer 2 · 2012-10-02T00:52:00

Solution 2

See this MSDN article which shows you the code:
http://msdn.microsoft.com/en-us/library/ms229644%28v=vs.80%29.aspx[^]

Posted 2-Oct-12 0:52am

JF2015

Comments

Sergey Alexandrovich Kryukov 2-Oct-12 21:27pm

I voted 4, not 5 only because you did not suggest the obvious and best solution, which would be the same as Solution 1 -- please see and appreciate it.
--SA

WebMaster · Answer 3 · 2012-10-02T05:28:00

Use || instead of && in first condition:

C#

private void txtorate_KeyPress(object sender,KeyPressEventArgs e)
{
    if(!char.Control(e.KeyChar) || (!char.IsDigit(e.KeyChar)) || e.KeyChar != '.')

        if (e.KeyChar == '.' && (sender as TextBox).Text.IndexOf('.') > -1)

                e.Handled = true;
}

I think it would help.

Textbox should get only numeric with only one decimal

3 solutions

Solution 1

Solution 2

Solution 3

Add your solution here

Preview 0

Existing Members

...or Join us