Android中单个类绑定两个XML布局的列表交互优化方法

你当前使用接口回调的实现思路是正确的，但存在耦合度高、样板代码多、强转有崩溃风险的问题，可以按照以下方式优化：

原有写法的可优化点 #

• 回调中传递Adapter内部的View实例，会导致外部调用方和Adapter的内部控件实现强耦合：一旦后续修改列表项的布局控件类型，外部的(v as TextView).text强转逻辑会直接触发崩溃
• 沿用Java风格的单抽象方法接口，在Kotlin中可以用语言特性大幅简化样板代码
• 未对点击位置的有效性做判断，adapterPosition在视图回收、动画执行等场景下可能返回-1（即RecyclerView.NO_POSITION），存在索引越界风险

优化后实现 #

1. 改造RecyclerAdapter #

用Kotlin高阶函数替代自定义接口，回调直接返回点击项对应的数据实体，移除View参数暴露：

// 高阶函数作为构造参数，直接返回点击位置对应的Country实体，不需要传View
class RecyclerAdapter(
    private val dataSet: List<Country>,
    private val onItemClick: (Country) -> Unit
) : RecyclerView.Adapter<RecyclerAdapter.ViewHolder>() {

    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder {
        val binding = RowBinding.inflate(LayoutInflater.from(parent.context), parent, false)
        return ViewHolder(binding)
    }

    override fun getItemCount(): Int = dataSet.size

    override fun onBindViewHolder(holder: ViewHolder, position: Int) {
        holder.bind(dataSet[position])
    }

    inner class ViewHolder(val binding: RowBinding) : RecyclerView.ViewHolder(binding.root) {
        // 初始化时绑定点击事件，避免onBind中重复创建监听
        init {
            binding.root.setOnClickListener {
                // 先获取有效位置，避免索引越界
                val position = bindingAdapterPosition
                if (position != RecyclerView.NO_POSITION) {
                    onItemClick.invoke(dataSet[position])
                }
            }
        }

        fun bind(country: Country) {
            with(binding) {
                rowImageView.setImageResource(country.imgRes)
                rowTextView.text = country.name
            }
        }
    }
}

2. 简化MainActivity中的调用 #

不需要匿名内部类实现接口，直接传入lambda即可，直接拿到数据实体更新UI，完全不需要操作View、做类型强转：

class MainActivity : AppCompatActivity() {
    private lateinit var binding : ActivityMainBinding
    private val data = arrayListOf<Country>()
    // 注：你当前写的imgRes、data1都是空数组，实际运行需要填充对应资源和名称数据
    private val imgRes = intArrayOf()
    private val data1 = arrayOf<String>()

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        binding = ActivityMainBinding.inflate(layoutInflater)
        setContentView(binding.root)

        for(i in imgRes.indices) {
            data.add(Country(imgRes[i], data1[i]))
        }

        // 直接传lambda处理点击，不需要实现接口
        val adapter = RecyclerAdapter(data) { selectedCountry ->
            binding.textView.text = selectedCountry.name
        }
        binding.recycler1.adapter = adapter
        binding.recycler1.layoutManager = StaggeredGridLayoutManager(2, StaggeredGridLayoutManager.VERTICAL)
    }
}

可选进阶优化 #

如果后续需要做列表数据的增量刷新（比如增删改某一项不需要全局刷新列表），可以用ListAdapter替代基础的RecyclerView.Adapter，结合DiffUtil自动计算数据差异，更新动画更流畅、性能更好。

注意：不要在onBindViewHolder中设置点击事件，这种写法会在每次列表滚动复用时重复创建点击监听对象，性能更差，你当前把点击逻辑写在ViewHolder的init块中的做法是正确的，优化后保留了这个优点。

内容的提问来源于stack exchange，提问作者Glo