我通常使用以下习语来检查String是否可以转换为整数。
public boolean isInteger( String input ) {
try {
Integer.parseInt( input );
return true;
}
catch( Exception e ) {
return false;
}
}
是我的错觉，还是你觉得这有点粗俗?还有什么更好的办法吗?
看看我的回答(带有基准，基于CodingWithSpike之前的回答)，看看为什么我改变了立场，接受了Jonas Klemming对这个问题的回答。我认为大多数人都会使用这个原始代码，因为它实现起来更快，也更易于维护，但是当提供非整数数据时，它就慢了几个数量级。
当前回答
对@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;
}
其他回答
这更短，但更短并不一定更好(它不会捕获超出范围的整数值，正如danatel的评论所指出的那样):
input.matches("^-?\\d+$");
就我个人而言，由于实现被保存在一个helper方法中，并且正确性胜过长度，所以我将使用类似于您所拥有的东西(减去捕获基Exception类而不是NumberFormatException)。
你也可以使用Scanner类，并使用hasNextInt() -这也允许你测试其他类型，如浮点数等。
做了一个快速的基准测试。异常实际上并没有那么昂贵，除非您开始弹出多个方法，JVM必须做大量工作才能使执行堆栈就位。如果保持相同的方法，他们就不会表现不好。
public void RunTests()
{
String str = "1234567890";
long startTime = System.currentTimeMillis();
for(int i = 0; i < 100000; i++)
IsInt_ByException(str);
long endTime = System.currentTimeMillis();
System.out.print("ByException: ");
System.out.println(endTime - startTime);
startTime = System.currentTimeMillis();
for(int i = 0; i < 100000; i++)
IsInt_ByRegex(str);
endTime = System.currentTimeMillis();
System.out.print("ByRegex: ");
System.out.println(endTime - startTime);
startTime = System.currentTimeMillis();
for(int i = 0; i < 100000; i++)
IsInt_ByJonas(str);
endTime = System.currentTimeMillis();
System.out.print("ByJonas: ");
System.out.println(endTime - startTime);
}
private boolean IsInt_ByException(String str)
{
try
{
Integer.parseInt(str);
return true;
}
catch(NumberFormatException nfe)
{
return false;
}
}
private boolean IsInt_ByRegex(String str)
{
return str.matches("^-?\\d+$");
}
public boolean IsInt_ByJonas(String str)
{
if (str == null) {
return false;
}
int length = str.length();
if (length == 0) {
return false;
}
int i = 0;
if (str.charAt(0) == '-') {
if (length == 1) {
return false;
}
i = 1;
}
for (; i < length; i++) {
char c = str.charAt(i);
if (c <= '/' || c >= ':') {
return false;
}
}
return true;
}
输出:
ByException: 31 ByRegex: 453(注意:每次都重新编译模式) ByJonas: 16
我也同意Jonas K的解决方案是最稳健的。看起来他赢了:)
你所做的是有效的，但你可能不应该总是这样检查。抛出异常应该为“异常”情况保留(不过，可能适合您的情况)，并且在性能方面代价非常高。
如果你不关心潜在的溢出问题，这个函数将比使用Integer.parseInt()快20-30倍。
public static boolean isInteger(String str) {
if (str == null) {
return false;
}
int length = str.length();
if (length == 0) {
return false;
}
int i = 0;
if (str.charAt(0) == '-') {
if (length == 1) {
return false;
}
i = 1;
}
for (; i < length; i++) {
char c = str.charAt(i);
if (c < '0' || c > '9') {
return false;
}
}
return true;
}
