NSFetchedResultsController使加载视图控制器非常慢

时间:2017-01-24 18:45:50

标签: ios swift performance core-data nsfetchedresultscontroller

(更新:在下面的编辑4中,我确实找到了问题的原因!)

我正在使用tableView NSFetchedResultsController。这就是我获取数据的方式(我在viewDidLoad()中称之为):

let fetchRequest: NSFetchRequest<Entry> = Entry.fetchRequest()
        let sortSections = NSSortDescriptor(key: #keyPath(Entry.section), ascending: false)
        let sortDate = NSSortDescriptor(key: #keyPath(Entry.date), ascending: true)
        fetchRequest.sortDescriptors = [sortSections, sortDate]
        fetchRequest.fetchBatchSize = 15 // this seems to have no impact
        fetchedResultsController = NSFetchedResultsController(fetchRequest: fetchRequest, managedObjectContext: managedObject, sectionNameKeyPath: #keyPath(Entry.section), cacheName: "EntriesCache")

这在某种程度上非常缓慢(当我转向包含此view controller的{​​{1}}时,我会注意到这一点。

在我的设备上,我在数据库中使用了大约200个table view个对象。 Entry出现的时间略多于1秒。但我也尝试了大约10个物体,它的速度并不快。 (奇怪的是,在模拟器上它的速度非常快)

我尝试使用view controller进行分析。在这1秒钟内,CPU处于100%。这是正常的吗?

在我注意到这种缓慢的表现之前,我没有这条线

Time Profiler

我添加了它,但没有改变。它甚至没有一点点快。此外,我在加载后打印了已获取对象的计数:

fetchRequest.fetchBatchSize = 15

它表示所有对象都被加载,而不只是其中的15个(因为在表格视图中你不能一次看到它)。那是为什么?

这是我用于print(fetchedResultsController.fetchedObjects?.count) 的{​​{1}}实体 Entry Entity

我不知道您需要哪些代码/信息才能帮助我(我在性能问题方面不是专家)。请告诉我,如果你还需要更多的东西。

谢谢你们!

编辑:

我如何访问managedObjectContext:

Entry

编辑2(也许我找到了原因?): 好的,所以我编辑了我的方案,以便向我显示所有SQL查询。首先,它加载15行(当15是table view时)几次。但之后它变得有趣:

我没有完全统计它,但我很确定它会对数据库中的每个对象进行以下查询(/查询)。我尝试了600个左右的对象,这些SQL查询需要花费很长时间才能完成:

lazy var managedObject: NSManagedObjectContext = {
        let managedObject = self.appDelegate.persistentContainer.viewContext
        return managedObject
    }()

我不知道究竟是什么,但我认为这会造成延误。在这些查询完成后,将显示视图控制器。

编辑3:

以下是我的fetchBatchSize数据源方法:

CoreData: sql: SELECT t0.Z_ENT, t0.Z_PK, Z_FOK_ENTRY FROM ZENTRYTEXT t0 WHERE  t0.ZENTRY = ? 
CoreData: annotation: sql connection fetch time: 0.0001s
CoreData: annotation: total fetch execution time: 0.0002s for 1 rows.
CoreData: annotation: to-many relationship fault "entryTexts" for objectID 0xd000000006480000 <x-coredata://C53DABDD-5D31-4ADE-B6E7-3ED69454B572/Entry/p402> fulfilled from database.  Got 1 rows
CoreData: sql: SELECT 0, t0.Z_PK, t0.Z_OPT, t0.ZTEXT, t0.ZENTRY, t0.Z_FOK_ENTRY FROM ZENTRYTEXT t0 WHERE  t0.Z_PK = ? 
CoreData: annotation: sql connection fetch time: 0.0001s
CoreData: annotation: total fetch execution time: 0.0002s for 1 rows.
CoreData: annotation: fault fulfilled from database for : 0xd000000007940002 <x-coredata://C53DABDD-5D31-4ADE-B6E7-3ED69454B572/EntryText/p485>

编辑4(我找到了原因):

问题在于这个功能:

table view

我玩弄了这个,现在我几乎可以肯定这条线是麻烦制造者:

func numberOfSections(in tableView: UITableView) -> Int {
        guard let sections = fetchedResultsController.sections else {
            return 0
        }
        return sections.count
    }

    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {        
        guard let sectionInfo = fetchedResultsController.sections?[section] else {
            return 0
        }

        return sectionInfo.numberOfObjects
    }

    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "bitCell") as! BitCell
        let entry = fetchedResultsController.object(at: indexPath)

        cell.configure(entry: entry)
        return cell
    }

    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        let entry = fetchedResultsController.object(at: indexPath)
        extendBitPopup.fadeIn(withEntry: entry, completion: nil)
    }

    func scrollViewDidScroll(_ scrollView: UIScrollView) {
        if scrollView.contentOffset.y >= 400 {
            UIView.animate(withDuration: 0.5, animations: { 
                self.arrowUpButton.alpha = 1.0
                self.arrowUpButton.isEnabled = true
                self.arrowUpButton.isUserInteractionEnabled = true
            })
        } else {
            UIView.animate(withDuration: 0.5, animations: {
                self.arrowUpButton.alpha = 0.0
                self.arrowUpButton.isEnabled = false
                self.arrowUpButton.isUserInteractionEnabled = false
            })
        }
    }


    func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
        let entry = fetchedResultsController.object(at: indexPath)
        guard !entry.isFault else {
            return 0
        }
        // this estimates the height the cell needs when the text is inserted
        return BitCell.suggestedHeight(forEntry: entry)
    }

    func tableView(_ tableView: UITableView, titleForHeaderInSection section: Int) -> String? {
        if let sectionInfo = fetchedResultsController.sections?[section] {
            let dateFormatter = DateFormatter()
            // Entry.section has this format: "yyyyMMdd" I chose this to make a section for each day. 
            dateFormatter.dateFormat = "yyyyMMdd"
            let date = dateFormatter.date(from: sectionInfo.name)!

            dateFormatter.dateStyle = .full
            dateFormatter.timeStyle = .none

            return dateFormatter.string(from: date)
        }

        return ""

    }


    func tableView(_ tableView: UITableView, canEditRowAt indexPath: IndexPath) -> Bool {
        return true
    }


    func tableView(_ tableView: UITableView, heightForFooterInSection section: Int) -> CGFloat {
        return 25
    }

    func tableView(_ tableView: UITableView, viewForFooterInSection section: Int) -> UIView? {
        let view = UIView()
        return view
    }

    func tableView(_ tableView: UITableView, editActionsForRowAt indexPath: IndexPath) -> [UITableViewRowAction]? {

        let moment = UITableViewRowAction(style: .normal, title: "Moment") { (action, indexPath) in
            let entry = self.fetchedResultsController.object(at: indexPath)
            entry.isMoment = !entry.isMoment
            self.appDelegate.saveContext()
            tableView.setEditing(false, animated: true)
        }
        moment.backgroundColor = AppTheme.baseGray

        let delete = UITableViewRowAction(style: .destructive, title: "Delete") { (action, index) in
            let entry = self.fetchedResultsController.object(at: indexPath)
            self.managedObject.delete(entry)
            self.appDelegate.saveContext()
            tableView.setEditing(false, animated: true)
        }
        delete.backgroundColor = AppTheme.errorColor

        return [delete, moment]
    }

如果我在此行之前返回一个静态CGFloat,视图控制器几乎立即加载(我用700个对象测试它)。此外,它然后只获取前50个项目(即func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat { let entry = fetchedResultsController.object(at: indexPath) guard !entry.isFault else { return 0 } return BitCell.suggestedHeight(forEntry: entry) } ),如果向下滚动,它只会加载更多。

如果我在此行之后返回,它会获取所有数据(根据许多SQL查询),它会变得非常慢,并且会出现整个延迟问题。

所以,我认为如果上面的这一行试图获得一个let entry = fetchedResultsController.object(at: indexPath) 的对象(可能它然后尝试从数据库中重新提取或其他东西),就会出现问题。

现在的问题是:如何解决这个问题?我需要fetchBatchSize对象来估计单元格高度,但如果我知道对象没有出现故障(如果这是问题),我只想调用此行。我怎么能这样做?

1 个答案:

答案 0 :(得分:2)

使用估计的高度委托方法,并返回固定大小。然后,表视图只应在需要显示该行时查询行的实际高度,以便它可以正确使用获取结果控制器的错误和批处理功能。

如果一个表有400行,并且你已经实现了heightForRow,那么它将为表中的每一行调用委托方法,以便它可以计算表视图的内容大小。在特定索引处询问对象的结果控制器会自动将其从故障转换,并且在任何情况下,返回零大小将完全弄乱表的内容大小。

如果您提供估计的大小,或者通过使用委托方法或将其设置为表上的属性,则表视图将仅调用已显示或即将显示的行的特定高度方法。它将使用估计的高度来猜测表格视图的内容大小。这意味着滚动时内容大小会略有波动,但这并不是很明显。