code review的时候,发现了如下代码。
第一反应是异常这么用,好奇怪。异常不应该这么用的啊,也许是为了图省事吧。
但是还是应该把异常用在异常情况下,而不是这么误用。这个用法和用异常判断文件的结束没有什么太大的区别。
仔细看了看,不仅仅是异常用的比较奇怪,这个方法是有漏洞的。
明明方法接口的约定是判断长度和数字,但是因为是用Long的valueOf方法,通过异常来实现,结果像"-100000000"这样本来不合法的字符串可以通过这个校验了。
重构后代码如下,直白的完成该方法的接口约定。
/**
* 验证一个字符串是一个长度为10,且内容都是数字。
* */
public static boolean validate(String str) {
if (str == null || str.length() != 10) {
return false;
}
// 纯数字
try {
Long.valueOf(str);
} catch (NumberFormatException ex) {
return false;
}
return true;
}
第一反应是异常这么用,好奇怪。异常不应该这么用的啊,也许是为了图省事吧。
但是还是应该把异常用在异常情况下,而不是这么误用。这个用法和用异常判断文件的结束没有什么太大的区别。
仔细看了看,不仅仅是异常用的比较奇怪,这个方法是有漏洞的。
明明方法接口的约定是判断长度和数字,但是因为是用Long的valueOf方法,通过异常来实现,结果像"-100000000"这样本来不合法的字符串可以通过这个校验了。
重构后代码如下,直白的完成该方法的接口约定。
/**
* 验证一个字符串是一个长度为10,且内容都是数字。
* */
public static boolean validate2(String str) {
if (str == null || str.length() != 10) {
return false;
}
char[] data = new char[str.length()];
str.getChars(0, str.length(), data, 0);
for (int i = 0; i < data.length; i++) {
char c = data[i];
if (c < '0' || c > '9') {
return false;
}
}
return true;
}