Swift中Switch状态绑定对象及CollectionView行访问问题

你当前的问题核心在于直接从Cell中获取状态的方式不可靠，而且因为indexPath无法直接获取，导致没法正确对应到每个图片的选择状态。更合理的做法是把Switch的状态和你的图片数据绑定在一起，而不是依赖Cell的状态（毕竟CollectionView的Cell是复用的，直接访问Cell很容易出错）。下面给你一套完整的解决方案：

第一步：创建绑定状态的数据模型 #

首先，我们需要一个数据模型来保存图片和对应的选择状态，代替原来单纯的UIImage数组：

struct ImageItem {
    let image: UIImage
    var isSelected: Bool = false // 这个属性用来绑定Switch的开关状态
}

把你原来的images数组类型改成[ImageItem]，这样每个图片都有自己独立的选择状态。

第二步：改造CustomCell，让它能响应Switch变化并更新数据 #

在你的自定义Cell中，添加一个闭包来传递Switch的状态变化，同时提供配置方法来根据数据模型设置Switch的初始状态：

class CustomCell: UICollectionViewCell {
    @IBOutlet weak var savingSwitch: UISwitch!
    // 用闭包来传递Switch的状态变化
    var onSwitchStateChanged: ((Bool) -> Void)?
    
    override func awakeFromNib() {
        super.awakeFromNib()
        // 给Switch添加状态变化的监听
        savingSwitch.addTarget(self, action: #selector(switchValueChanged(_:)), for: .valueChanged)
    }
    
    @objc private func switchValueChanged(_ sender: UISwitch) {
        // 把状态变化通过闭包传递出去
        onSwitchStateChanged?(sender.isOn)
    }
    
    // 根据数据模型配置Cell
    func configure(with item: ImageItem) {
        savingSwitch.isOn = item.isSelected
        // 这里别忘了设置你的图片视图
        // yourImageView.image = item.image
    }
}

第三步：在CollectionView的数据源方法中绑定状态 #

在cellForItemAt方法里，我们把Cell的Switch状态和对应位置的数据模型绑定，当Switch状态变化时，直接更新数据模型：

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "CustomCell", for: indexPath) as! CustomCell
    let currentItem = images[indexPath.item]
    cell.configure(with: currentItem)
    
    // 绑定闭包，更新对应位置的数据模型
    cell.onSwitchStateChanged = { [weak self] isSelected in
        guard let self = self else { return }
        self.images[indexPath.item].isSelected = isSelected
    }
    
    return cell
}

第四步：修改Save方法，直接从数据模型获取选中的图片 #

现在你不需要再访问Cell了，直接从数据模型里筛选出被选中的图片即可，这样既可靠又简洁：

@IBAction func saveTapped() {
    // 筛选出所有isSelected为true的图片
    let selectedImages = images.filter { $0.isSelected }.map { $0.image }
    
    // 执行你的保存逻辑
    for image in selectedImages {
        // 这里写保存图片的代码
        // saveImage(image)
    }
}

为什么原来的方法不可行？ #

1. Cell复用问题：CollectionView的Cell是复用的，当Cell滚出屏幕后，你通过cellForItem(at:)拿到的可能是其他位置的Cell，状态完全不对。
2. 无法获取所有Cell：cellForItem(at:)只能获取当前可见的Cell，不可见的Cell你拿不到，也就没法获取它们的Switch状态。
3. 状态和数据分离：把状态存在Cell里，数据和展示分离，不符合MVC的设计思路，也容易导致状态丢失或错误。

如果你非要尝试直接访问Cell（非常不推荐），可以遍历所有可见的IndexPath，但这种方式只能拿到当前屏幕上的Cell，不可见的Cell状态无法获取：

@IBAction func saveTapped() {
    var selectedImages = [UIImage]()
    guard let visibleIndexPaths = collectionView?.indexPathsForVisibleItems else { return }
    
    for indexPath in visibleIndexPaths {
        if let cell = collectionView?.cellForItem(at: indexPath) as? CustomCell, cell.savingSwitch.isOn {
            selectedImages.append(images[indexPath.item])
        }
    }
}

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