Calculations Based on CultureInfo.CurrentCulture

Question

0.00/5 (No votes)

See more:

Hi,

I have two text boxes, One will be used quantity and the other will be used to enter a Unit price.

what I would like to do is :

int or decimal TotalPrice = Unit price * quantity.

what I do in the textbox is:

XML

private void Price_Text_Leave(object sender, EventArgs e)
{
    Price_Text.Text = Price_Text.Text.Replace('.', ',');
    Double value;
    if (Double.TryParse(Price_Text.Text, out value))
        Price_Text.Text = String.Format(System.Globalization.CultureInfo.CurrentCulture, "{0:C2}", value);
    else
        Price_Text.Text = String.Empty;
}


private void Price_Text_KeyPress(object sender, KeyPressEventArgs e)
{
    if (!char.IsControl(e.KeyChar) && !char.IsDigit(e.KeyChar) && e.KeyChar != '.' && e.KeyChar != ',')
    {
        e.Handled = true;
    }
}

What I have tried is:

decimal TotalPriceValue = Convert.ToDecimal(UnitPrice.ToString()) * Convert.ToDecimal(Quantity_Text.Text.ToString());

and

int TotalPriceValue = int.parse(UnitPrice.ToString()) * int.parse(Quantity_Text.Text.ToString());

the error I get is :
Input string was not in a correct format.

The error is because of the currency letter(Im my case "R") amd the point "."

I have done this to overcome the error:

C#

string UnitPrice = Price_Text.Text.Replace(',', '.');
UnitPrice = UnitPrice.Replace('R', '0');
UnitPrice = UnitPrice.Replace(' ', '0');

but then I have the issue of inserting the decimal point at the correct place in the number after my calculation is done.

Is there any other calculation I can rather do to help this?

Please excuse my question I am new to c#.

Posted 16-Jun-14 23:49pm

Dr Drastic

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

phil.o · Accepted Answer · 2014-06-17T00:37:00

Why are you managing everything as string? A string is not very useful when it comes to make computations.
What you have to do is to get some decimal from your inputs, and make the computations on these decimals.
Something like:

C#

decimal price;
int quantity;
decimal totalPriceValue;
if (decimal.TryParse(Price_Text.Text, out price) && int.TryParse(Quantity_Text.Text, out quantity)) {
   totalPriceValue = price * quantity;
}
else {
   // Here there was a problem parsing the price and/or the quantity. You have to decide what to do in this case
}

Hope this helps.

PS: there is no point in calling ToString() on a property which already is a string (i.e., the Text property of your TextBoxes).