views:

95

answers:

4

No exception is thrown, function just halts at this statement:

int productQuantity = Convert.ToInt32("1.00");

and returns.

What am I doing wrong to convert this float to Int32?

Note: I am running in a BackgroundWorkerThread.

+5  A: 

An exception is being thrown in this case it's just not being surfaced in the debugger. This string is not in a format that is convertible to an Int32 type and hence throws and exception. You can verify this by wrapping it in a try/catch block if the IDE isn't cooperating.

The best approach here is probably to convert the string to a double and then manually cast it down to an int. This does open the door for data loss due to precision differences. But given your input is in a float style format this is unavoidable if you want the final product to be an int

JaredPar
+1 good points on the unsurfacable exception, easy to forget!
Abel
A: 

FormatException Input string was not in a correct format.

Sandeep Singh Rawat
+2  A: 

An exception is thrown, it's just that to see it you have to inspect the RunWorkerCompletedEventArgs.Error property in the event handler for BackgroundWorker.RunWorkerCompleted.

Any exception that is thrown from the background worker's thread when the background work is being done is assigned to that property.

Alex Humphrey
+2  A: 

You need to convert it to a double first, and then convert to Int32.

int productQuantity = Convert.ToInt32(double.Parse("1.00"));
Chris Almond
with strings I recommend to always use `TryParse`, saves you a lot of trouble.
Abel