Реальная производительность UICollectionView действительно плохая - PullRequest
Новая
       9

Реальная производительность UICollectionView действительно плохая

0 голосов
/ 09 января 2019

Я сейчас работаю над проектом и, в основном, все еще учусь.

Вначале я использовал UITableView, но мне требовалась автоматическая высота и 2 или 3 столбца (в зависимости от устройства). Для столбцов я использую UICollectionViewFlowLayout. В общем, я использую только Labels и StackViews; несколько предупреждений, таких как Unable to simultaneously satisfy constraints (но я изменил несколько приоритетов и работает, но производительность все еще очень плохая.

Я не получаю информацию из бэкэнда, ничего особенного.

CustomLayout выглядит следующим образом: 

public protocol CollectionViewFlowLayoutDelegate: class {
    func numberOfColumns() -> Int
    func height(at indexPath: IndexPath) -> CGFloat
}

public class CustomLayout: UICollectionViewFlowLayout {
    private let cellPadding: CGFloat = 4
    public var cache: [IndexPath : UICollectionViewLayoutAttributes] = [:]
    private var contentHeight: CGFloat = 0
    private var contentWidth: CGFloat {
        guard let collectionView = collectionView else {
            return 0
        }
        let insets = collectionView.contentInset
        return collectionView.bounds.width - (insets.left + insets.right)
    }

    public weak var flowDelegate: CollectionViewFlowLayoutDelegate?

    public override var collectionViewContentSize: CGSize {
        return CGSize(width: self.contentWidth, height: self.contentHeight)
    }

    public override func layoutAttributesForElements(in rect: CGRect) -> [UICollectionViewLayoutAttributes]? {
        var layoutAttributesArray = [UICollectionViewLayoutAttributes]()
        if cache.isEmpty {
            self.prepare()
        }
        for (_, layoutAttributes) in self.cache {
            if rect.intersects(layoutAttributes.frame) {
                layoutAttributesArray.append(layoutAttributes)
            }
        }
        return layoutAttributesArray
    }

    public override func layoutAttributesForItem(at indexPath: IndexPath) -> UICollectionViewLayoutAttributes? {
        return self.cache[indexPath]
    }

    public override func prepare() {
        super.prepare()
        cache.removeAll()
        guard let collectionView = self.collectionView else {
            return
        }
        let numberOfColumns = self.flowDelegate?.numberOfColumns() ?? 1
        self.contentHeight = 0
        let columnWidth = UIScreen.main.bounds.width / CGFloat(numberOfColumns) - CGFloat(numberOfColumns-1) * cellPadding
        var xOffset = [CGFloat]()
        for column in 0 ..< numberOfColumns {
            xOffset.append(CGFloat(column) * columnWidth)
        }
        var column = 0
        var yOffset = [CGFloat](repeating: 0, count: numberOfColumns)

        for item in 0 ..< collectionView.numberOfItems(inSection: 0) {
            let indexPath = IndexPath(item: item, section: 0)
            let photoHeight = self.flowDelegate?.height(at: indexPath) ?? 1
            let height = cellPadding * 2 + photoHeight
            let frame = CGRect(x: xOffset[column], y: yOffset[column], width: columnWidth, height: height)
            let insetFrame = frame.insetBy(dx: cellPadding, dy: cellPadding)
            let attributes = UICollectionViewLayoutAttributes(forCellWith: indexPath)
            attributes.frame = insetFrame
            self.cache[indexPath] = attributes
            self.contentHeight = max(self.contentHeight, frame.maxY)
            yOffset[column] = yOffset[column] + height
            column = column < (numberOfColumns - 1) ? (column + 1) : 0
        }
    }

    public override func shouldInvalidateLayout(forBoundsChange newBounds: CGRect) -> Bool {
        return true
    }
}

Мне нужны 2 или 3 столбца с автоматической высотой. Но что действительно важно, как и в любом другом проекте: Performance.

Заранее большое спасибо.

1 Ответ

0 голосов
/ 30 января 2019

Оказывается, что на моем ViewControllers на func height(at indexPath: IndexPath) -> CGFloat protocol я каждый раз вычислял высоту, а не только извлекал ее из массива.

...