Android中Firebase ValueEventListener回调致地图页面崩溃问题求助

核心问题拆解 #

你当前的实现确实存在致命隐患：ViewModel中使用addValueEventListener会持续监听Firebase数据变化，每次数据库更新都会触发onDataChange并调用回调。如果Fragment已经销毁（比如用户离开地图页面），回调里操作地图UI组件就会引发崩溃；另外你提供的payjoy_access_service异常是系统层面问题，不会直接导致应用崩溃，真正的崩溃原因应该是回调中操作了已失效的UI组件，只是被系统日志掩盖了。

正确实现方案 #

1. 用LiveData替代回调（推荐） #

ViewModel的核心设计就是通过LiveData管理数据，自动感知Fragment生命周期，从根源避免无效回调和内存泄漏。

ViewModel代码修改：

class MapsViewModel : ViewModel() {
    // 对外暴露不可变的LiveData，内部用MutableLiveData更新
    private val _eventsLiveData = MutableLiveData<MutableList<LocationEvent>>()
    val eventsLiveData: LiveData<MutableList<LocationEvent>> = _eventsLiveData
    
    private var dataListener: ValueEventListener? = null

    fun startListeningEvents() {
        // 避免重复添加监听
        if (dataListener != null) return
        
        dataListener = object : ValueEventListener {
            override fun onDataChange(dataSnapshot: DataSnapshot) {
                val eventList = mutableListOf<LocationEvent>()
                // 这里替换成你的数据解析逻辑
                for (snapshot in dataSnapshot.children) {
                    val event = snapshot.getValue(LocationEvent::class.java)
                    event?.let { eventList.add(it) }
                }
                // 用postValue确保在主线程更新UI
                _eventsLiveData.postValue(eventList)
            }

            override fun onCancelled(databaseError: DatabaseError) {
                Log.w("MapsViewModel", "loadEvents failed", databaseError.toException())
            }
        }
        locationDatabase.addValueEventListener(dataListener!!)
    }

    fun stopListeningEvents() {
        // 页面销毁时移除监听，释放资源
        dataListener?.let {
            locationDatabase.removeEventListener(it)
            dataListener = null
        }
    }
}

MapsFragment代码修改：

class MapsFragment : Fragment() {
    private lateinit var viewModel: MapsViewModel
    private var googleMap: GoogleMap? = null

    override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
        super.onViewCreated(view, savedInstanceState)
        // 初始化GoogleMap的逻辑（略）
        viewModel = ViewModelProvider(this)[MapsViewModel::class.java]
        
        // 观察LiveData，仅在Fragment活跃时接收数据更新
        viewModel.eventsLiveData.observe(viewLifecycleOwner) { eventList ->
            googleMap?.clear() // 先清除旧标记
            // 遍历列表绘制新标记
            eventList.forEach { event ->
                googleMap?.addMarker(
                    MarkerOptions()
                        .position(LatLng(event.latitude, event.longitude))
                        .title(event.title)
                )
            }
        }
    }

    override fun onResume() {
        super.onResume()
        // 进入页面时启动数据监听
        viewModel.startListeningEvents()
    }

    override fun onPause() {
        super.onPause()
        // 离开页面时停止监听，避免无效回调
        viewModel.stopListeningEvents()
    }
}

2. 保留回调但增加生命周期校验（兼容旧代码用） #

如果必须保留回调模式，需要在Fragment中维护回调有效性，并在ViewModel中支持移除监听：

ViewModel代码修改：

fun loadAllEvents(callback: (MutableList<LocationEvent>) -> Unit): ValueEventListener {
    val listener = object : ValueEventListener {
        override fun onDataChange(dataSnapshot: DataSnapshot) {
            val eventList = mutableListOf<LocationEvent>()
            // 数据解析逻辑（略）
            callback(eventList)
        }

        override fun onCancelled(databaseError: DatabaseError) {
            Log.w("MapsViewModel", "loadEvents failed", databaseError.toException())
        }
    }
    locationDatabase.addValueEventListener(listener)
    return listener
}

fun removeEventListener(listener: ValueEventListener) {
    locationDatabase.removeEventListener(listener)
}

MapsFragment代码修改：

private var dataListener: ValueEventListener? = null

override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
    super.onViewCreated(view, savedInstanceState)
    dataListener = viewModel.loadAllEvents { eventList ->
        // 先判断Fragment是否处于活跃状态，避免操作已销毁的UI
        if (!isAdded || viewLifecycleOwner.lifecycle.currentState != Lifecycle.State.RESUMED) {
            return@loadAllEvents
        }
        // 绘制标记逻辑（略）
    }
}

override fun onDestroyView() {
    super.onDestroyView()
    // 页面销毁时移除监听，释放资源
    dataListener?.let {
        viewModel.removeEventListener(it)
        dataListener = null
    }
}

关于错误日志的说明 #

你提供的payjoy_access_service异常是设备厂商定制系统的问题，属于系统层面报错，不会直接导致你的应用崩溃。真正的崩溃原因需要查看AndroidRuntime开头的完整崩溃日志，大概率是回调中操作了已销毁的Map组件。

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