Swift下拉刷新时Venue UITableViewCell加载错误UICollectionView的解决方法

这个问题我之前也踩过坑！核心原因就是你依赖collectionView.tag来关联TableView行索引的方式，在cell复用+数据刷新的场景下完全不可靠：当TableView复用旧的VenueCell时，嵌套的CollectionView的tag还保留着之前的行号，而刷新后filteredVenueArray的长度或内容已经变化，用旧tag去访问数组就很容易触发「Index out of range」错误。

下面是两种彻底解决的方案，优先推荐第一种：

方案1：让VenueListingCell自己管理CollectionView的数据源/代理 #

这种方式能让每个TableViewCell的CollectionView只负责自己对应的数据，彻底避免外部数据源的混乱。

步骤1：修改VenueListingCell类 #

给它添加存储当前行对应图片数组的属性，并且让它自己实现UICollectionView的数据源和代理：

class VenueListingCell: UITableViewCell {
    @IBOutlet weak var collectionView: UICollectionView!
    
    // 存储当前cell对应的场地图片数组
    var venueImageURLs: [String] = [] {
        // 数组更新时自动刷新CollectionView
        didSet {
            collectionView.reloadData()
        }
    }
    
    override func awakeFromNib() {
        super.awakeFromNib()
        // 让cell自己成为CollectionView的数据源和代理
        collectionView.dataSource = self
        collectionView.delegate = self
        // 提前注册CollectionViewCell（如果还没在xib里注册的话）
        collectionView.register(UINib(nibName: "VenueListingCellImage", bundle: nil),
                               forCellWithReuseIdentifier: "VenueListingCellImage")
    }
}

// 扩展实现UICollectionView相关方法
extension VenueListingCell: UICollectionViewDataSource, UICollectionViewDelegate {
    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        // 直接用当前cell的图片数组长度，不需要依赖外部数组和tag
        return venueImageURLs.count
    }
    
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "VenueListingCellImage",
                                                     for: indexPath) as! VenueListingCellImage
        let urlString = venueImageURLs[indexPath.item]
        // 注意这里要加URL的非空判断，避免强制解包崩溃
        if let imgURL = URL(string: urlString) {
            cell.configureCell(url: imgURL)
        }
        return cell
    }
    
    // 如果需要CollectionView的布局、点击事件等代理方法，直接在这里实现即可
}

步骤2：修改TableView的cellForRowAt方法 #

在给TableViewCell赋值时，直接把对应场地的图片数组传给cell：

func tableView(_ tableView: UITableView, cellForItemAt indexPath: IndexPath) -> UITableViewCell {
    let cell = tableView.dequeueReusableCell(withIdentifier: "VenueListingCell",
                                           for: indexPath) as! VenueListingCell
    // 获取当前行对应的场地数据
    let currentVenue = filteredVenueArray[indexPath.row]
    // 把图片数组传给cell
    cell.venueImageURLs = currentVenue.imgURLs
    // 其他cell的UI配置（比如场地名称等）
    return cell
}

步骤3：简化willDisplay方法 #

现在不需要再给CollectionView设置外部数据源代理了，去掉相关代码：

func tableView(_ tableView: UITableView, willDisplay cell: UITableViewCell, forRowAt indexPath: IndexPath) {
    if maxCellIndexAppeared > indexPath.row {
        return
    }
    maxCellIndexAppeared = indexPath.row
    
    // 保持原来的动画逻辑
    cell.alpha = 1
    let transform = CATransform3DTranslate(CATransform3DIdentity, -10, 0, 0)
    cell.layer.transform = transform
    UIView.animate(withDuration: 0.5) {
        cell.alpha = 1
        cell.layer.transform = CATransform3DIdentity
    }
    // 移除原来的guard和setCollectionViewDataSourceDelegate代码
}

方案2：修复tag的赋值逻辑（不推荐，但适合快速临时修复） #

如果你暂时不想重构代码，可以确保每次复用cell时，都正确更新CollectionView的tag：

修改TableView的cellForRowAt方法 #

在复用cell时，强制更新CollectionView的tag为当前行索引：

func tableView(_ tableView: UITableView, cellForItemAt indexPath: IndexPath) -> UITableViewCell {
    let cell = tableView.dequeueReusableCell(withIdentifier: "VenueListingCell",
                                           for: indexPath) as! VenueListingCell
    // 关键：每次复用cell时，更新CollectionView的tag为当前行索引
    cell.collectionView.tag = indexPath.row
    // 其他配置...
    return cell
}

同时在刷新数据时重置maxCellIndexAppeared #

下拉刷新时，记得把maxCellIndexAppeared重置为-1，确保willDisplay方法能正确处理所有行：

// 假设你的下拉刷新触发方法是这个
@objc func handleRefresh() {
    // 重新获取Firebase数据
    fetchVenueDataFromFirebase { [weak self] in
        guard let self = self else { return }
        // 重置maxCellIndexAppeared
        self.maxCellIndexAppeared = -1
        // 刷新TableView
        self.tableView.reloadData()
        // 结束刷新动画
        self.tableView.refreshControl?.endRefreshing()
    }
}

为什么方案1更好？ #

方案1把每个CollectionView的数据和逻辑封装到对应的TableViewCell里，符合单一职责原则，不仅能彻底解决复用和刷新的问题，以后维护起来也更方便——比如要给CollectionView加点击事件，直接在VenueListingCell里实现就行，不需要在ViewController里写一堆判断逻辑。

最后提醒：以后尽量避免用tag来传递业务数据，tag更适合用来做简单的视图标识，比如区分同一个页面里的多个相同类型视图，而不是用来关联数据源索引。

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