CODEWITHSUNDEEP
iosobjective-cuitableview
Chris
Chris

Reputation: 351

Reusable TableViewCell loads an old image

I have a dynamic tableview that can be searched through with a search box, and if I load an image in a cell in the background and the search text changes, the object in the cell may also change, and sometimes an old image that was loading in the background will finish and display when it shouldn't. Not every image is loaded in the same way, but this is the method that causes the problem:

    [self.CarImage sd_setImageWithURL:imageURL
                                 completed:^(UIImage *image, NSError *error, SDImageCacheType cacheType, NSURL *imageurl){
                                     [self.CarImage setAlpha:0.0];
                                     [UIImageView animateWithDuration:.5 animations:^{
                                         [self.CarImage setAlpha:1.0];
                                     }];
                                 }];

Any ideas would be appreciated

Upvotes: 0

Views: 75

Answers (2)

Ratul Sharker
Ratul Sharker

Reputation: 8011

Here the issue is, image is loaded from url after you have already reused the cell. You can store the url of the image, each and everytime the cell being reused the url of the images should be stored inside the custom cell class. Then inside the completion block check that the loaded image url and the stored image url are same or not using isEqual: method of NSURL. If they are same then everything is ok, cell is not being reused, otherwise the cell is been used for another row, then just dont set the image.

self.carImgUrl = imageURL;//here carImgUrl is a property which you gonna set everytime when the cell is being reused.
[self.CarImage sd_setImageWithURL:imageURL
                             completed:^(UIImage *image, NSError *error, SDImageCacheType cacheType, NSURL *imageurl){
                                 if([imageurl isEqual:self.carImgUrl])
                                 {
                                    [self.CarImage setAlpha:0.0];
                                     [UIImageView animateWithDuration:.5 animations:^{
                                         [self.CarImage setAlpha:1.0];
                                     }];
                                 }
                             }];

Upvotes: 0

Kuntal Gajjar
Kuntal Gajjar

Reputation: 802

You need to do two things here

1) in prepareforresue method Cancel current downloading request

2) in prepareforresue method set image to nil

Hope this will fix your issue

Upvotes: 1

Related Questions