AT
отправил картинкой, чтоб читалось
Замени на обычный var category:Category? = null, если надо возвращай дефолтную категорию
Size: a a a
2019 June 20
SV
отправил картинкой, чтоб читалось
nullable данные во вьюхолдере это норм кейс
MS
решение!)
MS
а почему нельзя тогда var category:Category by Delegates.notNull()
MS
избавит от нулабилити
MS
и будет стрелять,е сли кто-то воспользуется холдером без биндинга
KT
lateinit только для инжектов - это правда.но для вьюхолдеров, чтобы не страдать проверками на нулл, там где их не должно быть мы тоже используем. только ради небольшого сокращения.
AP
как отрефакторить в таком случае код, чтобы lateinit исчез? предоставляется ли это вообще возможным?)
inner class ViewHolder(view: View) :
RecyclerView.ViewHolder(view) {
private lateinit var category: Category
init {
view.setOnClickListener {
clickListener(category)
}
}
fun bind(item: Category) {
category = item
itemView.name.text = item.name
}
}зачем вообще хранить ссылку на с
ategory? Обычно у адаптера же есть getItem(position). Позицию внутри вьюхолдера можно получить с помощью getAdapterPosition()AP
ну и собственно т.к. вьюхолдер иннер класс, то у него есть доступ к адаптеру
KT
чтобы не страдать фигней. прокидывать позицию, доставать элементы и прочая странная логика
AP
init {
view.setOnClickListener {
clickListener(getItem(adapterPosition))
}
}вроде норм
MS
init {
view.setOnClickListener {
clickListener(getItem(adapterPosition))
}
}вроде норм
так у меня же это не в классе адаптера, а в классе адаптер делегата
AP
так у меня же это не в классе адаптера, а в классе адаптер делегата
а у делегата нет доступа к адаптеру?
MS
а у делегата нет доступа к адаптеру?
нету
AP
нету
можно передать в конструктор
ES
Можно еще клик сетать в bind
AP
Можно еще клик сетать в bind
тоже вариант, но тогда на каждый бинд будет конечно создаваться новый кликлистенер
ES
тоже вариант, но тогда на каждый бинд будет конечно создаваться новый кликлистенер
а как это будет влиять?
AP
а как это будет влиять?
ну, в смысле новый листенер на каждый бинд, лишний раз память выделять, мелочи конечно...