TableView запаздывает с данными - PullRequest
0 голосов
/ 19 мая 2018

Итак, я столкнулся с проблемой

Представление таблицы становится медленнее (отклики медленнее при прокрутке, нажатии) после перезагрузки / добавления строк пару раз


Так что послепользователь входит в систему, приложение загружает 10 "WorldMessages" .Он загружается в это табличное представление.

Если пользователь прокручивает страницу вниз, он вызывает функцию, которая загружает больше 10: loadOlderOwnWorldMessages()

Каждая ячейка имеет tapGestureRecognizer + longPressGestureRecognizer

И я должен упомянуть, что если пользователь перезагружает tableView, он очищает данные и загружает только первые 10 сообщений WorldMessages


Проблема

Я не знаю почему, но, например, если я перезагружаю tableView 50 раз и каждый раз, когда делаю прокрутку вниз или больше, то tableView замедляется.

Может быть из-зараспознавание жестов касанием / долгим нажатием или ограничения?

Приложение выглядит так:

enter image description here

Видео после задержки:

https://www.youtube.com/watch?v=65NkjS-Kz3M

(Если я закрою приложение и снова открою его, оно снова будет работать плавно, пока я не перезагрузлю егопару раз)

Код:

// Это не весь код, я удалил множество его строк, которые не были важныt (например, функции перезагрузки и т. д.

class ProfileViewController: UIViewController, UITableViewDelegate, UITableViewDataSource, UITabBarDelegate {

    // Classes
    let handleData = HandleData()
    let handleResponses = HandleResponses()
    let worldMessagesFunctions = WorldMessagesFunctions()
    let profileFunctions = ProfileFunctions()

    // View Objects
    @IBOutlet var tableView : UITableView!

    // Variables
    var userWorldMessages = [WorldMessage]()
    var lastContentOffsetY : CGFloat?

    var currentSelectedWorldMessageIndexPath : IndexPath?

    // Main Code
    override func viewDidLoad() {
        super.viewDidLoad()

        userWorldMessages = ownWorldMessages.shared.worldMessages
    }

    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
       return userWorldMessages.count
    }
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
let cell = tableView.dequeueReusableCell(withIdentifier: "ProfileWorldMessageCell", for: indexPath) as! ProfileWorldMessageCell

        let worldMessage = userWorldMessages[indexPath.row]

        cell.worldMessageData = worldMessage

        cell.messageLabel.text = worldMessage.message

        if let image = UIImage(named: "bubble") {
            let h = image.size.height / 2
            let w = image.size.width / 2
            cell.bubbleImageView.image = image
                .resizableImage(withCapInsets:
                    UIEdgeInsetsMake(h, w, h, w),
                                resizingMode: .stretch).withRenderingMode(.alwaysTemplate)

            cell.bubbleImageView.tintColor = appColors.worldMessageBubble

        }

        let calendar = NSCalendar.current
        let date = Date(timeIntervalSince1970: Double(worldMessage.leftTime))
        let dateFormatter = DateFormatter()
        dateFormatter.locale = Locale(identifier: "en_US")
        var dateFormat = "yyyy MMM dd"
        if calendar.isDateInToday(date) {
            // Today
            dateFormat = "HH:mm"
            dateFormatter.string(from: date)
        } else if (calendar.date(byAdding: .weekOfYear, value: -1, to: Date())! < date){
            // This last 7 days
            dateFormat = "EEEE HH:mm"
        } else if (calendar.date(byAdding: .month, value: -12, to: Date())! < date){
            // This year
            dateFormat = "MMM dd"
        } else {
            dateFormat = "yyyy MMM dd"
        }
        dateFormatter.dateFormat = dateFormat
        let strDate = dateFormatter.string(from: date)

        cell.timeLabel.text = strDate

        cell.commentsButton.setTitle("\(worldMessage.comments!) comments", for: .normal)
        cell.likesButton.setTitle("\(worldMessage.likes!) likes", for: .normal)

        // tapped
        let tapGestureRecognizer = UITapGestureRecognizer(target: self, action: #selector(bubbleTappedHandler))
        cell.bubbleButton.addGestureRecognizer(tapGestureRecognizer)
        cell.bubbleButton.isUserInteractionEnabled = true

        // long pressed
        let longPressGestureRecognizer = UILongPressGestureRecognizer(target: self, action: #selector(bubbleLongPressHandler))
        longPressGestureRecognizer.minimumPressDuration = 0.5
        cell.addGestureRecognizer(longPressGestureRecognizer)
        cell.isUserInteractionEnabled = true

        return cell
    }
    var cellHeights: [IndexPath : CGFloat] = [:]
    func tableView(_ tableView: UITableView, willDisplay cell: UITableViewCell, forRowAt indexPath: IndexPath) {
        cellHeights[indexPath] = cell.frame.size.height

        if indexPath.row == UserWorldMessagesStore.shared.worldMessages.count - 1 && userWorldMessagesCanLoadMore == true {
            loadOlderOwnWorldMessages()
        }
    }
    func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
        return UITableViewAutomaticDimension
    }
    func tableView(_ tableView: UITableView, estimatedHeightForRowAt indexPath: IndexPath) -> CGFloat {
        if cellHeights[indexPath] != nil {
            return CGFloat(Float(cellHeights[indexPath] ?? 0.0))
        }
        else {
            return UITableViewAutomaticDimension
        }
    }
    func scrollViewDidScroll(_ scrollView: UIScrollView) {
        if (lastContentOffsetY != nil){
            tableView.setContentOffset(CGPoint(x: 0, y: lastContentOffsetY!), animated: false)
        }
    }



    func loadOlderOwnWorldMessages(){
        profileFunctions.loadOlderOwnWorldMessages(startIndex: (userWorldMessages.count)) { response, newUserWorldMessages in
            if let response = response {
                if response.type == 1 {
                    DispatchQueue.main.async(execute: {() -> Void in
                        if newUserWorldMessages != nil {

                            var insertRows : [IndexPath] = []
                            var fromIndex = UserWorldMessagesStore.shared.worldMessages.count - 1
                            for worldMessage in newUserWorldMessages! {
                                UserWorldMessagesStore.shared.worldMessages.append(worldMessage)
                                insertRows.append(IndexPath(row: fromIndex, section: 1))
                                fromIndex += 1
                            }
                            self.userWorldMessages = UserWorldMessagesStore.shared.worldMessages

                            self.tableView.beginUpdates()
                            self.tableView.insertRows(at: insertRows, with: .automatic)
                            self.tableView.endUpdates()


                        }

                        })
                } else {
                    DispatchQueue.main.async(execute: {() -> Void in
                        self.handleResponses.displayError(title: response.title, message: response.message)
                    })
                }
            }
        }
    }

    @objc func bubbleTappedHandler(sender: UITapGestureRecognizer, should: Bool) {
        let touchPoint = sender.location(in: self.tableView)
        if let indexPath = tableView.indexPathForRow(at: touchPoint) {
            if indexPath == currentSelectedWorldMessageIndexPath {
                // it was already selected, so deselect it

                deselectCurrentSelectedWorldMessage()
            } else {
                // select new one, deselect old one if was selected

                if (currentSelectedWorldMessageIndexPath != nil){
                    deselectCurrentSelectedWorldMessage()
                }

                selectWorldMessage(indexPath: indexPath)
            }
        }
    }
    func selectWorldMessage(indexPath: IndexPath) {

        tableView.selectRow(at: indexPath, animated: true, scrollPosition: .none)
        currentSelectedWorldMessageIndexPath = indexPath

        if let cell = tableView.cellForRow(at: indexPath) as? ProfileWorldMessageCell {

            // Change some constraints
            cell.messageLabelTopConstraint.constant = 14
            cell.messageLabelBottomConstraint.constant = 14
            UIView.animate(withDuration: 0.15, animations: {
                self.view.layoutIfNeeded()
                self.tableView.beginUpdates()
                self.tableView.endUpdates()
            }, completion: nil)
        }
    }
    @objc func deselectCurrentSelectedWorldMessage(){
        UIMenuController.shared.setMenuVisible(false, animated: true)
        if (currentSelectedWorldMessageIndexPath == nil){
            return
        }
        let indexPath = currentSelectedWorldMessageIndexPath!

        if let cell = tableView.cellForRow(at: indexPath) as? ProfileWorldMessageCell {

            // Change back some constraints
            cell.messageLabelTopConstraint.constant = 10
            cell.messageLabelBottomConstraint.constant = 10
            UIView.animate(withDuration: 0.15, animations: {
                self.view.layoutIfNeeded()
                self.tableView.beginUpdates()
                self.tableView.endUpdates()
            }, completion: nil)
        }

        currentSelectedWorldMessageIndexPath = nil
    }

    @objc func bubbleLongPressHandler(sender: UILongPressGestureRecognizer, should: Bool) {
        // Show options like copy, delete etc.
    }



}

1 Ответ

0 голосов
/ 19 мая 2018

Есть несколько вещей, которые очень неверны с вашей cellForRow функцией:

  • , ячейки используются повторно, и вы игнорируете этот факт.Вы создаете новые экземпляры распознавателей жестов каждый раз, когда вызывается cellForRow, а когда ячейка используется повторно, вы добавляете еще один.Это означает, что если ваша ячейка была повторно использована в 50-й раз, в нее будет добавлено 50 распознавателей жестов.Вы должны создавать свои распознаватели жестов только один раз
  • Вы создаете DateFormatter и каждый раз устанавливаете его свойство dateFormat, и это действительно очень дорогой процесс.Вместо этого у вас должен быть статический экземпляр formatter для каждого формата даты
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...