一种简单并且统一的处理异常的办法

之前做的一个项目中,有一些对异常处理的封装,如果抛出的异常是项目特定的异常,则直接抛出。如果不是,则进行一个封装。

现在的做法是这样的:
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(), "");
        }
    }

你可能感兴趣的:(tomcat)