在Android Kotlin中，将onclick事件传递给viewholder的正确方法是什么？

Question

这两种方式有什么区别吗？ 我一直在使用第二种方式，到目前为止它有效，但是我在阅读教程文章时发现了第一种方式。

第一个：

class FlowersAdapter(private val onClick: (Flower) -> Unit) :
ListAdapter<Flower, FlowersAdapter.FlowerViewHolder>(FlowerDiffCallback) {

/* ViewHolder for Flower, takes in the inflated view and the onClick behavior. */
class FlowerViewHolder(itemView: View, val onClick: (Flower) -> Unit) :
    RecyclerView.ViewHolder(itemView) {
    private val flowerTextView: TextView = itemView.findViewById(R.id.flower_text)
    private val flowerImageView: ImageView = itemView.findViewById(R.id.flower_image)
    private var currentFlower: Flower? = null

    init {
        itemView.setOnClickListener {
            currentFlower?.let {
                onClick(it)
            }
        }
    }

    /* Bind flower name and image. */
    fun bind(flower: Flower) {
        currentFlower = flower

        flowerTextView.text = flower.name
        if (flower.image != null) {
            flowerImageView.setImageResource(flower.image)
        } else {
            flowerImageView.setImageResource(R.drawable.rose)
        }
    }
}

}

First way of writing

第二个：

class FlowerViewHolder(itemView: View) :
    RecyclerView.ViewHolder(itemView) {
    private val flowerTextView: TextView = itemView.findViewById(R.id.flower_text)
    private val flowerImageView: ImageView = itemView.findViewById(R.id.flower_image)
    private var currentFlower: Flower? = null

    /* Bind flower name and image. */
    fun bind(flower: Flower) {
        currentFlower = flower

        flowerTextView.text = flower.name
        if (flower.image != null) {
            flowerImageView.setImageResource(flower.image)
        } else {
            flowerImageView.setImageResource(R.drawable.rose)
        }

        itemView.setOnClickListener {
                onClick(flower)
        }
    }
}

Second way of writing

感谢您花时间和精力告诉我这些差异。

Answer 1

从关注点分离的角度来看，所有clickListeners 都应该在Activity 或Fragment 和Adapters 中处理，在您的情况下Flower 和以RecyclerView 可以使用的方式将它们显示在屏幕上。

话虽如此，clickListeners 的核心逻辑是从bind 方法移到活动/片段中，而这正是 firstMethod 的全部内容。事实上，我没有注意到使用 FirstMethod 比使用第二种方法有任何性能改进，但我坚持使用 FirstOne，因为它更多的是代码组织。

Answer 2

恕我直言，我觉得适配器应该对点击侦听器或有关ViewHolder 的任何细节一无所知；所以我不会通过适配器传递回调。

我喜欢将回调传递给我的ViewHolder，但不是映射到init 块，而是在适配器的onBind 钩子上执行此操作，我将视图作为参数接收。另外，我将ViewHolders 直接传递或更新到我的Adapters。然后有一些通用函数来计算数据集是否发生了变化。

如果您这样做，您的好处是您可以构建 1 个通用适配器并在其他地方使用它，而无需真正介意稍后可能需要实现多少不同类型的 ViewHolder，因为它完全不可知。

TLDR;

因此，根据您为我们提供的信息，我将使用这两种方法的优点。将回调绑定到bind钩子中，并通过ViewHolder的构造函数传递回调