_fetchedResultsController objectAtIndexPath:indexPath freezes app

Question

It takes too much time on [_fetchedResultsController objectAtIndexPath:indexPath], is there any alternative way ? heightForRowAtIndexPath takes too much time on this point. I don't know this is wrong question or not, but please help me to get the proper solution.

   - (CGFloat)tableView:(UITableView *)tableView heightForRowAtIndexPath:(NSIndexPath   *)indexPath
 {
    ThreadInfo *info=[_fetchedResultsController objectAtIndexPath:indexPath];
    NSString* objectId = info.threadID;
    if (![self.idToCellHeight objectForKey: objectId])
    {
        CGFloat height = [self calculateHeightForId:info];
        [self.idToCellHeight setObject:[NSNumber numberWithFloat:height] forKey: objectId];
    }
    return [[self.idToCellHeight objectForKey:objectId] floatValue];

}

I'am really stuck at this point. – Jan May 18 '15 at 11:47 — Jan, May 18 '15 at 11:47

score 0 · Answer 1 · answered May 18 '15 at 15:42

0

Have you run Instruments? Run the Time Profiler. What is taking up the time? Find it and eliminate it.

If you can't find it, post the trace here for others to look at it.

answered May 18 '15 at 15:42

Marcus S. Zarra

46,571
9
101
182

score 0 · Accepted Answer · edited May 23 '17 at 12:32

The alternative way is to use self-sizing cells. You won't have to calculate (or cache) any row heights.

Add Auto Layout constraints to your tableViewCell's subviews which will cause the cell to adjust its height based on the subview's contents.
Enable row height estimation.

self.tableView.rowHeight = UITableViewAutomaticDimension; self.tableView.estimatedRowHeight = 44.0;

For more information, see the detailed walkthrough by smileyborg in his answer to Using Auto Layout in UITableView for dynamic cell layouts & variable row heights.

_fetchedResultsController objectAtIndexPath:indexPath freezes app

2 Answers2