我通常使用以下习语来检查String是否可以转换为整数。
public boolean isInteger( String input ) {
try {
Integer.parseInt( input );
return true;
}
catch( Exception e ) {
return false;
}
}
是我的错觉，还是你觉得这有点粗俗?还有什么更好的办法吗?
看看我的回答(带有基准，基于CodingWithSpike之前的回答)，看看为什么我改变了立场，接受了Jonas Klemming对这个问题的回答。我认为大多数人都会使用这个原始代码，因为它实现起来更快，也更易于维护，但是当提供非整数数据时，它就慢了几个数量级。
当前回答
这里有几个回答说尝试解析为整数并捕获NumberFormatException，但您不应该这样做。
这种方法将创建异常对象，并在每次调用它时生成堆栈跟踪，而且它不是整数。
Java 8中更好的方法是使用流:
boolean isInteger = returnValue.chars().allMatch(Character::isDigit);
其他回答
对@Jonas K answer进行了一点改进，这个函数将排除一个像“*”这样的操作符。
public boolean isInteger(String str) {
// null pointer
if (str == null) {
return false;
}
int len = str.length();
// empty string
if (len == 0) {
return false;
}
// one digit, cannot begin with 0
if (len == 1) {
char c = str.charAt(0);
if ((c < '1') || (c > '9')) {
return false;
}
}
for (int i = 0; i < len; i++) {
char c = str.charAt(i);
// check positive, negative sign
if (i == 0) {
if (c == '-' || c == '+') {
continue;
}
}
// check each character matches [0-9]
if ((c < '0') || (c > '9')) {
return false;
}
}
return true;
}
Number number;
try {
number = NumberFormat.getInstance().parse("123");
} catch (ParseException e) {
//not a number - do recovery.
e.printStackTrace();
}
//use number
这更短，但更短并不一定更好(它不会捕获超出范围的整数值，正如danatel的评论所指出的那样):
input.matches("^-?\\d+$");
就我个人而言，由于实现被保存在一个helper方法中，并且正确性胜过长度，所以我将使用类似于您所拥有的东西(减去捕获基Exception类而不是NumberFormatException)。
您拥有它，但是您应该只捕获NumberFormatException。
对于那些像我一样多年后才来到这里的读者，对于这个问题，我有一个更通用的解决方案。
/**
* Checks, if the string represents a number.
*
* @param string the string
* @return true, if the string is a number
*/
public static boolean isANumber(final String string) {
if (string != null) {
final int length = string.length();
if (length != 0) {
int i = 0;
if (string.charAt(0) == '-') {
if (length == 1) {
return false;
}
i++;
}
for (; i < length; i++) {
final char c = string.charAt(i);
if ((c <= PERIOD) || ((c >= COLON))) {
final String strC = Character.toString(c).toUpperCase();
final boolean isExponent = strC.equals("E");
final boolean isPeriod = (c == PERIOD);
final boolean isPlus = (c == PLUS);
if (!isExponent && !isPeriod && !isPlus) {
return false;
}
}
}
return true;
}
}
return false;
}
