Input sring was not in correct formate

Question

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

Michal Habalcik · Answer

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 sa · Answer

can you show to me how to assign label33 and label34 to 0 by try parse

Michal Habalcik · Answer

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 sa · Answer

OK i accept try parse but i don't know how to assign ==0 in statement try parse

ahmed sa · Answer

Thanks if i using try parse as following if(int.TryParse(Label1.Textout myValue)) { //the rest of your code here } How to assign ==0

Nanhe Siddique · Answer

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(sout result); int byConverting = Convert.ToInt32(s); But among of all I think the TryParse is best. it may help you

Vincent Maverick Durano · Answer

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.Textout myValue)) { //the rest of your code here }