RecyclerView加载SQLite收藏数据仅单个图标状态异常求助

嘿，我来帮你捋捋这个问题！你的代码里藏了个很容易踩的小坑——在onBindViewHolder的forEach循环里，你每次遍历收藏数据时都会直接给toggle_favorite设置选中状态，这就导致后面的循环会完全覆盖前面的结果。举个例子：假设你有3条收藏数据，前两条和当前item的id匹配，第三条不匹配，那最后一次循环会把状态设为false，前面的true就被冲掉了；只有当最后一条收藏数据刚好和当前item匹配时，状态才会是true，这就是为什么只有最后一条对应的图标能正常显示。

另外还有个隐藏的性能问题：每次绑定item都创建DatabaseHelper并查询数据库，这会让RecyclerView滑动时频繁操作数据库，卡顿是迟早的事。

修复步骤 #

1. 提前加载收藏ID集合，避免重复查询

先在Adapter里存一个收藏ID的集合，这样不用每次绑定都查数据库：

class MovieAdapter(...) : RecyclerView.Adapter<MovieAdapter.MyViewHolder>() {
    // 新增变量保存收藏ID集合，类型根据你的实际ID类型调整
    private var favoriteIds = emptySet<String>()

    // 新增方法更新收藏数据
    fun updateFavoriteData(favoriteData: List<YourFavoriteDataType>) {
        favoriteIds = favoriteData.map { it.id }.toSet()
        notifyDataSetChanged() // 通知列表更新状态
    }
    // ... 其他原有代码
}

然后在你的Fragment/Activity里，一次性加载收藏数据并传给Adapter：

// 比如在onViewCreated或者数据初始化时
val favoriteData = DatabaseHelper(requireContext()).loadFavoriteData()
movieAdapter.updateFavoriteData(favoriteData)

2. 修改onBindViewHolder，直接判断状态

把原来的forEach循环去掉，直接用集合的contains方法判断当前item是否在收藏里：

override fun onBindViewHolder(holder: MyViewHolder, position: Int) {
    val view = holder.itemView
    val currentItem = data.results[position]
    // 直接判断当前item的ID是否在收藏集合中
    view.toggle_favorite.isChecked = favoriteIds.contains(currentItem.id.toString())
}

这样一来，每个item的状态只会被设置一次，不会被后续循环覆盖，同时也避免了重复查询数据库的性能问题。

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