4

Такая проблема. У меня есть метод

public DriverBadge updateDriverBadges() {
        DriverBadge driverBadge;
        try {
            ServerProxy.GetInstance().GetDriverBadge(new IRemoteCallback<DriverBadge>() {
                @Override
                public void Callback(Exception ex, DriverBadge badge) throws Exception {
        ...
                    driverBadge = badge;
                }
            });
        } catch (Exception e) {
            e.printStackTrace();
        }


        return driverBadge;
    }

Я вызываю с UI метод, который мне ворачивает результат. В этом методе вызывается метод с колбеком, в котором этот результат приходит от сервера. проблема в том, что на вот этом месте driverBadge = badge; компилятор ругается. Как я могу вернуть результат пришедший в колбеке, своим методом?

Кусок кода, где ошибка: введите сюда описание изображения

0

3 ответа 3

6

В Java из анонимного класса нельзя вернуть не final-переменные, поэтому в таком виде вам результат в основном классе не получить.

Самое простое - сделать класс не анонимным, а, к примеру, внутренним или еще лучше, имплементировать интерфейс (коллбэк) в основной класс и переопределить метод в нем - так вы получите полный доступ к возвращаемым значениям в основном классе, передавая эти значения в поля основного класса.

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

Можно еще написать некий метод основного класса, который вы будете вызывать из анонимного класса. Он будет принимать в качестве аргументов какие то значения этого анонимного класса и делать с ними определенные действия.

PS: из кусочка кода в вопросе нельзя сделать завершенные примеры реализаций сказанного выше, а писать абстрактные примеры "с нуля" мне лень :), так что прошу прощения (может тоскливыми зимними вечерами допишу позже).
Вообще, со всеми этими реализациями каждый программист под андроид сталкивается постоянно, когда получает результаты от различных листенеров (слушателей событий), так что с примерами проблем быть не должно.

2

Я не знаю, что это за API-метод (по getDriverBadge гуглится только этот ваш вопрос здесь и на en.SO), но если бы он мог сразу вернуть DriverBadge (или кинуть исключение), он бы это сделал. А раз он работает через callback, видимо, он работает асинхронно и может вызвать этот callback значительно позже. Поэтому обновлять локальную переменную (хоть с одноэлементным массивом как у @holodnsk, хоть превратив её в поле) скорее всего небезопасно: возможна ситуация, когда после выхода из вызова GetDriverBadge ваш callback ещё не будет вызван, а вызовется когда-нибудь позже, может даже из другого потока. Это, кстати, объясняет, почему локальную переменную нельзя менять из анонимного класса. Java гарантирует, что все локальные переменные меняются только текущим методом и текущим потоком. Это невероятно крутая гарантия и очень упрощает многопоточное программирование, делая его надёжным. Отказаться от неё нельзя, иначе Java будет не Java.

Чтобы разрешить эту проблему, вам нужно пересмотреть в корне подход к решению вашей задачи. Для чего updateDriverBadges() возвращает значение? Чтобы потом нарисовать значок в пользовательском интерфейсе? Так и вызовите обновление пользовательского интерфейса прямо из callback, а метод updateDriverBadges() пусть ничего не возвращает.

2
  • Очень хорошо, что вы обратили внимание на эту особенность коллбэка - об этом надо было сказать. Фактически ваше решение укладывается в четвертый пункт моего ответа: вызвать метод основного класса из анонимного и передать значение аргументом, чтобы он что то сделал с этим значением. Так же в данном вопросе речь идет о апдейте, то есть какое то значение уже существует и его надо обновить - в данном случае опреации с полями допустимы. В целом же ваш ответ очень полезен для понимания действия коллбэков.
    – pavlofff
    16 сен 2015 в 7:58
  • @pavlofff, операция с полем, конечно, допустима в том смысле, что приложение не упадёт. Но и UI не перерисуется тогда, ему ж никто не скажет об этом. 16 сен 2015 в 8:33
-1

объяви final массив с одним элементом, и обращайся к этому элементу как твоей душе угодно, однако обрати внимание на ответ Tagir Valeev

public DriverBadge updateDriverBadges() {
        final DriverBadge[] driverBadge = new DriverBadge[1];
        try {
            ServerProxy.GetInstance().GetDriverBadge(new IRemoteCallback<DriverBadge>() {
                @Override
                public void Callback(Exception ex, DriverBadge badge) throws Exception {
        ...
                    driverBadge[0] = badge;
                }
            });
        } catch (Exception e) {
            e.printStackTrace();
        }


        return driverBadge;
    }
2
  • эээ, а вы хоть протестировали своё произведение? Как вы думаете какой оператор первый выполниться - return driverBadge; или driverBadge[0] = badge;? И что вернёт метод?
    – s_klepcha
    16 сен 2015 в 6:39
  • 2
    Сама по себе идея заворачивать значение в какой-то массив из одного элемента видится, как миниум, костылем.
    – pavlofff
    16 сен 2015 в 8:11

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.