Практично ли передавать объекты в качестве параметра другому контроллеру в Conductor?

Question

[terminator-light]

Как понял, передавать можно, но могут ли случиться непредвиденные ситуации?

CheckoutController:

public void init(){
     btn.setOnClickListener(v->getRouter().pushController(RouterTransaction.with(address == null ?  new AddressController(this) : new AddressController(address, this)));
}
@Override
public void setAddress(Address address) {
     this.address = address;
     tvAddress.setText(address.street+" "+address.house);
}

AddressController:

public <T extends Controller & CheckoutListener> AddressController(Address address, T listener) {
        this(new BundleBuilder()
        .putString("street", address.street)
        .putString("house", address.house)
        .build());
        setTargetController(listener);
}

public AddressController(Bundle bundle){
        address.street = bundle.getString("street");
        address.house = bundle.getString("house");
}

public void init(){
        btnSave.setOnClickListener(v->{
            getRouter().popCurrentController();
            Address address = new Address();
            address.street =  etStreet.getText().toString();
            address.house =  etHouse.getText().toString();
            CheckoutListener listener = (CheckoutListener) getTargetController();

            if (listener != null) {
                listener.setAddress(address);
            }
        });
}

Или в SharedPreference лучше сохранять?

Answer 1

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

Никакого смысла в том, что ты сделал, нет. Ничего не сохранится при пересоздании. Более того, оно упадёт, тк ты не сделал пустой конструктор.
Аргументы нужно передавать через бандл getArgs().
Сохранять в shared preferences тупо, тк это персистентное хранилище.

Comments

[terminator-light]

Есть 2 контроллера: CheckoutController и AddressController.
В первом есть форма, в которую должен вставиться адрес из AddressController. Во втором тоже форма, после заполнения которой адрес передается в CheckoutController.
Если в CheckoutController адрес уже заполнен, при открытии AddressController адрес из CheckoutController передается в AddressController.
Т.е. у меня двухсторонняя передача данных.

Оно не падает, т.к. достаточно одного конструктора либо пустого, либо с Bundle (" does not have a constructor that takes a Bundle argument or a default constructor. Controllers must have one of these in order to restore their states.")

Изменил код, так ты имел в виду?

public AddressController(Bundle bundle){
        super(bundle);
        Timber.d("Bundle");
        address.street = bundle.getString(ADDRESS_STREET);
        address.house = bundle.getString(ADDRESS_HOUSE);
        address.building = bundle.getString(ADDRESS_BUILDING);
        address.apartment = bundle.getString(ADDRESS_APARTMENT);

    }

    public <T extends Controller & CheckoutListener> 
    AddressController(Address address, T listener) {
        this.address = address;
        getArgs().putString(ADDRESS_STREET, address.street);
        getArgs().putString(ADDRESS_HOUSE, address.house);
        getArgs().putString(ADDRESS_BUILDING, address.building);
        getArgs().putString(ADDRESS_APARTMENT, address.apartment);

        setTargetController(listener);
    }

У меня вопрос был: можно ли передавать целый объект в контроллер или лучше расщепив на примитивные типы?

[terminator-light]

столкнулся с тем, что презентер уничтожается при уничтожении контроллера. А мне нужно восстановить состояние view, беря из презентера данные. Код сохранения получается довольно большим, особенно если полей у объекта много. Что будет, если я сохраню его в репозитории, и оттуда достану и восстановлю?

[tiroman]

terminator-light, Не пробовали смотреть в сторону Moxy и MVP или в сторону Jetpack и MVVM?

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

terminator-light,
Да, это уже более осмыслено, хоть и много кода.
В бандл можно запихать Parcelable, который можно реализовывать не руками, а AutoParcelable, например. У контроллеров есть onSaveInstanceState, там можно сохранять состояние, которое не-аргументы, убрать бойлерплейт можно IcePick'ом, например.
Пихнешь ты это в персистентный репозиторий или нет, решать тебе. Отталкиваться надо от смысла данных. Если это какие-то данные, которые юзер вбивает несколько раз, как адрес доставки, то есть смысл запоминать последние N адресов. Если это, например, текст поиска, ему вряд ли место в репозитории.

[terminator-light]

Денис Загаевский, как раз адрес доставки, только один раз вбивает, но может вернуться редактировать. Спасибо большое за совет!

[terminator-light]

tiroman, чистый MVP использую. MVVM, не знаю, можно ли использовать с Conductor

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

terminator-light, MVP, MVVM, MVC это всего лишь паттерны. Они перпендикулярны кондуктору или там фрагментам. И в этих паттернах, в отличие от того же VIPER'a, не описано, как общаются между собой части приложения(например презентеры).

[terminator-light]

Денис Загаевский, в android studio есть встроенный databinding, как с котроллерами Conductor будет работать? или не важно, фрагмент это или контроллер?

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

terminator-light, мне кажется, не важно, если я правильно помню, ему ты всё равно рутовую вьюху должен дать.

[terminator-light]

Денис Загаевский, может, будет ссылка на пример, как это реализовать вместе с MVP с изменением данных? Где-то вьюмодел наследуется от ViewModel, где-то от BaseObservable

[terminator-light]

Денис Загаевский, пока не очень понятно

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

terminator-light, нет, не будет