0

Здравствуйте.

Использую официальное VK SDK и хочу получить список пользователей VK начиная с конкретного id и заканчивая каким-то числом. В документации ничего про множественные значения параметров не нашел и решил сделать тупо циклом while. Например, получить 20 пользователей.

do{
            VKRequest request = VKApi.users().get(VKParameters.from(VKApiConst.FIELDS, "photo_50", VKApiConst.USER_ID, String.valueOf(id)));

            request.executeWithListener(new VKRequest.VKRequestListener() {
                @Override
                public void onComplete(VKResponse response) {
                    super.onComplete(response);

                    VKList<VKApiUserFull> vkList = (VKList<VKApiUserFull>) response.parsedModel;

                    for (VKApiUserFull userFull : vkList) {
                        ... 
                        //фильтрация значений. Если проходят проверку, то счетчик увеличивается. Если нет, то - break
                        count++;
                    }
                }
            });
            id++;
        } while (count < 20);

Как уже было сказано выше, этот тупой цикл работает. Но есть проблема в том, что после своего выполнения он почему-то начинается заново. И продолжается до бесконечности. Сам цикл находится внутри метода doInBackground AsyncTask'а. При этом, после окончания цикла выполняется метод onPostExecute. А потом снова начинается цикл. Причем с того же самого момента, где он был остановлен. Т.е. с индексами 21, 22, 23 и т.д. Сидел с дебагером больше часа, но так и не смог понять, почему он уходит в бесконечный цикл. Вызывается AsyncTask по нажатию Button'a, поэтому зациклиться вызов там не мог.

Если описать все кратко, то все идет по такому сценарию:

  1. Запускается Activity.
  2. Запускается AsyncTask по нажатию клавиши.
  3. Цикл проходит нужное число раз внутри doInBackground и прекращается.
  4. Запускается onPostExecute в AsyncTask.
  5. Проходит пару секунд и цикл снова запускается с того места, где он был остановлен.

Где мой промах?

UPD То, что счетчик инкрементируется внутри самого запроса - не случайная ошибка. Он увеличивается только после того, как значения прошли фильтр. Именно в этом и состоит моя пробелма: мне нужно как-то остановить цикл не после 20 прохождений по нему, а после получения 20 объектов внутри onComplete запроса.

Избавиться от счетчика тоже не получается: while (arrayList.size() < 20) тоже приводит к бесконечному циклу.

Заранее благодарю за ответ.

1 ответ 1

4

Вы не там увеличиваете значение счётчика. Вы делаете это в методе onComplete, а он вызывается только после успешного выполнения запроса... Вообще не очень понятно что вы хотели этим кодом получить. И что за внутренний цикл ещё?..

У вас сейчас такой порядок действий в программе:

  1. Заходим в do
  2. Стартуем запрос
  3. Инкреминируем id
  4. Заходим в условие продолжения цикла и проверяем что count меньше 20
  5. Возвращаемся к п.1 до тех пор пока не выполнится успешно 20 запросов.

Именно успешно выполнится, а не стартует, т.к. увеличиваете вы счётчик только после окончания запроса, а не после его старта. Да ещё и в цикле каком-то...

Вам надо как-то так:

do{
    VKRequest request = VKApi.users().get(VKParameters.from(VKApiConst.FIELDS, "photo_50", VKApiConst.USER_ID, String.valueOf(id)));

    request.executeWithListener(new VKRequest.VKRequestListener() {
        @Override
        public void onComplete(VKResponse response) {
            super.onComplete(response);

            VKList<VKApiUserFull> vkList = (VKList<VKApiUserFull>) response.parsedModel;

            for (VKApiUserFull userFull : vkList) {
                ...
               //count++;
            }
        }
    });
    count++;
    id++;
} while (count < 20);

Если же вам надо именно после выполнения задачи инкременировать счётчик, то и следующие задачи надо запускать только после выполнения предыдущей. Т.е. как-то так:

int id = 0;

public void foo(int count) {
    if (count>=20){return;}

    VKRequest request = VKApi.users().get(VKParameters.from(VKApiConst.FIELDS, "photo_50", VKApiConst.USER_ID, String.valueOf(id)));

    request.executeWithListener(new VKRequest.VKRequestListener() {
        @Override
        public void onComplete(VKResponse response) {
            super.onComplete(response);

            VKList<VKApiUserFull> vkList = (VKList<VKApiUserFull>) response.parsedModel;

            for (VKApiUserFull userFull : vkList) {
                ... 
                //фильтрация значений. Если проходят проверку, то счетчик увеличивается. Если нет, то - break
                count++;
            }
            foo(count);
        }
    });
    id++;
}
10
  • Это, конечно, верно, но мне нужно каким-то образом инкрементрировать счетчик внутри самого запроса, потому что там у меня находится (вместо ...) фильтрация получаемых значений. Т.е. мне нужно, не чтобы цикл прошел 20 раз, а чтобы было получено 20 объектов с нужными мне параметрами. Если запустить цикл в таком виде, в котором Вы предложили, то на выходе получится ArrayList из 1-2 объектов (остальные были отфильтрованы). А мне нужно получить именно 20 объектов. Если делать While до arrayList.size(), то тоже уходит в бесконечный цикл. В общем, данное решение мне не совсем подходит.
    – ahgpoug
    17 окт 2016 в 9:08
  • @ahgpoug, посмотрите обновление ответа - попробуйте запускать задачи рекурсивно, стартуя следующую только после выполнения текущей.
    – ЮрийСПб
    17 окт 2016 в 11:23
  • Посмотрел ответ. Как минимум, использование переменной count здесь невозможно, потому что в данном случае она должна быть final. Решил избавиться от нее, убрав ее из параметров метода и заменив проверку на if (arrayList.size() >= 20). Метод, вроде, выполняется, но делает это не нужное число раз. Т.е. в результате получается arrayList размера рандомного размера от 0 до 20. Не очень понимаю, почему это происходит, ведь рекурсия даже не доходит до того момента, когда ее нужно остановить.
    – ahgpoug
    17 окт 2016 в 12:01
  • UPD: вообще этот рекурсивный метод почему-то больше 7-8 не возвращает. Поставил условие на >=100, но возвращается он все равно 4-8.
    – ahgpoug
    17 окт 2016 в 14:22
  • @ahgpoug, может у вас не все запросы успешны, onComplete где-то не вызвался и больше ничто метод не вызвает. Вам бы надо добавить обработку ошибок
    – ЮрийСПб
    17 окт 2016 в 14:47

Ваш ответ

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

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