Kotlin+Firebase聊天应用RecyclerView重复渲染问题求助

嘿，我帮你找到了问题所在！你的RecyclerView重复渲染，核心原因是在循环中多次创建并设置Adapter，同时每次Firebase查询完成后都触发全量刷新，导致列表被重复加载多次。

问题根源 #

1. 重复创建并设置Adapter：在retrieveMessageList的循环里，每遍历一个usersMsgList元素，就会发起一次Firebase查询，每个查询完成后都新建UserMsgAdapter并设置给RecyclerView。假设usersMsgList有N个元素，就会执行N次recycler_view_msgList.adapter = userMsgAdapter，这会导致RecyclerView重复渲染N次。
2. 不必要的全量刷新：每次查询回来都调用notifyDataSetChanged()，而不是针对新增数据做局部刷新，加剧了重复渲染的问题。

修复方案 #

我们需要把Adapter的初始化移到循环外，只做一次；同时优化数据更新的方式，避免重复设置Adapter和无效刷新。

修改后的retrieveMessageList代码：

private fun retrieveMessageList() {
    mUsers = ArrayList()
    // 只初始化一次Adapter并绑定到RecyclerView
    userMsgAdapter = UserMsgAdapter(context!!, mUsers as ArrayList<Users>, true)
    recycler_view_msgList.adapter = userMsgAdapter

    var completedQueries = 0
    val totalQueries = usersMsgList!!.size

    for (eachMessageList in usersMsgList!!) {
        eachMessageList.getId()?.let { receiverId ->
            val ref = FirebaseDatabase.getInstance().reference.child("Users")
                .orderByChild("uid").equalTo(receiverId)
            
            ref.addListenerForSingleValueEvent(object : ValueEventListener {
                override fun onDataChange(p0: DataSnapshot) {
                    for (dataSnapshot in p0.children) {
                        val user = dataSnapshot.getValue(Users::class.java)
                        user?.let {
                            val insertPosition = mUsers.size
                            (mUsers as ArrayList).add(it)
                            // 局部刷新：通知Adapter插入了单个元素，比全量刷新更高效
                            userMsgAdapter?.notifyItemInserted(insertPosition)
                        }
                    }
                    completedQueries++
                    // 可选：所有查询完成后做收尾操作（比如排序）
                    if (completedQueries == totalQueries) {
                        // 例如：mUsers.sortBy { ... }
                        // userMsgAdapter?.notifyDataSetChanged()
                    }
                }

                override fun onCancelled(p0: DatabaseError) {
                    completedQueries++
                    // 处理查询失败的情况
                }
            })
        }
    }
}

同时，在onDataChange中处理ChatList更新时，要先清空旧数据，避免残留：

override fun onDataChange(p0: DataSnapshot) {
    (usersMsgList as ArrayList).clear()
    for (dataSnapshot in p0.children) {
        val messageList = dataSnapshot.getValue(MessageList::class.java)
        messageList?.let { (usersMsgList as ArrayList).add(it) }
    }
    (usersMsgList as ArrayList<MessageList>).reverse()
    
    // 清空旧的用户列表并通知Adapter
    mUsers?.clear()
    userMsgAdapter?.notifyDataSetChanged()
    
    retrieveMessageList()
}

额外优化建议 #

• 如果Users节点的uid是主键，直接用child(receiverId)查询，比orderByChild("uid").equalTo(receiverId)更高效，因为后者是遍历整个节点，而前者直接定位到具体用户。
• 可以尝试用Kotlin协程结合Firebase的get()方法替代addListenerForSingleValueEvent，代码会更简洁易维护。

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