Size: a a a

2019 June 29

NY

Nikita Yatskivskiy in GitFox
Это плохо?
источник

AD

Aleksey D. in GitFox
Не всегда удобно. В конструкторе 10+ зависимостей. Но это уже #offtop.
источник

S

Sergey8827 in GitFox
Aleksey D.
А я сижу с проектом, в котором в одном Presenter есть 5 Interactor.
презентер то привязан к 1 экрану
источник

S

Sergey8827 in GitFox
Sergey8827
презентер то привязан к 1 экрану
если бы у экрана (активити) было например 2 презентенра на 2 фрагмента (табы). то начинается проблема как то синхронизироваться
источник

AD

Aleksey D. in GitFox
Sergey8827
если бы у экрана (активити) было например 2 презентенра на 2 фрагмента (табы). то начинается проблема как то синхронизироваться
Общая модель. База данных, PublishProcessor, что угодно.
источник

NY

Nikita Yatskivskiy in GitFox
Можно интеракторы или репозитории шарить между ними. И использовать механизм подписок.
источник

NY

Nikita Yatskivskiy in GitFox
Или в лоб - использовать parent fragment или activity.
источник

AK

Andrey Khitryy in GitFox
Aleksey D.
А я сижу с проектом, в котором в одном Presenter есть 5 Interactor.
для этого придумали FatModel уже давно
источник

S

Sergey8827 in GitFox
Alexey Pushkarev
Так Сехас потом же признал что пример был ошибочен. Ну, ничего на ошибках учимся
блин люпобытно. Сехас как никак представил клин. Как раз его имел виду, так как изучал. А в чем основная ошибка примера андроид10?))
источник

AP

Alexey Pushkarev in GitFox
Sergey8827
блин люпобытно. Сехас как никак представил клин. Как раз его имел виду, так как изучал. А в чем основная ошибка примера андроид10?))
В разбиении на модули например
источник

S

Sergey8827 in GitFox
окей))
источник
2019 July 01

AA

Alidibir Akhbulatov in GitFox
В приложении сейчас есть экран MyEvents. И возник такой вопрос: а если затем, допустим, решите добавить таб "Activity" на экране юзера, будет создан отдельный экран UserEvents? Если да, то они будут отличаться только тем, что в MyEvents есть тулбар и не нужно передавать user ID, а все остальное продублируется. Если их объединить, можно сделать параметр user ID nullable, т.е. если null, загрузить список для текущего юзера (my events), иначе скрыть тулбар и грузить user events. С одной стороны это позволит избежать дублирования кода, но nullable user ID и скрытие тулбара в зависимости от него тоже не очень. Всё-таки лучше отдельные экраны хоть и с дублированием?
источник

AA

Alidibir Akhbulatov in GitFox
@terrakok хотелось бы узнать мнение по вопросу выше)
источник

KT

Konstantin Tskhovrebov in GitFox
1) дублирование кода - это не плохо
2) главный экран - это из-за гитлаб апи только ивенты юзера. скоро там появится нормальная лента событий
источник

R

Raserad in GitFox
Konstantin Tskhovrebov
1) дублирование кода - это не плохо
2) главный экран - это из-за гитлаб апи только ивенты юзера. скоро там появится нормальная лента событий
То есть иногда для гибкости я могу дублировать некоторые участки кода?
источник

KT

Konstantin Tskhovrebov in GitFox
дублирование кода - это не плохо. дублирование логики - это очень плохо
источник

AA

Alidibir Akhbulatov in GitFox
Konstantin Tskhovrebov
1) дублирование кода - это не плохо
2) главный экран - это из-за гитлаб апи только ивенты юзера. скоро там появится нормальная лента событий
А в случае экрана MyEvents (с тулбаром и без user id) и UserEvents (без тулбара и с user id) есть смысл разделять экраны или лучше объединить? При условии, что на обеих экранах список одинаковый
источник

KT

Konstantin Tskhovrebov in GitFox
разъединение всегда предпочтительнее.
главное помнить: дублирование логики - это очень плохо
источник

AA

Alidibir Akhbulatov in GitFox
Konstantin Tskhovrebov
разъединение всегда предпочтительнее.
главное помнить: дублирование логики - это очень плохо
Бизнес-логика дублироваться не будет, т.к. скрыта в интеракторе, а реализация пагинатора будет одинаковой в презентерах для обеих экранов. Но можно ли отнести реализацию пагинатор к логике?
источник

AA

Alidibir Akhbulatov in GitFox
Думаю, что нет, т.к. ViewController про обновление данных, а данные грузит через интерактор
источник