Parsing decimal with thousands separator

Question

I have the following block of code:

string price = \"1,234.56\";
decimal value = 0;
var allowedStyles = (NumberStyles.AllowDecimalPoint & NumberStyles.AllowT         


        

Answer 1

The result of this binary and (&) will always be 0 (false, or NumberStyles.None). That's why it doesn't allow decimal and thousand separators:

var allowedStyles = (NumberStyles.AllowDecimalPoint & NumberStyles.AllowThousands);

Change to binary or (|):

var allowedStyles = (NumberStyles.AllowDecimalPoint | NumberStyles.AllowThousands);

Answer 2

If you AND-combine the NumberStyles-flag, you will get None.

00100000 (AllowDecimalPoint)
&
01000000 (AllowThousands)
--------
00000000 (None)

Try to OR-combine them: NumberStyles.AllowDecimalPoint | NumberStyles.AllowThousands

00100000 (AllowDecimalPoint)
|
01000000 (AllowThousands)
--------
01100000 (AllowDecimalPoint, AllowThousands)

Additionally, I'm afraid that you can't parse both styles (US style and DE style) with one statement.

So I'd try both:

string price = "1,234.56";
decimal value = 0;
var allowedStyles = (NumberStyles.AllowDecimalPoint | NumberStyles.AllowThousands);

if (Decimal.TryParse(price, allowedStyles, CultureInfo.GetCultureInfo("DE-de"), out value))
{
    Console.Write("Danke!");
}
else if (Decimal.TryParse(price, allowedStyles, CultureInfo.GetCultureInfo("EN-us"), out value))
{
    Console.Write("Thank you!");
}
else
{
    throw new InvalidFormatException();
}