一种简单并且统一的处理错误的办法

一种简单并且统一的处理异常的办法
之前做的一个项目中，有一些对异常处理的封装，如果抛出的异常是项目特定的异常，则直接抛出。如果不是，则进行一个封装。

现在的做法是这样的：

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(), "");
        }
    }

一种简单并且统一的处理错误的办法

相关推荐