лиснер видимо не понял суть вопроса.
1. зачем передавать clickListener во вьюмодель через фабрику и конструктор, если он потом сетится в адаптер того же фрагмента и никак не используется во вьюмодели? и вообще так делать - не тру джедайский путь. ничего не поменялось, создавать у вьюмодели метод fun onListItemClick(clickedItem: SomeItem) и лиснер во фрагменте будет примерно такой
val listener: (SomeItem) -> Unit = { viewModel.onListItemClick(it) }
его уже передавать в адаптер, а в адаптере уже при байндинге холдера сетить ему View.OnClickListener { listener(someItem) }. а чтобы не создавать экземпляр этого лиснера при каждом байнде можно этот лиснер передавать в конструктор холдера и сетить его на нужную вьюху в блоке init. Но при этом придется завести переменную, в которой будет храниться элемент списка и сетить его при байндинге.
2) геттеров у вьюмодели не должно быть, как и у вьюхи. гуглить по запросу unidirectional data flow android
Насчёт первого пункта я пересмотрю, но насчёт второго если честно не понял замечания то есть как я понял AboutListItemViewModel быть вообще не должно? я же получаются из xml ссылаюсь так раз таки на эти значения.
Обсуждают сегодня