iOS开发：Table View Cell存储模型对象是否为最佳实践？

这个问题算是iOS开发里关于Cell职责划分的经典场景了，直接给结论：使用Delegate模式（配合唯一标识或IndexPath）的方案更贴近最佳实践，咱们来拆解下两种方案的问题和优势：

方案一：Cell持有Movie对象直接修改的弊端 #

• 职责越界：Table View Cell的核心职责应该只是展示UI，现在让它直接修改数据源里的模型对象，等于把业务逻辑塞进了UI组件里，违反了单一职责原则。后续如果isFavorite的逻辑变复杂（比如需要触发收藏接口、同步到本地数据库、通知其他页面），你就得去改Cell的代码，Cell会变得越来越臃肿，维护成本飙升。
• 内存与耦合风险：Cell强引用Movie对象，万一数据源有其他强引用，或者Cell复用没处理好，可能会导致不必要的内存占用，甚至循环引用。而且Cell和Movie模型绑定死了，以后要换模型结构，Cell也得跟着改，耦合度太高。
• 状态同步隐患：如果其他地方修改了Movie的isFavorite状态（比如用户在另一个页面取消了收藏），Cell因为持有旧的对象引用，可能不会自动刷新，导致显示的状态和实际不一致，得额外做很多同步逻辑。

方案二：Delegate + IndexPath（或唯一标识）的优势 #

• 职责清晰：Cell只负责触发“点击收藏”的事件，具体的业务逻辑（修改模型、更新数据源、调用接口等）全交给ViewController（或专门的数据源管理者）处理，完美契合MVC/MVVM这类架构的职责划分。Cell只做UI展示和事件转发，干净又好维护。
• 降低耦合与内存风险：Cell不需要持有整个Movie对象，只存个indexPath或者更稳妥的Movie唯一ID（比如movieId），既避免了不必要的引用，也让Cell和模型解耦——以后换模型，只要Cell能拿到唯一ID就行，不用大改。
• 状态同步更可靠：当数据源更新后，只需要调用tableView.reloadRows(at: [indexPath], with: .automatic)就能刷新Cell的UI，保证显示和实际状态一致。而且后续要改收藏逻辑，只需要在Delegate的实现里调整，完全不用碰Cell的代码。

额外优化：用唯一ID代替IndexPath #

其实存indexPath有个小隐患：如果用户点击按钮后，数据源在Delegate响应前发生了变化（比如插入/删除了其他行），indexPath就失效了，会导致操作错误的Movie对象。所以更稳妥的方式是给Cell传递Movie的唯一标识符（比如id: String），在Delegate里通过ID找到对应的Movie，示例代码如下：

Cell代码示例 #

protocol MovieCellDelegate: AnyObject {
    func didToggleFavorite(forMovieWithId movieId: String)
}

class MovieCell: UITableViewCell {
    weak var delegate: MovieCellDelegate?
    private var currentMovieId: String?
    
    func setup(with movie: Movie) {
        // 配置UI：比如设置电影名、收藏按钮状态
        currentMovieId = movie.id
        favoriteButton.isSelected = movie.isFavorite
    }
    
    @IBAction func favoriteButtonTapped(_ sender: UIButton) {
        guard let movieId = currentMovieId else { return }
        delegate?.didToggleFavorite(forMovieWithId: movieId)
    }
}

ViewController的Delegate实现 #

extension MovieListViewController: MovieCellDelegate {
    func didToggleFavorite(forMovieWithId movieId: String) {
        guard let movieIndex = movies.firstIndex(where: { $0.id == movieId }) else { return }
        let targetMovie = movies[movieIndex]
        targetMovie.isFavorite = !targetMovie.isFavorite
        
        // 这里可以加网络请求、本地存储等逻辑
        // ...
        
        // 刷新对应Cell
        tableView.reloadRows(at: [IndexPath(row: movieIndex, section: 0)], with: .fade)
    }
}

这种方式既保证了职责分离，又避免了IndexPath失效的问题，是更健壮的实现方式。

内容的提问来源于stack exchange，提问作者Toma Radu-Petrescu