Following is resulting in an Exception:

String p="1,234";
Double d=Double.valueOf(p); 
System.out.println(d);

Is there a better way to parse "1,234" to get 1.234 than: p = p.replaceAll(",",".");?

  • 17
    In my experience, replaceAll(), as you suggested, is the best way to do this. It doesn't depend on the current locale, it's simple, and it works. – Joonas Pulakka Dec 1 '10 at 11:07
  • @JoonasPulakka your suggestion works only if the current default locale uses a dot as a decimal separator. Right ? – Marco Altieri Dec 24 '16 at 0:36
  • 1
    @Marco Altieri: replaceAll(",",".") replaces all commas with dots. If there are no commas, then it does nothing. Double.valueOf() works (only) with strings that use dot as decimal separator. Nothing here is affected by current default locale. docs.oracle.com/javase/8/docs/api/java/lang/… – Joonas Pulakka Dec 27 '16 at 19:38
  • 3
    The only problem with replaceAll(",",".") is that it'll only work if there is a single comma: ie: 1,234,567 will throw java.lang.NumberFormatException: multiple points. A regex with positive lookahead will suffice p.replaceAll(",(?=[0-9]+,)", "").replaceAll(",", ".") More at: regular-expressions.info/lookaround.html – artemisian Feb 13 '17 at 17:17
  • 2
    There is no problem. The NumberFormatException is good. How can you know which comma is the right one? The format is wrong and all you can do is show a better readable message than the exception to the user. – The incredible Jan Jan 31 at 11:56
up vote 166 down vote accepted

Use java.text.NumberFormat:

    NumberFormat format = NumberFormat.getInstance(Locale.FRANCE);
    Number number = format.parse("1,234");
    double d = number.doubleValue();
  • 7
    This only works if the current default locale happens to use a comma as a decimal separator. – Joonas Pulakka Dec 1 '10 at 11:05
  • 6
    To further mess things up, some locales use comma as a thousands separator, in which case "1,234" would parse to 1234.0 instead of throwing an error. – Joonas Pulakka Dec 1 '10 at 11:11
  • 12
    The problem with NumberFormat is that it will silently ignore invalid characters. So if you try to parse "1,23abc" it will happily return 1.23 without indicating to you that the passed-in String contained non-parsable characters. In some situations that might actually be desirable, but I don't think it's usually the desired behavior. – E-Riz Jan 17 '13 at 19:37
  • 5
    for TURKEY, you should use NumberFormat.getInstance(new Locale(tr_TR)) – Günay Gültekin Jul 27 '13 at 9:45
  • 1
    for who uses what seperator see en.wikipedia.org/w/… – fiffy Nov 21 '14 at 7:09
Double.parseDouble(p.replace(',','.'))

...is very quick as it searches the underlying character array on a char-by-char basis. The string replace versions compile a RegEx to evaluate.

Basically replace(char,char) is about 10 times quicker and since you'll be doing these kind of things in low-level code it makes sense to think about this. The Hot Spot optimiser will not figure it out... Certainly doesn't on my system.

This is the static method I use in my own code:

public static double sGetDecimalStringAnyLocaleAsDouble (String value) {

    if (value == null) {
        Log.e("CORE", "Null value!");
        return 0.0;
    }

    Locale theLocale = Locale.getDefault();
    NumberFormat numberFormat = DecimalFormat.getInstance(theLocale);
    Number theNumber;
    try {
        theNumber = numberFormat.parse(value);
        return theNumber.doubleValue();
    } catch (ParseException e) {
        // The string value might be either 99.99 or 99,99, depending on Locale.
        // We can deal with this safely, by forcing to be a point for the decimal separator, and then using Double.valueOf ...
        //http://stackoverflow.com/questions/4323599/best-way-to-parsedouble-with-comma-as-decimal-separator
        String valueWithDot = value.replaceAll(",",".");

        try {
          return Double.valueOf(valueWithDot);
        } catch (NumberFormatException e2)  {
            // This happens if we're trying (say) to parse a string that isn't a number, as though it were a number!
            // If this happens, it should only be due to application logic problems.
            // In this case, the safest thing to do is return 0, having first fired-off a log warning.
            Log.w("CORE", "Warning: Value is not a number" + value);
            return 0.0;
        }
    }
}
  • 5
    What if the default Locale is something like German, where a comma denotes a decimal place? You could pass in, for example "1,000,000" which wouldn't parse into German Locale and would then be replaced by "1.000.000" which is not a valid Double. – Eddie Curtis Jan 27 '15 at 17:01
  • Hi @jimmycar, I've just updated my answer to use the the current version of my static method. I hope this solves your problem! Pete – Pete Sep 3 '15 at 14:36

This would do the job:

Double.parseDouble(p.replace(',','.')); 
  • 5
    The initial question said "Is there a better way to parse "1,234" to get 1.234 than: p = p.replaceAll(",",".");", if you think replace significantly differs from using replaceAll, please explain why. – SuperBiasedMan Aug 5 '15 at 9:24

As E-Riz points out, NumberFormat.parse(String) parse "1,23abc" as 1.23. To take the entire input we can use:

public double parseDecimal(String input) throws ParseException{
  NumberFormat numberFormat = NumberFormat.getNumberInstance(Locale.getDefault());
  ParsePosition parsePosition = new ParsePosition(0);
  Number number = numberFormat.parse(input, parsePosition);

  if(parsePosition.getIndex() != input.length()){
    throw new ParseException("Invalid input", parsePosition.getIndex());
  }

  return number.doubleValue();
}

If you don't know the correct Locale and the string can have a thousand separator this could be a last resort:

    doubleStrIn = doubleStrIn.replaceAll("[^\\d,\\.]++", "");
    if (doubleStrIn.matches(".+\\.\\d+,\\d+$"))
        return Double.parseDouble(doubleStrIn.replaceAll("\\.", "").replaceAll(",", "."));
    if (doubleStrIn.matches(".+,\\d+\\.\\d+$"))
        return Double.parseDouble(doubleStrIn.replaceAll(",", ""));
    return Double.parseDouble(doubleStrIn.replaceAll(",", "."));

Be aware: this will happily parse strings like "R 1 52.43,2" to "15243.2".

You can use this (the French locale has , for decimal separator)

NumberFormat nf = NumberFormat.getInstance(Locale.FRANCE);
nf.parse(p);

Or you can use java.text.DecimalFormat and set the appropriate symbols:

DecimalFormat df = new DecimalFormat();
DecimalFormatSymbols symbols = new DecimalFormatSymbols();
symbols.setDecimalSeparator(',');
symbols.setGroupingSeparator(' ');
df.setDecimalFormatSymbols(symbols);
df.parse(p);
  • Yes... if we don't set the thousand grouping separator and just use French format, a number in Spanish format (1.222.222,33) will be converted to "1 222 222,33", which is not what I want. So thanks! – WesternGun Mar 30 '17 at 14:55
  • Another thing is, Spanish locale is not listed as "default` and I cannot construct a Locale with correct format with new Locale("es", "ES") and then automatically parse the number string with NumberFormat, with , as the decimal separator and . as the thousand group separator. Only DecimalFormat works. – WesternGun Mar 30 '17 at 15:11

You of course need to use the correct locale. This question will help.

Your Answer

 

By clicking "Post Your Answer", you acknowledge that you have read our updated terms of service, privacy policy and cookie policy, and that your continued use of the website is subject to these policies.

Not the answer you're looking for? Browse other questions tagged or ask your own question.