Swift中从Firebase实时数据库读取音乐数据时加载器状态控制及网络异常处理问题咨询

你的核心问题在于只处理了数据获取成功的场景，完全没覆盖失败分支——当网络断开时，Firebase的observeSingleEvent成功回调根本不会触发，导致completion永远无法执行，加载器就一直停在界面上，也没有任何错误反馈。下面是具体的修复方案：

步骤1：重构回调为Result类型 #

首先把getAllMusics的completion改为Result类型，这样能同时传递成功的数据和错误信息，让调用方可以处理两种情况：

func getAllMusics(completion: @escaping (Result<[ItemModal], Error>) -> Void) {
    var musicArray = [ItemModal]()
    
    // 使用带error参数的completion block，捕获网络/数据库错误
    ref.child("music").observeSingleEvent(of: .value, withCompletionBlock: { snapshot, error in
        // 先判断是否有错误（比如断网、权限不足等）
        if let error = error {
            completion(.failure(error))
            return
        }
        
        // 解析数据
        let enumerator = snapshot.children
        while let rest = enumerator.nextObject() as? DataSnapshot {
            guard let data = try? JSONSerialization.data(withJSONObject: rest.value as Any, options: []) else {
                // 解析JSON数据失败，自定义错误返回
                let parseError = NSError(domain: "MusicParser", code: 0, userInfo: [NSLocalizedDescriptionKey: "数据解析失败"])
                completion(.failure(parseError))
                return
            }
            
            do {
                let itemModal = try JSONDecoder().decode(ItemModal.self, from: data)
                musicArray.append(itemModal)
            } catch {
                // 模型解码失败，返回解码错误
                completion(.failure(error))
                return
            }
        }
        
        // 所有数据解析完成，返回成功结果
        completion(.success(musicArray))
    })
}

步骤2：更新didLoad中的逻辑 #

现在在didLoad里，我们可以分别处理成功和失败的情况，确保无论哪种结果都能关闭加载器，并给用户相应提示：

func didLoad() {
    view?.showLoader()
    getAllMusics { [weak self] result in
        guard let self = self else { return }
        
        // 无论成功失败，先关闭加载器
        self.view?.hideLoader()
        
        switch result {
        case .success(let musicList):
            self.musicArray = musicList
        case .failure(let error):
            // 根据错误类型给出不同提示，这里可以优化判断逻辑
            let errorMessage: String
            if (error as NSError).code == -1009 { // 断网错误码
                errorMessage = "无网络连接，请检查网络设置"
            } else {
                errorMessage = "加载音乐失败：\(error.localizedDescription)"
            }
            // 弹出错误提示（这里假设你的view有showToast或alert方法）
            self.view?.showErrorAlert(message: errorMessage)
        }
    }
}

额外优化建议 #

• 使用[weak self]避免循环引用：在闭包中捕获self时加上[weak self]，防止内存泄漏。
• 自定义错误枚举：可以定义一个MusicLoadingError枚举，把解析失败、网络错误等情况统一管理，让错误处理更清晰。
• 超时处理：Firebase默认没有超时设置，如果需要，可以自己添加一个定时器，超过指定时间（比如10秒）就触发错误提示，避免用户无限等待。

内容的提问来源于stack exchange，提问作者berkehan Öztürk