Input sring was not in correct formate

ahmed sa

11y1.1kVisual Basic .NET

Hi i work in windows form application viusal studio 2008

I have windows form have two labeles label 33, label 34

label 33 represent quantity required

label 34 represent quantity shipped

if (Convert.ToInt32(label33.Text) == 0 && Convert.ToInt32(label34.Text) == 0)

{

Sales.SalesClass SalesClass3 = new Sales.SalesClass();

SalesClass3.DeleteProcessThird("Data Source=192.168.1.3;Initial Catalog=ALTAWI Group-Live;User ID=admin;Password=3210;Connection Lifetime=3;Max Pool Size=3;Connection Timeout=30");

}

problem when converting string to integer

it give me error excption

Input sring was not in correct formate

how to solve this problem

7 Replies

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

Michal Habalcik11y ago
well, something like this:

int lbl33 = -1;
int lbl34 = -1;
if (int.TryParse(label33.Text, out lbl 33) && int.TryParse(label34.Text, out lbl 34))

{
if (lbl33 == 0 && lbl34 == 0)
{

Sales.SalesClass SalesClass3 = new Sales.SalesClass();

SalesClass3.DeleteProcessThird("Data Source=192.168.1.3;Initial Catalog=ALTAWI Group-Live;User ID=admin;Password=3210;Connection Lifetime=3;Max Pool Size=3;Connection Timeout=30");

}
} else {
//strings were not in a correct format
}
ahmed sa11y ago
can you show to me how to assign label33 and label34 to 0 by try parse
Michal Habalcik11y ago
Using TryParse should be one of your best practices.

One more note to add. In scenarios like this it is extremelly helpful to debug the program, so you can easily check (usually within few seconds) what value you are actually trying to convert.
ahmed sa11y ago
OK i accept try parse but i don't know how to assign ==0 in statement try parse
ahmed sa11y ago
Thanks if i using try parse as following if(int.TryParse(Label1.Text,out myValue)) { //the rest of your code here } How to assign ==0

Nanhe Siddique11y ago

there are several way to cast string to integer

string s = "123";

 int byCasting = (int)s;

int byParsing = int.Parse(s);

 int result; int.TryParse(s,out result);

 int byConverting = Convert.ToInt32(s);

But among of all I think the TryParse is best.

it may help you

Vincent Maverick Durano11y ago
Make sure that your label contains a valid integer value or else the convert method will fail. I strongly suggest you to use int.TryParse instead like:

int myValue =0;

if(int.TryParse(Label1.Text,out myValue))
{
//the rest of your code here
}