之前做的一个项目中,有一些对异常处理的封装,如果抛出的异常是项目特定的异常,则直接抛出。如果不是,则进行一个封装。
现在的做法是这样的:
catch (InputValidationFault ive) {
throw new InputValidationFault(ive.getMessage(), "");
} catch (InputLimitFault ile) {
throw new InputLimitFault(ile.getMessage(), "");
} catch (SystemUnavailableFault suf) {
throw new SystemUnavailableFault(suf.getMessage(), "");
} catch (Exception e) {
throw new SystemUnavailableFault(e.getMessage(), "");
}
整个项目中掺杂了大量类似的代码,因为需要确保项目特定的异常不会被转化成System Unavailable Fault,因此不得不在之前首先把它们都catch住并重新抛出去。
这里有点奇怪的,为什么要重新new一个异常抛出呢? 应该是该开发人员对于Java技术的不熟练,以至于写出这样的代码。 其实可以改良成这样一个版本:
catch (InputValidationFault ive) {
throw ive;
}
这样对项目中的senior developer造成了很大的负担,需要不时的review其他人的代码,确保没有错误。
一直在想是否能够对这段代码进行封装,会减少很多犯错的机会,也减轻开发人员的负担。
今天看Tomcat源码,发现了一个不错的做法,将之前的代码稍微改了下,应该可以解决这个问题:
catch (Throwable t) {
handleThrowable(t);
}
...
public static void handleThrowable(Throwable t) throws InputValidationFault,
InputLimitFault, SystemUnavailableFault{
if (t instanceof InputValidationFault) {
throw (InputValidationFault) t;
} else if (t instanceof InputLimitFault) {
throw (InputLimitFault) t;
} else if (t instanceof SystemUnavailableFault) {
throw (SystemUnavailableFault) t;
} else {
throw new SystemUnavailableFault(t.getMessage(), "");
}
}