KT
и так раньше и делалось в гитфоксе
Size: a a a
2019 January 10
KT
теперь все гораздо лучше
KT
и нет привязки к мокси, а у вас жесткая связь
Р
да с этим согласен, не очень красиво что открытие/закрытие в разных местах
Р
@terrakok я так и не понял до конца, что с DKA и Moxy, она правильно убивает презентеры в режиме DKA или нет? Или ты по сути продублировал эту же логику для убийства скоупов чтобы сделать открытие/закрытие в одном месте (BaseFragment)?
KT
1) написал свою (использует другие параметры), которая повторяет ЖЦ презентера Мокси, то есть не умирает при DKA, но для этого есть костыли.
2) у Мокси презентер не умирает, когда срабатывает DKA. это может и норм поведение, но надо помнить, что вьюМодел из арх_компонентов умирает в аналогичном случае. То есть есть два поведения, а какое правильнее - вопрос спорный
2) у Мокси презентер не умирает, когда срабатывает DKA. это может и норм поведение, но надо помнить, что вьюМодел из арх_компонентов умирает в аналогичном случае. То есть есть два поведения, а какое правильнее - вопрос спорный
KT
если умирать при DKA, то костыли не понадобятся
Р
я честно говоря с DKA особо не работал, может не до конца понимаю суть. На сколько я понял, при DKA активити убивается но она может восстановить состояние из saveInstanceState? Если так, то она нормально восстановит теги презентеров и они будут переиспользованы, зачем они должны умирать, по идее же всё нормально. View пересоздалась и всё ok. Не?
ES
и нет привязки к мокси, а у вас жесткая связь
тут получается, что дилема:
1) если создатели Moxy изменят ЖЦ Presenter, то нам надо будет все равно менять свое поведение в BaseFragment, чтобы скоупы имели одинаковый ЖЦ с DI.
2) чтобы наше решение заработало и для другой библиотеки, то ее Presenter/ViewModel должен иметь такой ЖЦ, что и наше решение в BaseFragment.
1) если создатели Moxy изменят ЖЦ Presenter, то нам надо будет все равно менять свое поведение в BaseFragment, чтобы скоупы имели одинаковый ЖЦ с DI.
2) чтобы наше решение заработало и для другой библиотеки, то ее Presenter/ViewModel должен иметь такой ЖЦ, что и наше решение в BaseFragment.
Р
тут получается, что дилема:
1) если создатели Moxy изменят ЖЦ Presenter, то нам надо будет все равно менять свое поведение в BaseFragment, чтобы скоупы имели одинаковый ЖЦ с DI.
2) чтобы наше решение заработало и для другой библиотеки, то ее Presenter/ViewModel должен иметь такой ЖЦ, что и наше решение в BaseFragment.
1) если создатели Moxy изменят ЖЦ Presenter, то нам надо будет все равно менять свое поведение в BaseFragment, чтобы скоупы имели одинаковый ЖЦ с DI.
2) чтобы наше решение заработало и для другой библиотеки, то ее Presenter/ViewModel должен иметь такой ЖЦ, что и наше решение в BaseFragment.
+1
ES
и получилось, что мы написали решение именно для Moxy.
KT
я честно говоря с DKA особо не работал, может не до конца понимаю суть. На сколько я понял, при DKA активити убивается но она может восстановить состояние из saveInstanceState? Если так, то она нормально восстановит теги презентеров и они будут переиспользованы, зачем они должны умирать, по идее же всё нормально. View пересоздалась и всё ok. Не?
все дело в том, что активити никогда в реальности не умирает без смерти процесса. а нам приходится костылями поддерживать этот кейс. DKA в природе не существует
Rl
По хорошему presenter должен быть в scope, верно?
Rl
Если говорить вообще
ES
я честно говоря с DKA особо не работал, может не до конца понимаю суть. На сколько я понял, при DKA активити убивается но она может восстановить состояние из saveInstanceState? Если так, то она нормально восстановит теги презентеров и они будут переиспользованы, зачем они должны умирать, по идее же всё нормально. View пересоздалась и всё ok. Не?
GitFox - приложение, которое ориентировано на программистов, и у некоторых пользователей этот режим включен. А крэши ловить и забивать на них не круто!!!
Р
все дело в том, что активити никогда в реальности не умирает без смерти процесса. а нам приходится костылями поддерживать этот кейс. DKA в природе не существует
я не до конца понимаю суть проблемы. Moxy определённым образом работает с DKA (не убивает презентеры), и если с этим всё ok то просто привязываем ЖЦ скоупов к презентерам (закрытие), да не очень красиво. Но в чём реальная проблема, есть какая-то утечка памяти или что, что-то не закрывается? по повтоду того что хочется отвязать скоупы от Moxy (в случае перехода на ViewModel) с этим понятно.
ES
я честно говоря с DKA особо не работал, может не до конца понимаю суть. На сколько я понял, при DKA активити убивается но она может восстановить состояние из saveInstanceState? Если так, то она нормально восстановит теги презентеров и они будут переиспользованы, зачем они должны умирать, по идее же всё нормально. View пересоздалась и всё ok. Не?
ваше решение и Кости решают определенную задачу.
Это самое важное.
Это самое важное.
KT
я не до конца понимаю суть проблемы. Moxy определённым образом работает с DKA (не убивает презентеры), и если с этим всё ok то просто привязываем ЖЦ скоупов к презентерам (закрытие), да не очень красиво. Но в чём реальная проблема, есть какая-то утечка памяти или что, что-то не закрывается? по повтоду того что хочется отвязать скоупы от Moxy (в случае перехода на ViewModel) с этим понятно.
в том, что открываете скоупы в одном месте, а закрываете в другом.
это принципиальная архитектурная проблема
это принципиальная архитектурная проблема
KT
если на архитектуру плевать, то это не важно
KT
и все действительно работает