Типо на первый взгляд это решает много проблем, но как мне показалось это немного противоречит архитектуре mvvm
Вообще жесть , она противоречит принципам Solid , вообще если вы хотите в дальнейшем что-то добавить или изменить во viewModel то это будет касаться и к другим фрагментам остальные кейсы подумаете сами
Если всё пихать в одну и ту же view model, то да, это жесть 😊
Да вообще чужому программисту будет геморроем для чтения , что где когда , всю эту билиберду надо будет запоминать , если логика там сложная то эти 2 фрагмента будет тупым билибердой
а если он прям очень маленький
Норм, я считаю. Кстати, не нужно в Котлин делать такой null check. Лучше user?.let { ... }
Я хз почему Гугл пушит SharedViewModel, имхо срань. Передавать данные лучше по классике через аргументы
Чем лучше такая запись?
Короче, например. Кроме того, при null check в стиле Java Котлин может выдавать предупреждение, что нет гарантии того, что переменная не окажется равна null в момент использования.
Обсуждают сегодня