你知道ApacheCommon包中isNumeric方法是坑吗?
发布网友
发布时间：2024-04-01 12:00
共1个回答
热心网友
时间：2024-04-01 15:58
本文就来分享一个这个坑的情况，以免其它tx也掉坑中
费话不多说，来看代码： package chapter4; import org.apache.commons.lang.StringUtils; /** * Created by MyWorld on 2016/3/24. */ public class StringUtilsIsNumericChecker { public static void main(String[] args) { System.out.println(StringUtils.isNumeric("1")); System.out.println(StringUtils.isNumeric("-1")); } }
围观的tx，认为上面api的输出结果会是什么呢？两个true? 好吧，执行一行看看结果： true false
什么情况，-1不是数字吗？ 为什么是false呢 来分析下源码： public static boolean isNumeric(String str) { if (str == null) { return false; } int sz = str.length(); for (int i = 0; i sz; i++) { if (Character.isDigit(str.charAt(i)) == false) { return false; } } return true; }
源码中判断是否数字的依据是JDK的API： java.lang.Character.isDigit(str.charAt(i)) 看看个API的源码： public static boolean isDigit(char ch) { return isDigit((int)ch); }
看看isDigit(int codePoint)的源码： public static boolean isDigit(int codePoint) { boolean bDigit = false; if (codePoint = MIN_CODE_POINT codePoint = FAST_PATH_MAX) { bDigit = CharacterDataLatin1.isDigit(codePoint); } else { int plane = getPlane(codePoint); switch(plane) { case(0): bDigit = CharacterData00.isDigit(codePoint); break; case(1): bDigit = CharacterData01.isDigit(codePoint); break; case(2): bDigit = CharacterData02.isDigit(codePoint); break; case(3): // Undefined case(4): // Undefined case(5): // Undefined case(6): // Undefined case(7): // Undefined case(8): // Undefined case(9): // Undefined case(10): // Undefined case(11): // Undefined case(12): // Undefined case(13): // Undefined bDigit = CharacterDataUndefined.isDigit(codePoint); break; case(14): bDigit = CharacterData0E.isDigit(codePoint); break; case(15): // Private Use case(16): // Private Use bDigit = CharacterDataPrivateUse.isDigit(codePoint); break; default: // the argument's plane is invalid, and thus is an invalid codepoint // bDigit remains false; break; } } return bDigit; }
下面还有更深的调用，貌似还涉及到ASCII码了。 水太深，就不继续看了。 有一点是肯定的，这个API不是通过类似Regex expression的方式来判断是数字，而通过每个字符的ASCII的值类确定的 回到API的isNumeric(String str)， 看看Doc是怎么说的： /** * pChecks if the String contains only unicode digits. * A decimal point is not a unicode digit and returns false./p * * pcodenull/code will return codefalse/code. * An empty String (length()=0) will return codetrue/code./p * * pre * StringUtils.isNumeric(null) = false * StringUtils.isNumeric("") = true * StringUtils.isNumeric(" ") = false * StringUtils.isNumeric("123") = true * StringUtils.isNumeric("12 3") = false * StringUtils.isNumeric("ab2c") = false * StringUtils.isNumeric("12-3") = false * StringUtils.isNumeric("12.3") = false * /pre * * @param str the String to check, may be null * @return codetrue/code if only contains digits, and is non-null */
看完上面的Doc，感觉水好深。 这个API的方法名直接命名为isInt不就完了。方法名很容易误导人 这也是给开发的tx敲了一个警钟， api使用之前一定要确认清楚，至少看看Doc文档吧
