Подскажите, а у fragmentа может быть несколько ViewModel? У меня

Question

Подскажите, а у fragmentа может быть несколько ViewModel? У меня

есть логика, которая должна быть на всех используемых фрагментах, вот думаю, что если я создам абстрактный класс фрагмента, в который буду инжектить commonViewModel? Т.е. в реализации фрагмента своя вьюмодель, а у абстрактного родителя своя.
Не сойдёт ли с ума инжектор зависимостей?

#it #programming #russian #software

0

08.05.2021

23 ответов

23 просмотра

Сергей Автор вопроса

brrr
Мб как вариант использовать вм активити

Увы, нужна именно привязка к конкретным фрагментам чтобы юзать правильный fragmentManager. В общем инжектор действительно сходит с ума, т.к. сразу видит 2 модели которые надо заинжектить, но умеет инжектить только одну. Тут как я понял два варианта: 1. Инжектить обе вьюмодели в реализации фрагмента (Но тогда в каждой реализации надо не забывать об этом) 2. Вместо инжекта тянуть конкретный инстанс, что я и сделал. И вроде бы это то, что надо Реализация спокойно инжектит поля помеченные inject, а абстрактный класс ручками тянет нужный ему инстанс и они не мешают друг другу

0

08.05.2021

Oleg

Можно иметь shared viewmodel, привязанную к активити. Другой вопрос, нужно ли)

0

08.05.2021

Pêcheur

Сергей
Увы, нужна именно привязка к конкретным фрагментам...

у тебя вьюмодель инжектится прямо в поле фрагмента? @Inject val viewModel = .... вот так?

0

08.05.2021

Сергей Автор вопроса

Pêcheur
у тебя вьюмодель инжектится прямо в поле фрагмента...

Да, почти

0

08.05.2021

Сергей Автор вопроса

Oleg
Можно иметь shared viewmodel, привязанную к активи...

Увы, но не подходит, у меня у каждого фрагмента должна быть своё личное состояние, т.е. каждому нужна своя вью модель, но такая же как у всех. Мой вариант в целом рабочий, я только пытаюсь сейчас сделать так, чтобы вьюмодель стопила подписку, когда привязанный к ней фрагмент невидим пользователю, пока сделал это прокидывая во вьюмодель события onDesroyView и onCreateView

0

08.05.2021

Roman Chumachenko

Слушай, а почему бы не сделать подкласс viewmodel с этой логикой? А потом наследуй VM фрагмента от этого класса, когда нужно

0

08.05.2021

Pêcheur

Сергей
Увы, но не подходит, у меня у каждого фрагмента до...

а что там за подписка такая, которую во вьюмоедли нужно стопить? Мб ее не нужно стопить? Мб она должна быть во фрагменте, а не во вьюмодели?

0

08.05.2021

Сергей Автор вопроса

Roman Chumachenko
Слушай, а почему бы не сделать подкласс viewmodel ...

Думал об этом, на что-то странное выходит. Кейс такой: прилетают события - надо показать диалог. Т.е. на базовую VM, должен подписаться базовый фрагмент, а чтобы это сделать, нам в базовом фрагменте уже надо иметь ссылку на вьюмодель, получается надо её объявлять в базовом, а потом кастовать в наследнике к нужной... Попробую сделать так тестовый проект, а то чёт каша.

0

08.05.2021

Сергей Автор вопроса

Pêcheur
а что там за подписка такая, которую во вьюмоедли ...

Во фрагменте сейчас и есть подписка, но нужно сохранять стейт полученный через неё, для этого используется deprecated retainInstance, от которого я пытаюсь избавиться.

0

08.05.2021

Pêcheur

Сергей
Думал об этом, на что-то странное выходит. Кейс та...

а на чем реактивщина у тебя? лайвдата? флоу? Rx?

0

08.05.2021

Roman Chumachenko

Сергей
Думал об этом, на что-то странное выходит. Кейс та...

Не надо кастовать, иметь не надо тоже. Абстрактное поле для получения VM можно сделать в базе фрагмента. Возвращать будет не AbsVewModel, а дженерик

0

08.05.2021

Сергей Автор вопроса

Pêcheur
а на чем реактивщина у тебя? лайвдата? флоу? Rx?

Repo -> VM - RX, VM -> View - LiveData

0

08.05.2021

Сергей Автор вопроса

Roman Chumachenko
Не надо кастовать, иметь не надо тоже. Абстрактное...

Попробую реализовать, спасибо

0

08.05.2021

Pêcheur

Сергей
Repo -> VM - RX, VM -> View - LiveData

для сохранения стэйта в Rx есть BehaviorSubject

0

08.05.2021

Сергей Автор вопроса

Roman Chumachenko
Не надо кастовать, иметь не надо тоже. Абстрактное...

Увы не взлетело, споткнулся на том, что чтобы унаследоваться от VM, мне ей надо передать в конструктор нужные зависимости (репозиторий), а значит придётся его пихать во всех наследников

0

08.05.2021

Roman Chumachenko

Сергей
Увы не взлетело, споткнулся на том, что чтобы унас...

Тут можно схитрить - сделай инжект в поле)

0

08.05.2021

Jorik Fat

Сергей
Увы, нужна именно привязка к конкретным фрагментам...

а зачем Вам именно ViewModel?

0

08.05.2021

Roman Chumachenko

Можешь сделать эту VM - компонентом коина. Грязноватый трюк, но в конструкторы не надо бросать ничего

0

08.05.2021

Сергей Автор вопроса

Roman Chumachenko
Можешь сделать эту VM - компонентом коина. Грязнов...

У меня toothpick вместо коина. Но в итоге я походу сделал, получилась весёлая дичь с Warning'ами "Unchecked cast" и "Leaking 'this' in constructor of non-final class", но работает. Использовать это я конечно же не буду... https://gist.github.com/FirstSpectr/da22944e9300cbd003490818ff838e84

0

08.05.2021

Roman Chumachenko

Сергей
У меня toothpick вместо коина. Но в итоге я походу...

Можно базовый фрагмент сделать универсальнее, чтобы не делать кастов. BaseFragment<VM : BaseViewModelImpl> { abstract val viewModel: VM } Инжектить будешь конкретную реализацию вм в конкретной реализации фрагмента

0

08.05.2021

Сергей Автор вопроса

Roman Chumachenko
Можно базовый фрагмент сделать универсальнее, чтоб...

Да, стало красивее, спасибо, обновил гист

0

08.05.2021

Roman Chumachenko

Сергей
Да, стало красивее, спасибо, обновил гист

👌

0

08.05.2021