java.lang.NumberFormatException: Invalid float: "55,4" 分析

问题

新版本上线，4天时间一个错误出现一百多次，如下：

java.lang.NumberFormatException: Invalid float: "55,4"    at java.lang.StringToReal.invalidReal(StringToReal.java:63)    at java.lang.StringToReal.initialParse(StringToReal.java:164)    at java.lang.StringToReal.parseFloat(StringToReal.java:323)    at java.lang.Float.parseFloat(Float.java:306)    at com.boohee.database.OnePreference.setLatestWeight(OnePreference.java:169)    at com.boohee.utils.AccountUtils$1.ok(AccountUtils.java:102)    at com.boohee.one.http.JsonCallback.onResponse(JsonCallback.java:69)    at com.boohee.one.http.JsonCallback.onResponse(JsonCallback.java:33)    at com.android.volley.toolbox.JsonRequest.deliverResponse(JsonRequest.java:72)    at com.android.volley.ExecutorDelivery$ResponseDeliveryRunnable.run(ExecutorDelivery.java:99)    at android.os.Handler.handleCallback(Handler.java:739)    at android.os.Handler.dispatchMessage(Handler.java:95)    at android.os.Looper.loop(Looper.java:148)    at android.app.ActivityThread.main(ActivityThread.java:7326)    at java.lang.reflect.Method.invoke(Native Method)    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)

分析

发现此问题后感觉很熟悉，以前应该是见过，但是忘记怎么解决的了，没办法，踩过的坑再踩一遍。涉及的代码如下：

public class OnePreference{    ...    public static void setLatestWeight(float weight) {       getInstance(MyApplication.getContext()).putFloat(PREFS_LATEST_WEIGHT, Float.parseFloat(NumberUtils.formatFloatWithOneDot(weight)));    }    ...}public class NumberUtils {    ...    public static String formatFloatWithOneDot(float number) {        number = Math.round(number * 10) / 10f;        return String.format("%.1f", number);    }    ...}

• 确认问题

  android无法复现，iOS后台未报此错误

• 怀疑是数据的问题

  这是一个记录体重的功能，新版体重输入已经做了限制，所以怀疑可能是老数据的问题。因此找后端的同事降此数据做处理：如果此字段无法解析为数字则过滤掉。
  但是遗憾的，后端发布之后问题依然存在。

• 分析代码

  setLatestWeight(float weight) 这个传入的是float，所以之前的假设是错误的 —— 如果是数据的问题，那根本走不到方法里。
  这样的话那就是 formatFloatWithOneDot(float number) 这个方法的问题，但是想了下之前都有这么写过 —— 将float类型格式化为一位小数的字符串，应该没问题啊。那会是怎样呢？可能很自然会想到国际化的问题。遂作了一个简单的测试：

  public class ExampleUnitTest {@Testpublic void testStringLocale() throws Exception {    Locale[] locales = new Locale[]{            Locale.CANADA,            Locale.CANADA_FRENCH,            Locale.CHINESE,            Locale.ENGLISH,            Locale.FRANCE,            Locale.GERMAN,            Locale.GERMANY,            Locale.ITALIAN,            Locale.ITALY,            Locale.JAPAN,            Locale.JAPANESE,            Locale.KOREA,            Locale.KOREAN,            Locale.PRC,            Locale.ROOT,            Locale.SIMPLIFIED_CHINESE,            Locale.TAIWAN,            Locale.TRADITIONAL_CHINESE,            Locale.UK,            Locale.US    };    String weightString = null;    for (Locale locale : locales) {        try {            weightString = formatFloatWithOneDot(locale, 55.4f);            float weight = Float.parseFloat(weightString);        } catch (NumberFormatException e) {            System.out.println(locale + ">>>>>" + weightString + ">>>>>>>>>> error");            continue;        }        System.out.println(locale + ">>>>>" + weightString);    }}public static String formatFloatWithOneDot(Locale locale, float number) {    number = Math.round(number * 10) / 10f;    return String.format(locale, "%.1f", number);}}

  输出结果如下：

  en_CA>>>>>55.4fr_CA>>>>>55,4>>>>>>>>>> errorzh>>>>>55.4en>>>>>55.4fr_FR>>>>>55,4>>>>>>>>>> errorde>>>>>55,4>>>>>>>>>> errorde_DE>>>>>55,4>>>>>>>>>> errorit>>>>>55,4>>>>>>>>>> errorit_IT>>>>>55,4>>>>>>>>>> errorja_JP>>>>>55.4ja>>>>>55.4ko_KR>>>>>55.4ko>>>>>55.4zh_CN>>>>>55.4>>>>>55.4zh_CN>>>>>55.4zh_TW>>>>>55.4zh_TW>>>>>55.4en_GB>>>>>55.4en_US>>>>>55.4Process finished with exit code 0

  我们可以看到，在不同的语言下 String.format(locale, "%.1f", number) 格式化的结果是不一样的，55.4可能会被格式化为55,4，然后就出现了上面的错误。将我们的测试机语言设置成出错的语言，问题可以复现。

  再来看下后台的错误：

  !错误信息

  • 大部分是华为的设备，然后是三星，索尼
  • 各个版本都有，从4.4到最新的7.0
  • 不同的渠道都有，主要的渠道来自华为

  这些信息也从侧面反映了问题的原因。看来华为的国际化之路还是很屌的。

解决

• 由于我们并没有做国际化的适配，因此这里改起来还是很方便的，直接给 String.format() 方法指定Locale信息：

  String.format(Locale.ENGLISH, "%.1f", number);

  or

  String.format(Locale.CHINA, "%.1f", number);

• NumberFormat(DecimalFormat)