Errors on Tax Calculation in C# .net

Question

0.00/5 (No votes)

See more:

Hello,

C#

private void textBox2_TextChanged(object sender, EventArgs e)
        {
            float t;
            float g;

            t = float.Parse(textBox3.Text);
            g = float.Parse(textBox10.Text);
            float j;

            j = g * (t / 100) + g;


            Convert.ToDecimal(textBox2.Text = j.ToString());
        }

Here input string is not correct format error in textbox3.text how to resolve Help me......

Posted 22-Dec-13 0:10am

Vivek.anand34

Updated 22-Dec-13 0:26am

Nelek

v2

Add a Solution

2 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 · 2013-12-22T00:33:00

Get your user to type correct values?
Or use the right textbox?
You are handling the textBox2 event, but you don;t reference textBox2 at all - I suspect you meant 2 rather than 3.

But even then, that is a poor way to treat users.
Before you use a value, check it, and report problems to your user: at the moment, if he leaves a field blank your program crashes. If he miskeys and gets a comma instead of a dot, it crashes. If he types his name, it crashes. That really is poor - it would annoy the heck out of you, wouldn't it?

Try this:

C#

private void textBox2_TextChanged(object sender, EventArgs e)
    {
    float t;
    float g;
    if (!float.TryParse(textBox3.Text, out t))
        {
        MessageBox.Show("Please type a number in textbox 3!");
        return;
        }
    if (!float.TryParse(textBox10.Text, out g))
        {
        MessageBox.Show("Please type a number in textbox 10!");
        return;
        }

    float j = g * (t / 100) + g;
    string result = j.ToString();
    if (textBox2.Text != result)
        {
        textBox2.Text = result;
        }
    }

Notice the bit at the bottom: if you just set the value of TextBox2.Text each time, it will immediately cause a new event to occur!

And do yourself a favour: stop using VS default names for controls! Calling your controls a name that describes what the user uses them for makes your code much more reliable and readable!

textBox2  => tbTaxDue
textBox10 => tbTaxRate

and so forth.

agent_kruger · Answer 2 · 2013-12-22T19:55:00

Solution 2

for financial calculation us double datataype and to resolve the problem restict them from typing alphabets and other special characters.

Posted 22-Dec-13 19:55pm

agent_kruger