iOS UITableViewCell中的砌体非常慢

时间:2015-04-01 21:48:06

标签: ios iphone uitableview masonry-ios-osx

我正在开发适用于iPhone的iOS应用程序,并拥有一个UITableView,其中包含可变高度的自定义单元格。对于我正在使用 Masonry https://github.com/Masonry/Masonry的单元格布局。一切都是我喜欢的,我唯一的问题是当一个新的细胞进入屏幕时有一些滞后,足以严重阻碍用户体验。我对执行时间做了一些测试,似乎这个滞后的大部分都在函数内部:

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath;

iPhone 5上的执行时间大约为十分之一秒。此外,我发现大部分时间都发生在该功能中:

-(void)updateConstraints;

我的自定义UITableViewCell类。在这个函数中,我使用Masonry在单元格中设置了各种视图约束。所以我基本上把这种滞后缩小到了Masonry的调用。如果您想知道,整个函数看起来像这样:

-(void)updateConstraints
{
    [super updateConstraints];

    [self.thumbnailImageView mas_makeConstraints:^(MASConstraintMaker *make) {
        make.top.equalTo(self.contentView.mas_top).offset(5);
        make.left.equalTo(self.contentView.mas_left).offset(5);
        make.right.equalTo(self.contentView.mas_right).offset(-5);
        make.height.equalTo(@(self.thumbnailImageView.frame.size.width)).with.priorityHigh();

    }];

    [self.likeButton mas_makeConstraints:^(MASConstraintMaker *make) {
        make.top.equalTo(self.thumbnailImageView.mas_bottom).offset(5);
        make.left.equalTo(self.captionTextView.mas_right);
    }];

    [self.dislikeButton mas_makeConstraints:^(MASConstraintMaker *make) {
        make.top.equalTo(self.thumbnailImageView.mas_bottom).offset(5);
        make.left.equalTo(self.likeButton.mas_right);
        make.right.equalTo(self.contentView.mas_right).offset(5);
    }];

    [self.captionTextView mas_makeConstraints:^(MASConstraintMaker *make) {
        make.top.equalTo(self.thumbnailImageView.mas_bottom).offset(5);
        make.left.equalTo(self.contentView.mas_left);
        make.right.equalTo(self.likeButton.mas_left);
        make.height.equalTo(@([self captionHeight]));
    }];

    [self.timeLabel mas_makeConstraints:^(MASConstraintMaker *make) {
        make.top.greaterThanOrEqualTo(self.captionTextView.mas_bottom).offset(5).with.priorityHigh();
        make.bottom.equalTo(self.contentView.mas_bottom);
        make.left.equalTo(self.contentView.mas_left).offset(5);
    }];

    [self.scoreLabel mas_makeConstraints:^(MASConstraintMaker *make) {
        make.left.equalTo(self.captionTextView.mas_right);
        make.top.equalTo(self.likeButton.mas_bottom);
        make.right.equalTo(self.contentView.mas_right);
    }];

    [self.numCommentsLabel mas_makeConstraints:^(MASConstraintMaker *make) {
        make.top.greaterThanOrEqualTo(self.scoreLabel.mas_top);
        make.bottom.equalTo(self.contentView.mas_bottom);
        make.right.equalTo(self.contentView.mas_right);
    }];
}

我想知道的是,是否有任何方法可以减少执行时间并消除这种明显的延迟。我为每个单元设置的约束是每个单元的完全相同的约束,但有一些例外。这只是AutoLayout的一个问题,还是我正在做的事情非常错误?总的来说这只是一个糟糕的方法吗?

1 个答案:

答案 0 :(得分:0)

经过更多的探索,我发现问题的根本原因是我的细胞没有得到适当的回收。我的cellForRowAtIndexPath函数最初看起来像这样:

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath 
{
    static NSString *postTableIdentifier = @"PostTableCell";

    PostTableViewCell *cell = (PostTableViewCell *)[tableView dequeueReusableCellWithIdentifier:postTableIdentifier];
    if (cell == nil)
    {       
        NSArray *nib = [[NSBundle mainBundle] loadNibNamed:postTableIdentifier owner:self options:nil];
        cell = [nib objectAtIndex:0];
    }

    //rest of function...
}

这里tableView查找具有@" PostTableCell"的reuseIdentifier的单元格。但是,loadNibNamed:function不设置单元的reuseIdentifier,因此dequeReusableCellWithIdentifier:找不到再循环的单元格。我在这里找到了一个很好的解决方案:How can I recycle UITableViewCell objects created from a XIB?我会让它自己解决, 但是我的新代码看起来像这样:

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath 
{
    static NSString *postTableIdentifier = @"PostTableCell";

    PostTableViewCell *cell = (PostTableViewCell *)[tableView dequeueReusableCellWithIdentifier:postTableIdentifier];
    if (cell == nil)
    {
        //register nib once
        [tableView registerNib:[UINib nibWithNibName:postTableIdentifier bundle:nil] forCellReuseIdentifier:postTableIdentifier];
        cell = (PostTableViewCell *)[tableView dequeueReusableCellWithIdentifier:postTableIdentifier];
    }

    //rest of function...
}