为何for循环创建的数组无法被UICollectionView访问？报数组越界错误

嘿，我来帮你梳理下这个问题的核心原因，以及对应的解决办法～

你遇到的数组越界错误，本质是Firebase数据加载的异步特性导致的：

你在viewDidLoad里的for循环中发起了多个Firebase的observe(.value)请求，但这些请求是异步执行的——也就是说，viewDidLoad代码执行完的时候，这些请求可能还没从服务器拿到数据，redFightersArray和blueFightersArray还是空的。但你的numberOfItemsInSection返回的是boutsIDArray.count，这时候CollectionView会尝试去cellForItemAt里取数组元素，空数组自然就会触发越界错误了。

下面是具体的解决方案，分步骤来：

1. 改用一次性数据请求（而非实时监听） #

从你的需求来看，你应该是一次性加载选手数据，不需要实时监听数据变化，所以用observeSingleEvent(of: .value)代替observe(.value)，避免不必要的实时回调。

2. 用DispatchGroup等待所有请求完成 #

因为你要遍历boutsIDArray发起多个请求，需要等所有请求都完成、数组填充好之后，再刷新CollectionView。这时候DispatchGroup就派上用场了。

3. 安全处理数据解析（避免强制解包） #

你代码里的as!强制解包很容易导致崩溃，改用可选值绑定来安全解析数据。

4. 确保在主线程刷新UI #

UI操作必须在主线程执行，所以刷新CollectionView的代码要放在主线程队列里。

修改后的完整代码示例 #

// viewDidLoad里的数据加载部分
override func viewDidLoad() {
    super.viewDidLoad()
    
    // 先清空数组，防止重复添加数据
    redFightersArray.removeAll()
    blueFightersArray.removeAll()
    
    let dispatchGroup = DispatchGroup()
    
    for bid in boutsIDArray {
        dispatchGroup.enter() // 进入分组
        
        ref.child("Bouts").child(eventid).child(bid).observeSingleEvent(of: .value, with: { snapshot in
            defer {
                dispatchGroup.leave() // 不管解析成功失败，都要离开分组
            }
            
            // 安全解析数据
            guard let dictionary = snapshot.value as? [String: Any],
                  let redName = dictionary["red fighter"] as? String,
                  let blueName = dictionary["blue fighter"] as? String else {
                print("解析选手数据失败，Bout ID: \(bid)")
                return
            }
            
            // 添加到数组
            self.redFightersArray.append(redName)
            self.blueFightersArray.append(blueName)
        })
    }
    
    // 所有请求完成后，在主线程刷新CollectionView
    dispatchGroup.notify(queue: .main) {
        self.collectionView.reloadData()
    }
}

// 修改CollectionView的数据源方法，避免越界
public func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
    // 用填充后的数组长度作为返回值，空数组时返回0，不会触发cellForItemAt
    return redFightersArray.count
}

public func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "cell", for: indexPath) as! ResultsCollectionViewCell
    
    // 这里因为numberOfItemsInSection返回的是数组长度，所以indexPath.item不会越界
    cell.redFighterLabel.text = redFightersArray[indexPath.item]
    cell.blueFighterLabel.text = blueFightersArray[indexPath.item]
    
    return cell
}

额外注意事项 #

• 如果之后需要实时监听Firebase数据变化（比如选手信息更新），可以换回observe(.value)，但要注意每次回调时重新清空并填充数组，然后刷新CollectionView。
• 尽量避免使用!强制解包，遇到可选值时用guard let或者if let处理，能大幅降低崩溃概率。

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