Код рефакторинга для генерации RuntimeException вместо возврата значения

Мне нужно реорганизовать существующий код из-за дублирования кода.

Следующая структура встречается более 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.

Любая идея, как преобразовать эти две строки кода в новый метод и сохранить логику исходного метода нетронутой?

java refactoring runtimeexception

31.05.2019 01:08

Пользовательский скаляр GraphQL

Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...

Как вычислять биты и понимать побитовые операторы в Java - объяснение с примерами

В компьютерном программировании биты играют важнейшую роль в представлении и манипулировании данными на двоичном уровне. Побитовые операции...

Поднятие тревоги для долго выполняющихся методов в Spring Boot

Приходилось ли вам сталкиваться с требованиями, в которых вас могли попросить поднять тревогу или выдать ошибку, когда метод Java занимает больше...

Принятие принципов SOLID в Spring Boot: Создание обслуживаемых и масштабируемых приложений

Коллекции (ArrayList , HashSet , HashMap)

Полный курс Java для разработчиков веб-сайтов и приложений

Получите сертификат Java Web и Application Developer, используя наш курс.

Перейти к ответу Данный вопрос помечен как решенный

Ответы 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;
}

31.05.2019 01:11

Другие вопросы по теме

Как мне создать класс, который поможет мне регистрировать действия пользователя моего приложения? (вопрос о чистом кодировании и рефакторинге кода)

PhpStorm: заменить строковый литерал константой класса

Рефакторинг шаблона регулярного выражения в шаблон вкуса Java

Приведение моих readPackageFiles в соответствие с функцией "Parallel"

Рефакторинг записи ArrayList в файл

Алгоритм улитки - управление максимальной / минимальной позицией для обхода

Очистка повторяющихся тегов Material UI

Динамически находите правильный метод, чтобы не повторять один и тот же код

Как обрезать внутри цикла for в Twig

Как я могу очистить эту функцию сортировки выбором в Python?

Код рефакторинга для генерации RuntimeException вместо возврата значения

Ответы 1

Другие вопросы по теме

Похожие вопросы