为何在cellForItemAt中设置Cell属性是错误做法？差异解析

嘿，这个问题问到点子上了——很多刚上手UICollectionView/UITableView的开发者都会有这个困惑：明明两种写法运行起来效果一样，为啥前者会被吐槽？其实核心原因不是“代码跑不起来”，而是这种做法违背了iOS开发的最佳实践原则，会给后续维护和扩展埋下不少坑，具体来说有这几点：

1. 违反“单一职责”原则 #

cellForItemAt（或者cellForRowAt）的核心职责应该是数据绑定——把数据源里的模型传递给Cell，仅此而已。而Cell本身的视图样式（背景色、图片、标签文本、布局调整），理应由Cell自己来管理。

打个比方：你去餐馆吃饭，服务员（控制器）的职责是把菜单（数据）递给厨师（Cell），而不是自己上手炒菜（设置Cell样式）。如果啥活都让服务员干，服务员会忙不过来，后期换菜或者调整口味也特别麻烦。

2. 复用Cell时容易出现样式错乱 #

虽然现在你的代码看起来运行正常，但Cell是会被复用的！如果在cellForItemAt里直接设置样式，很容易因为复用没处理好导致“串样式”：
比如你给某一行Cell设置了红色背景，当这个Cell被滑出屏幕再滑回来时，它可能会被复用到另一个不需要红色背景的位置上，如果没手动重置，就会显示错误的样式。

而把样式逻辑放在自定义Cell里，你可以通过prepareForReuse方法统一重置所有视图的初始状态，从根源上避免这种复用bug：

override func prepareForReuse() {
    super.prepareForReuse()
    titleLabel.text = nil
    iconImageView.image = nil
    backgroundColor = .white
}

3. 可维护性极差 #

假设你的项目里有3个不同的控制器都用到了同一个CustomCell，现在产品要求把Cell的背景色从白色改成浅灰色。如果样式都写在cellForItemAt里，你得去3个控制器里找对应的代码修改；但如果样式逻辑封装在Cell内部，你只需要改CustomCell里的一处代码，所有用到这个Cell的地方都会自动生效。

而且随着项目变大，cellForItemAt里的代码会越来越臃肿，后期排查问题或者加新功能时，找代码都会头疼。

4. 扩展性受限 #

以后如果要给Cell加状态切换（比如选中态、加载中态）、动画效果，或者根据不同的模型类型展示不同样式，在Cell内部处理会非常方便。你只需要在Cell里加一个configure(with:)方法，把所有逻辑都封装进去，控制器只需要调用这个方法就行，完全不用关心Cell内部是怎么实现的。

举个对比例子：

不推荐的写法（cellForItemAt里直接设置） #

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath) as! CustomCell
    let model = dataSource[indexPath.item]
    cell.titleLabel.text = model.title
    cell.backgroundColor = model.isHighlighted ? .red : .white
    cell.iconImageView.image = UIImage(named: model.iconName)
    return cell
}

推荐的写法（Cell内部封装配置逻辑） #

// CustomCell.swift
class CustomCell: UICollectionViewCell {
    @IBOutlet weak var titleLabel: UILabel!
    @IBOutlet weak var iconImageView: UIImageView!
    
    override func prepareForReuse() {
        super.prepareForReuse()
        titleLabel.text = nil
        iconImageView.image = nil
        backgroundColor = .white
    }
    
    func configure(with model: CellModel) {
        titleLabel.text = model.title
        backgroundColor = model.isHighlighted ? .red : .white
        iconImageView.image = UIImage(named: model.iconName)
        // 以后加状态切换、动画都可以在这里加
    }
}

// 控制器里的cellForItemAt
func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath) as! CustomCell
    let model = dataSource[indexPath.item]
    cell.configure(with: model)
    return cell
}

总的来说，不是说在cellForItemAt里设置样式就一定会立刻出bug，而是这种做法从架构设计上就不合理，长期来看会让代码变得难以维护和扩展。把Cell的视图逻辑封装在Cell内部，才是更专业、更可持续的写法。

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