getting error when i try this code

14y2.1kVisual Basic .NET

double days = Convert.ToDouble(txt_working.Text);
double sundays = Convert.ToDouble(txt_netpay.Text);
double a = Convert.ToDouble(txt_basicscale.Text);
txt_basicamount.Text = Convert.ToDouble((a / days) * sundays).ToString();

when i try this above code i got error as

in second line input sting was not in a correct format.

please help me

5 Replies

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

Vulpes14y ago
Narasiman,

I've altered your original code to use double.TryParse.

If a textbox contains an invalid number, then 0 is assigned to the 'out' parameter rather than an exception being thrown.

You also need to make sure that days is non-zero, otherwise you'll get an infinite answer when you attempt to divide by it!

double days, sundays, a;

double.TryParse(txt_working.Text, out days);
double.TryParse(txt_netpay.Text, out sundays);
double.TryParse(txt_basicscale.Text, out a);

if (days != 0)
{
txt_basicamount.Text = (a / days * sundays).ToString();
}
else
{
txt_basicamount.Text = "0"; // say
}
0
Praveen Kumar14y ago
@narasiman,

You don't have to open a new thread for code from previous thread. If you have problem in solutions provided by some user then reply in the same thread.

Also you don't have to post the same question multiple times.
0
Satyapriya Nayak14y ago
Narasiman,

Why repeatedly you are doing the same mistake by creating new thread rather discussing with the same thread itself.

http://www.c-sharpcorner.com/Forums/Thread/178649/salary-amount-to-be-calculated.aspx

I think your questions you are posting are not clear enough for the members to give suitable answers. So please sure that when you are posting any question elaborate it deep and clearly.

Thanks
0
WH Xu14y ago
you can use tryparse() method .
0
Vulpes14y ago
It means that what's in txt_netpay.Text can't be converted to a double.

The most likely reason is that the textbox is empty.

You therefore need to check for this or use the double.TryParse() method rather than Convert.ToDouble.

The same applies to the other two textboxes as well.
0