Как управлять состоянием Activity, внутри onActivityResult после вызова startActivityForResult?

Question

[archduke]

Например, есть список элементов внутри RecyclerView. И нужно добавить фотографию для определенного элемента в списке после клика по нему. Как это сделать наиболее коректно, чтоб внутри onActivityResult можно было доставить фотографию к элементу по позиции, которая была нажата.
Мне известен только один вариант, это держать нажатую позицию внутри переменной у Activity, и использовать ее в onActivityResult. Но возможно, есть подход который позволяет избежать создание и управление дополнительным стейтом в Activity?

Answer 1

[Александр Юдаков]

Если не хочется хранить состояние, можно передавать идентификатор элемента туда-обратно в интентах:

1. При вызове startActivityForResult добавляем в интент идентификатор элемента.
2. При возврате результата методом setResult() добавляем в интент кроме фотографии, еще и идентификатор, полученный ранее.
3. При обработке полученного результата в onActivityResult() получаем из интента фотографию и идентификатор элемента.

UPD:
Признаться, у меня для подобных задач реализован более удобный API.
При вызове startActivityForResult() сразу передаю обработчик результата.
А вся работа с requestCode скрыта в черном ящике.

public class ShellActivity extends AppCompatActivity {

    public interface ActivityResultHandler {
        void onActivityResult(int resultCode, Intent intent);
    }

    public void startActivityForResult(Intent intent, ActivityResultHandler handler) {
        // implementation
    }
}

Upd 2. А в Kotlin-е еще интереснее получается, вызов в одну сроку:

/**
 * Возвращает Intent (если RESULT_OK), либо null (если RESULT_CANCELED)
 * */
suspend fun ShellActivity.startActivityForResult(request: Intent): Intent? {
    return suspendCoroutine {
        this.startActivityForResult(request) { resultCode, response ->
            if (resultCode == RESULT_OK) {
                it.resume(response!!)
            } else {
                it.resume(null)
            }
        }
    }
}

Comments

[Денис Загаевский]

Скорее всего, он открывает активити камеры. Если это не так и это действительно его активити - ССЗБ, не надо так делать.

[archduke]

Пример с запуском активити просто для наглядности взял, подобная ситуация так же может возникнуть и с запросом Permissions, если надо вызвать какой то метод после проверки и запроса Permissions, и у метода несколько может быть папаметров, получается тоже придется все параметры заводить в состояние, иначе в onRequestPermissionsResult, получить параметры для метода не получится.
А если свое Activity, то в интенте можно передавать параметр позиции, но в этом случае получается что Activity не очень изолирована, она должна еще управлять параметром позиции, хотя ее задача только возвращать результат для получения фотогафии.
Возможно в этом случае не плохим может показаться вариант, если запускать отдельное Activity, которое будет что то вроде прокси, которое будет сохранять состояние и запускать Activity для получения фотографии, и у него уже в onActivityResult доставлять результат в вызывающую активити с позицией элемента и фотографией полученной из вызова? Хотя может и такой себе вариант, не очень.

[archduke]

Александр Юдаков , а можно подробней как это работает? Получается ActivityResultHandler все равно куда то нужно сохранять в глобальное состояние, чтоб его можно было вызвать в onActivityResult?

[Александр Юдаков]

archduke, состояние хранится в вызывающей Activity.
Упрощенный вариант реализации выглядит так:

import android.content.Intent;
import android.support.v7.app.AppCompatActivity;
import java.util.HashMap;

public class ShellActivity extends AppCompatActivity {

    public interface ActivityResultHandler {
        void onActivityResult(int resultCode, Intent intent);
    }

    public void startActivityForResult(Intent intent, ActivityResultHandler handler) {
        int nextRequestCode = ++customRequestCode;
        activityResultHandlers.put(nextRequestCode, handler);
        super.startActivityForResult(intent, nextRequestCode);
    }

    public void onActivityResult(int requestCode, int resultCode, Intent intent) {
        ActivityResultHandler handler = activityResultHandlers.remove(requestCode);
        if (handler != null) {
            handler.onActivityResult(resultCode, intent);
            return;
        }

        super.onActivityResult(requestCode, resultCode, intent);
    }

    private int customRequestCode = 37565;

    final HashMap<Integer, ActivityResultHandler> activityResultHandlers =
            new HashMap<>();
}

В продакшене используется немного усложненный вариант, с использованием WeakReferenc-ов. Но сути дела это не меняет.

[archduke]

Александр Юдаков, Спасибо. А как быть если уничтожится Activity(система убьет), а потом восстановится - получается просто результат будет проигнорирован и никак не обработан? Или есть какая то логика, которая при восстановлении заполняет обработчики?

[Александр Юдаков]

archduke, в теории — да, будет как вы написали. А на практике для освобождения оперативки Андроид действует иначе — убивает целиком те приложения, с которыми пользователь долго не работал.

В общем, можно на этот момент забить.

Из опыта есть одно корпоративное приложение, 50 пользователей, половина из них интенсивно работает каждый день. За 2,5 года, судя по логам, проблем подобных нет. Приложение в мобиле работает без перезапуска неделями (кроме UI, в приложении есть фоновый сервис).

Будь на месте разработчика Андроида, в последнюю очередь я бы грохал приложение, вызвавшее startActivityForResult().

Answer 2

[Денис Загаевский]

Где-то должна быть сущность(например, презентер, интерактор), которая хочет обновить список. Она должна иметь возможность ждать события возврата результата с помощью, например, листенера или подписки на стрим таких событий. Активити(или фрагмент) при получении результата оповещает все такие сущности по цепочке. Связывать конкретное событие с конкретным подписчиком можно с помощью requestCode. Как-то так, идея, надеюсь, понятна, остальное - детали реализации. Главное, хорошо абстрагировать.

Comments

[archduke]

Это получается что стейт все равно тогда придется переносить из Activity в определенную сущность, чтоб не потерять номер позиции в списке, который должен принять результат? Все видется так, что глобального стейта не избежать, его можно перенести в другой компонент, но не избежать?

[Денис Загаевский]

archduke, когда ты открываешь другую активити, твоя активити имеет право сдохнуть. Поэтому тебе при любом раскладе придется сохранять состояние. В коде самой активити должен быть вообще минимум логики.