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(",",".");?
Join Stack Overflow to learn, share knowledge, and build your career.
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(",",".");?
NumberFormat format = NumberFormat.getInstance(Locale.FRANCE);
Number number = format.parse("1,234");
double d = number.doubleValue();
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;
}
}
}
This would do the job:
Double.parseDouble(p.replace(',','.'));
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);
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.
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:38replaceAll(",",".")is that it'll only work if there is a single comma: ie: 1,234,567 will throwjava.lang.NumberFormatException: multiple points. A regex with positive lookahead will sufficep.replaceAll(",(?=[0-9]+,)", "").replaceAll(",", ".")More at: regular-expressions.info/lookaround.html – artemisian Feb 13 '17 at 17:17