使用ViewModel与Room更新RecyclerView时，编辑项总是修改上一项

你用RecyclerView的adapterPosition（列表位置）作为Room实体的主键ID执行更新，而列表位置会随数据变化（删除、重新加载、排序）改变，导致更新了错误的条目。Room的更新操作依赖实体的唯一主键ID，而非列表中的临时位置。

修复步骤 #

1. 调整数据传递逻辑：传递实体主键ID而非列表位置

假设你的MyCartItemsDatabase实体类主键字段为id（若实际字段名不同，替换为对应名称）：

• 修改CommunicatorEdit接口：

interface CommunicatorEdit {
    fun passData(
        itemId: Int, // 替换原position为实体主键ID
        title: String,
        image: Bitmap,
        price: Double,
        priceI: Double,
        num: Int,
        des: String
    )
}

• 修改CartEditAdapter的onClick方法，传递实体ID：

override fun onClick(v: View?) {
    val position = adapterPosition
    if (position != RecyclerView.NO_POSITION) {
        val item = cartItems[position]
        listener.passData(
            item.id, // 传入实体主键ID
            item.cartTitle!!,
            item.cartImage!!,
            item.cartPriceL!!,
            item.cartPriceI!!,
            item.cartNum!!,
            item.cartDes!!
        )
    }
}

• 修改EditCartFragment的passData方法，存储实体ID到Bundle：

override fun passData(itemId: Int, title: String, image: Bitmap, price: Double, priceI: Double, num: Int, des: String) {
    val bundle = Bundle()
    bundle.putInt("edit_item_id", itemId) // 替换原edit_pos为实体ID
    bundle.putString("edit_name", title)
    bundle.putString("edit_image", getImageUri(requireContext(), image).toString())
    bundle.putDouble("edit_price", price)
    bundle.putDouble("edit_priceI", priceI)
    bundle.putInt("edit_num", num)
    bundle.putString("edit_des", des)

    // Fragment跳转代码保持不变
}

2. 修改EditSingleItemFragment，使用实体ID执行更新

• 更新变量定义与参数接收：

private var editItemId: Int?= null // 替换原editPos变量
// ...

override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
    super.onViewCreated(view, savedInstanceState)
    editItemId = arguments?.getInt("edit_item_id") // 获取实体主键ID
    // 其他参数接收逻辑不变
}

• 修改updateDatabase方法，用实体ID创建更新对象：

private suspend fun updateDatabase(){
    val cartedTitle = editSTitle.text.toString()
    val cartedImage = getBitmap(editImage)
    val cartedPrice = editSLPrice.text.toString().toDouble()
    val cartedPriceI = editSIPrice.text.toString().toDouble()
    val cartedItemNum = editSNum.text.toString().toInt()
    val cartedDes = editSDes.text.toString()
    // 用实体主键ID创建对象，而非列表position
    val item = MyCartItemsDatabase(editItemId!!, cartedTitle, cartedImage,
        cartedPrice, cartedPriceI, cartedItemNum, cartedDes)
    cartViewModel.updateCart(item)
}

3. 优化Adapter数据更新（可选但推荐）

当前setData方法用notifyDataSetChanged()会刷新整个列表，易引发position混乱，建议用DiffUtil高效更新：

fun setData(newCartItems: List<CartItemsDatabase>){
    val diffResult = DiffUtil.calculateDiff(CartItemsDiffCallback(this.cartItems, newCartItems))
    this.cartItems = newCartItems
    diffResult.dispatchUpdatesTo(this)
}

// 新增DiffCallback类
class CartItemsDiffCallback(
    private val oldList: List<CartItemsDatabase>,
    private val newList: List<CartItemsDatabase>
) : DiffUtil.Callback() {
    override fun getOldListSize() = oldList.size
    override fun getNewListSize() = newList.size

    override fun areItemsTheSame(oldItemPosition: Int, newItemPosition: Int): Boolean {
        // 用主键ID判断是否为同一条目
        return oldList[oldItemPosition].id == newList[newItemPosition].id
    }

    override fun areContentsTheSame(oldItemPosition: Int, newItemPosition: Int): Boolean {
        // 对比实体内容是否一致，可根据实际字段调整
        val oldItem = oldList[oldItemPosition]
        val newItem = newList[newItemPosition]
        return oldItem.cartTitle == newItem.cartTitle &&
                oldItem.cartImage == newItem.cartImage &&
                oldItem.cartPriceL == newItem.cartPriceL &&
                oldItem.cartPriceI == newItem.cartPriceI &&
                oldItem.cartNum == newItem.cartNum &&
                oldItem.cartDes == newItem.cartDes
    }
}

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