我有一个自定义的 UICollectionViewCell。左右滚动时(水平方向的UICollectionView)每次增加0.2MB内存占用。

我相信我在单元格对象中正确地实现了 prepareForReuse()；在其中我删除了单元格的所有 subview 。

在我的 Collection View 单元格上使用对象的 didSet，我在我的单元格中调用 setupViews()。我添加了一个带有约束的 UIImageView 并将其添加为 subview 。这很好。

但是，当我使用 UILabel() 时，似乎出现了内存泄漏。当我查看 Instruments 时，我可以看到:VM: UILabel (CALayer) 每次在两个单元格之间滚动时都会重复创建! UIImageView 不会发生这种情况。

以防万一它是相关的，这是我的单元格中的 prepareForReuse 方法:

override func prepareForReuse() {
    super.prepareForReuse()

    self.moreButtonDelegate = nil

    for subview in subviews {
        subview.removeConstraints(subview.constraints)
        subview.removeFromSuperview()
    }

    self.removeFromSuperview() // BURN EVERYTHING
}

这是我的代码:

private func setupViews() -> Void {
    let imageView = myImageView // A lazy class property returns this

    innerView.addSubview(imageView) // innerView is just another UIView within this cell

    // Now I add constraints for imageView
}

因此，没有内存泄漏。看起来 ARC 正确地清理了所有内容，因为即使是图像，内存使用也不会呈指数增长。

但是，当我在 imageView 下面添加它时...

let address = UILabel()
address.translatesAutoresizingMaskIntoConstraints = false
address.text = "TEST"
address.font = UIFont.systemFont(ofSize: 22)
address.adjustsFontSizeToFitWidth = true

// Then I add constraints

我得到一个新的 VM: UILabel (CALayer) 行出现在单元格之间的每个滚动中，因此内存使用量跳跃。看一看:

我做错了什么？我正在使用 Xcode 9、iOS 11.2 模拟器。

最佳答案

我不确定这会解决您的特定问题，但我相信您误解了 prepareForReuse 并且我认为这很可能是您的代码有问题的地方。因此，让我们看一下您的实现:

override func prepareForReuse() {
    super.prepareForReuse()

    self.moreButtonDelegate = nil

    for subview in subviews {
        subview.removeConstraints(subview.constraints)
        subview.removeFromSuperview()
    }

    self.removeFromSuperview() // BURN EVERYTHING
}

我相信您对 prepareForReuse 的看法完全错误。重用的要点是减少创建cell的 View (对象实例化、创建 View 层级、布局等)带来的开销。你不想烧掉一切!相反，您希望尽可能多地保留 contentView。理想情况下，您将只更改 View 的内容(即:UILabel 中的 text，UIImageView 中的 image，等)，或者一些属性(backgroundColor 等)。

您可以使用 prepareForReuse 来取消您开始呈现单元格的一些重量级操作，但是当单元格从 View 中移除并且应该在其他地方重用时，这些操作可能还没有结束。例如，当您从 Web 下载内容时，用户可能会快速滚动，并且在下载和显示 Web 图像之前单元格会离开屏幕。现在，如果单元格被重复使用，则很可能会显示旧的下载图像 - 因此在 prepareForReuse 中，您可以取消此操作。

结论 - 我相信你在 prepareForReuse 中所做的操作都没有真正帮助 - 反之亦然，因为 Collection View 将不得不再次重新创建单元格的整个 UI从头开始(这意味着对象实例化等的所有开销)。我给你的第一个建议是放弃整个 prepareForReuse 实现。

其次，在您放弃 prepareForReuse 实现后，重构单元格，以便它只创建一次 UI，最好是在它的 initializer 中:

class UI: UITableViewCell {
    override init(style: UITableViewCellStyle, reuseIdentifier: String?) {
        super.init(style: style, reuseIdentifier: reuseIdentifier)

        setupViews()
    }
}

然后，在 cellForItemAt 中配置它的内容，这意味着为标签设置文本，为 ImageView 设置图像等。

最后，请记住 documentation 是什么说到它(我自己强调):

Performs any clean up necessary to prepare the view for use again.

只做真正需要做的事，一事不做。

在过去的一年里，我实现了许多tableView 和collectionView 数据源，但我真的只需要使用两次prepareForReuse(例如带有我上面提到的图片下载)。

编辑

我的意思的例子:

struct Model {
    var name: String = ""
}

class CustomCell: UITableViewCell {
    // create it once
    private let nameLabel = UILabel()

    override init(style: UITableViewCellStyle, reuseIdentifier: String?) {
        super.init(style: style, reuseIdentifier: reuseIdentifier)

        // setup view once
        setupViews()
    }

    required init?(coder aDecoder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }

    private func setupViews() {
        // add it to view
        self.contentView.addSubview(nameLabel)
        // setup configuration
        nameLabel.textColor = UIColor.red

        // lay it out
        nameLabel.translatesAutoresizingMaskIntoConstraints = false
        NSLayoutConstraint.activate([
            nameLabel.topAnchor.constraint(equalTo: self.contentView.topAnchor, constant: 8),
            nameLabel.bottomAnchor.constraint(equalTo: self.contentView.bottomAnchor, constant: -8),
            nameLabel.leftAnchor.constraint(equalTo: self.contentView.leftAnchor, constant: 8),
            nameLabel.rightAnchor.constraint(equalTo: self.contentView.rightAnchor, constant: -8),
            ])
    }

    // this is what you call in cellForRowAt
    func configure(for model: Model) {
        nameLabel.text = model.name
        // someImageView.image = model.image
        // etc.
    }

    override func prepareForReuse() {
        super.prepareForReuse()
        // if it is super important, reset the content, cancel operations, etc., but there is no reason to recreate the UI

        // so e.g. this might be ok (although in this case completely unnecessary):
        nameLabel.text = nil

        // but you definitely don't want to do this (that's done once at the cell initialization):
        // nameLabel = UILabel()
        // setupViews()
    }
}

class CustomTableViewController: UITableViewController {

    var models: [Model] = [Model(name: "Milan")]

    override func viewDidLoad() {
        super.viewDidLoad()
        tableView.register(CustomCell.self, forCellReuseIdentifier: "customCell")
    }

    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return models.count
    }

    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "customCell", for: indexPath) as! CustomCell
        // you just want to set the contents, not to recreate the UI components
        cell.configure(for: models[indexPath.row])
        return cell
    }
}

Moreover, always work with cell's contentView, not directly with the cell. Notice that I used this:

self.contentView.addSubview(nameLabel)

而不是这个:

self.addSubview(nameLabel)

Official docs :

The content view of a UITableViewCell object is the default superview for content displayed by the cell. If you want to customize cells by simply adding additional views, you should add them to the content view so they will be positioned appropriately as the cell transitions into and out of editing mode.

关于ios - 使用 UILabel 时自定义 UICollectionViewCell 和内存泄漏，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/48330547/