欢迎您访问程序员文章站本站旨在为大家提供分享程序员计算机编程知识!
您现在的位置是: 首页

实践中的重构12_不要乱用异常

程序员文章站 2022-03-03 18:39:37
...
code review的时候,发现了如下代码。

	/**
* 验证一个字符串是一个长度为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;
}