重构代码以抛出 RuntimeException 而不是返回值

Question

由于代码重复，我需要重构现有代码。

以下结构在一个疯狂的类中出现超过 10 次：

public MyType doSomething(...) {

    MyType myType = ........

    if (myType == null) {

        final String message = "...";

        LOGGER.error(message);

        throw new XxxRuntimeException(message));

    }

    return myType;

}

我想重构LOGGER.error并throw new RuntimeException使用这样的新方法：

private void logErrorAndThrowRuntimeException(String message) {

    LOGGER.error(message);

    throw new XxxRuntimeException(message));

}

if问题是重构后条件内没有返回值。

我无法将异常类型从 更改RuntimeException为Exception因为此应用程序具有疯狂的逻辑并且需要抛出 RuntimeExceptin。

知道如何将这两行代码重构为一个新方法并保持原始方法的逻辑不变吗？

Answer 1

声明一个 Throwable 返回类型：private XxxRuntimeException logErrorAndThrowRuntimeException(String message) {    LOGGER.error(message);    // You can throw here, or return if you'd prefer.    throw new XxxRuntimeException(message));}然后你可以在调用站点抛出这个来表示if主体不能正常完成：public MyType doSomething(...) {    MyType myType = ........    if (myType == null) {        final String message = "...";        throw logErrorAndThrowRuntimeException(message);    }    return myType;}