UICollectionView滚动时indexPath.item返回错误值的技术问题

看起来你遇到了UICollectionView复用机制和索引处理的坑，这在iOS开发中太常见了，我来帮你一步步梳理解决：

首先分析核心问题 #

你提到滚动时indexPath.item打印值异常，比如从0跳至2、从3跳至0，结合你的代码来看，主要有几个潜在问题：

1. Cell复用导致的按钮事件重复绑定：你每次在cellForItemAt里给按钮添加点击事件，但复用cell时旧的事件没有被移除，导致按钮关联的索引混乱；
2. 错误的时机设置CollectionView属性：你在cellForItemAt里重复设置collectionView.bounces = false，这会干扰CollectionView的正常布局逻辑；
3. 额外需要确认：你的numberOfItemsInSection是否正确返回了10（卡组数量），以及collectionImageArray的元素数量是否和卡组数量匹配。

具体修复步骤 #

1. 修正Cell复用的事件绑定问题

在你的HomeViewCell类中重写prepareForReuse方法，每次复用Cell时清除旧的按钮事件，避免重复绑定：

class HomeViewCell: UICollectionViewCell {
    // 你的其他代码...
    
    override func prepareForReuse() {
        super.prepareForReuse()
        // 清除按钮之前的所有事件绑定
        collectionImage.removeTarget(nil, action: nil, for: .allEvents)
    }
}

2. 正确获取点击Cell的索引

修改你的handleCollectionTap方法，通过点击的按钮反向找到对应的Cell，再获取正确的indexPath，而不是依赖之前绑定的旧索引：

@objc func handleCollectionTap(sender: UIButton) {
    // 逐步向上查找Cell（根据你的Cell布局调整superview的层级）
    guard let cell = sender.superview?.superview as? HomeViewCell else {
        print("无法找到对应的Cell")
        return
    }
    guard let indexPath = collectionView.indexPath(for: cell) else {
        print("无法获取Cell的indexPath")
        return
    }
    let selectedDeckIndex = indexPath.item
    print("选中的卡组索引：\(selectedDeckIndex)")
    
    // 在这里处理跳转到卡片查看ViewController的逻辑，传入selectedDeckIndex
}

3. 调整CollectionView属性的设置时机

把collectionView.bounces = false移到viewDidLoad里，只设置一次即可，不要在cellForItemAt里重复执行：

override func viewDidLoad() {
    super.viewDidLoad()
    // 你的其他初始化代码...
    collectionView.bounces = false
}

4. 确保数据源的正确性

检查numberOfItemsInSection的实现，确保返回的是卡组的实际数量（比如10或者collectionImageArray.count）：

override func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
    return collectionImageArray.count // 或者直接写10，确保和卡组数量一致
}

同时，在cellForItemAt里添加越界保护，避免因索引错误导致崩溃：

override func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell { 
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "cellId", for: indexPath) as! HomeViewCell 
    
    let imageNumber = indexPath.item
    // 添加越界检查
    guard imageNumber < collectionImageArray.count else {
        print("警告：索引\(imageNumber)超出数组范围")
        return cell
    }
    
    getNumber(imageNumber: imageNumber) 
    let image = collectionImageArray[imageNumber].withRenderingMode(.alwaysOriginal) 
    cell.collectionImage.setImage(image, for: .normal) 
    cell.collectionImage.addTarget(self, action: #selector(handleCollectionTap), for: .touchUpInside) 
    print("当前显示的Cell索引：\(imageNumber)") 
    return cell 
}

为什么这样能解决问题？ #

• prepareForReuse是Cell复用前的回调，在这里清除旧事件，能保证每个Cell的按钮只绑定当前的点击事件；
• 通过indexPath(for: cell)获取索引，是从CollectionView的数据源中直接获取当前Cell的位置，完全不受复用影响；
• 把CollectionView的属性设置移到viewDidLoad，避免了重复修改布局属性导致的滚动异常。

按照这些步骤调整后，你应该能正确获取选中的卡组索引，加载对应的卡片内容了～

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