Input string was not in a correct format

Question

3.00/5 (3 votes)

See more:

Here I am working on a window application. I'm using access database.In this combo box I have to select a particular value and according to this value calculation have to be done. But when I am selecting the value it will be calculated perfectly and after that showing the error message.(Input string was not in a correct format.)

C#

private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
       {
           int tot = 0;
           tot = (Convert.ToInt32(txtPay.Text) + Convert.ToInt32(txtGp.Text) + Convert.ToInt32(txtDa.Text));//This line shows the error.
           txtTotal.Text = Convert.ToString(tot);
           int a = 0;
           if (comboBox1.SelectedIndex == 0)
           {
               a =(tot * 1) / 3;
               txtNet.Text = Convert.ToString(a);
           }
           else if (comboBox1.SelectedIndex == 1)
           {
               a = (tot * 2) / 3;
               txtNet.Text = Convert.ToString(a);
           }
           else if(comboBox1.SelectedIndex==2)
           {
               a = tot;
               txtNet.Text = Convert.ToString(a);
           }

       }

Posted 13-Dec-13 11:08am

bigyan sahoo

Updated 13-Dec-13 19:23pm

adriancs

v4

Add a Solution

Comments

BillWoodruff 14-Dec-13 2:40am

If you have a problem like this, always set a break-point before the error occurs, and then single-step through the code examining the values of the variables used.

4 solutions

Solution 4

try this

C#

private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
        {
            int tot = 0;
            tot = (Convert.ToInt32(txtPay.Text) + Convert.ToInt32(txtGp.Text) + Convert.ToInt32(txtDa.Text));//This line shows the error.
            txtTotal.Text = tot.toString();
            int a = 0;
            if (comboBox1.SelectedIndex == 0)
            {
                a =(tot * 1) / 3;
                txtNet.Text = a.toString();
            }
            else if (comboBox1.SelectedIndex == 1)
            {
                a = (tot * 2) / 3;
                txtNet.Text = a.toString();
            }
            else if(comboBox1.SelectedIndex==2)
            {
                a = tot;
                txtNet.Text = a.toString();
            }
 
        }

Posted 13-Dec-13 23:34pm

An@nd Rajan10

Comments

agent_kruger 14-Dec-13 7:26am

he said in 2 line and you didn't even change a single letter in it.

CHill60 14-Dec-13 7:51am

Doesn't fix the problem

Solution 3

txtTotal.Text = Tot.ToString();

just use above syntax it will work

Posted 13-Dec-13 18:10pm

Gaurav Makwana

Solution 2

Hi
Try like this, cpallini is right..

Its better to Trim the values when you are reading from Textboxes.

C#

int tot = 0;
            int pay, gp, da;
            int.TryParse(txtPay.Text.Trim(), out pay);
            int.TryParse(txtGp.Text.Trim(), out gp);
            int.TryParse(txtDa.Text.Trim(), out pay);
            tot = pay + gp + da;

Posted 13-Dec-13 17:21pm

Karthik_Mahalingam

Comments

bigyan sahoo 15-Dec-13 8:56am

thanks for your help

Karthik_Mahalingam 15-Dec-13 10:11am

Welcome :)

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)

CPallini · Accepted Answer · 2013-12-13T11:49:00

Solution 1

One (or more) of your text boxes contains a string value that is not convertible to an int. You may either catch the exception or use the Int32.TryParse method[^].

Posted 13-Dec-13 11:49am

CPallini

Comments

Sergey Alexandrovich Kryukov 13-Dec-13 19:41pm

5ed :-)
—SA

[no name] 13-Dec-13 23:44pm

+++++5