код рефакторинга для выдачи 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;
}