Как лучше убрать повтороение кода?

Question

[Дмитрий]

Гении ооп, подскажите как лучше избавиться от повторяющихся кусочков кода в методах, методы разные, возращают разную информуацию, но суть что один и тот же кусочек приходится писать каждый раз

if (mGetRequest != null && mGetRequest.isExecuting())
            return;

        ParamsBuilder paramsBuilder = new ParamsBuilder();
        paramsBuilder.setMethod(BOOKS_METHOD);
        paramsBuilder.setEnumParse(EnumParse.BOOKS);

        if (page != null) paramsBuilder.addParams("page", String.valueOf(page));
        if (search != null) paramsBuilder.addParams("s", search);
        if (userId != null) paramsBuilder.addParams("uid", userId.toString());

        mGetRequest = new HttpRequest(listener);
        mGetRequest.execute(paramsBuilder);

вот этот кусочек (mGetRequest != null && mGetRequest.isExecuting()) приходиться проверять постоянно

Comments

[Therapyx]

а тут что происходит? isExecuting(). Если его надо просто постоянно вызывать, то этого врятле избежать без заумных костылей)) Картина в общем не ясно видна тут..

[Дмитрий]

ну как бы обычное обращение в инет, над проверить, сейчас asynctask работает ли, если работает, то нахер всё

Answer 1

[Евгений Корначев]

Метод, который принимает на вход функциональный интерфейс(лямбду с кодом), внутри метода проверяется условие, и в зависимости от условия лямбда выполняется или нет.

Answer 2

[Дмитрий Макаров]

Не являюсь специалистом по Java, я в основном по С# практикуюсь.
На C# делаю через метод расширения принимающем лямбду. Если объект не null, то запускается лямбда, принимая объект как параметр. Метот расширения возвращает результат лямбды, если объект не null, и null, если объект null.
Удобно. Можно делать цепочки вызовов не проверяя вручную на null или делать вложенные вызовы (кому как удобнее)....
Насколько знаю в Java8 есть и методы расширения и лямбды Новое в Java8.
Евгений Корначев видимо именно это и предлагает.

Answer 3

[maaGames]

bool isExecuting()
{
return mGetRequest != null && mGetRequest.isExecuting();
}

Comments

[Дмитрий]

гениально :-D но этот вариант не подходит, возможно метод сам во что то обернуть чтобы вызывался этот кусок кода в потом уже нужный мне

[sirs]

Дмитрий Конуров: В чем проблема вынести этот метод в утилитный класс, сделать его static и подключать к своему коду через static import?