Search code examples
objective-cuitableviewobjective-c-blocksreuseidentifier

UITableViewCell using reuseidentifier giving unwanted result with callback block


When the callback block for loadImage is run below, the table cell may have since been reused. So the image is applied to "imageView" is not relevant to this reused cell, it's the image for the old cell.

If I make the identifier unique for each cell that has an image, the problem goes away. But this gives poor performance with many results.

Can I somehow use the same reuse identifier with a callback block and have the images turn up in the correct cells?

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

    NSDictionary *place;
    PlaceTableViewCell *cell; // UITableViewCell subclass
    NSString *identifier = @"PlaceTableViewCell";

    if (cell == nil) {

        NSArray *objects;

        objects = [[NSBundle mainBundle] loadNibNamed:@"PlaceTableViewCell" owner:self options:nil];

        for(id object in objects) {

            if([object isKindOfClass:[PlaceTableViewCell class]]) {                
                cell = (PlaceTableViewCell *)object;                
                break;
            }
        }
    }

    UIImageView *imageView;
    if((imageView = (UIImageView*)[cell viewWithTag:1])) {

        NSString *filename;
        int placeImageId = 0;
        place = [places objectAtIndex:indexPath.row];

        if(place) {

            placeImageId = [[d objectForKey:@"placeImageId"] intValue];

            if(placeImageId) {

                [[RestAPIConnector sharedInstance] loadImage :placeImageId :@"thumb" :^(NSString *response){

                    NSDictionary *image = [response JSONValue];

                    if ([image objectForKey:@"img"]) {

                        NSString *b64Img = [image objectForKey:@"img"];
                        UIImage *ui = [UIImage imageWithData:[Base64 decode:b64Img]];

                        imageView.image = ui;
                    }
                }];            
            }
        }
    }

    return cell;
}

Solution

  • here is what I'm doing.

    instead of using the cell directly, I'm passing in the index path

    if(user.profileImage == nil)
    {
        if (self.tableView.dragging == NO && self.tableView.decelerating == NO) {
            NSLog(@"file for user %d doesn't exist", [user.userId intValue]);
            [self startUserProfileImageDownload:user forIndexPath:indexPath];
        }
    }
    else
    {
        cell.profileImageView.image = user.profileImage;
    
    }
    

    once the download is complete, use the index path to retrieve the cell, and update the image

    MessageCell *cell = (MessageCell *)[self.tableView cellForRowAtIndexPath:path];
    
    // Display the newly loaded image
    cell.profileImageView.image = user.profileImage;
    CALayer *roundedLayer = [cell.profileImageView layer];
    

    MessageCell is my custom cell. if you don't have use customer cell, you can use Tag to retrieve the imageView back.