2012-12-13 7 views
7

Jestem całkiem nowy w Objective-C, więc mam nadzieję, że to wszystko ma sens. Pobrałem obrazy z serwera i wyświetlono je w widoku obrazu w metodzie collectionView cellForItemAtIndexPath:. Problem, który napotykam, polega na tym, że obrazy nie są buforowane. Wygląda na to, że za każdym razem, gdy komórka jest ponownie wykorzystywana, powiązany obraz z serwera jest ponownie pobierany.Buforowanie obrazu i UICollectionView

W mojej metody viewDidLoad Tworzę NSMutableDictionary:

imageDictionary = [[NSMutableDictionary alloc]initWithCapacity:50.0]; 

Od czytanie dokumentacji i patrząc na odpowiedzi na podobne pytania myślałem, że to plus Poniższy kod wystarczy. Byłem w tym od kilku dni i wiem, że jest coś, czego mi brakuje lub koncepcja, której nie chwytam.

#pragma mark - UICollectionView Data Source 
- (NSInteger)collectionView:(UICollectionView *)view numberOfItemsInSection:(NSInteger)section;{ 
    NSLog(@"Begin retrieving photos"); 
    return [self.photos count]; 
} 

-(NSInteger)numberOfSectionsInCollectionView:(UICollectionView *)collectionView{ 
    return 1; 
} 

- (UICollectionViewCell *)collectionView:(UICollectionView *)collectionView cellForItemAtIndexPath:(NSIndexPath *)indexPath;{ 
    CollectionViewCell *cell = [collectionView dequeueReusableCellWithReuseIdentifier:@"MY_CELL" forIndexPath:indexPath]; 
    cell.imageView.image = nil; 

    if (cell.imageView.image == nil) { 
    dispatch_queue_t downloadQueue = dispatch_queue_create("image downloader", NULL); 
    dispatch_async(downloadQueue, ^{ 
     NSData *data = [NSData dataWithContentsOfURL:[NSURL URLWithString:[[self.photos objectAtIndex:indexPath.row] objectForKey:@"fullimage"]]]; 
     UIImage *image = [UIImage imageWithData:data]; 
     [imageDictionary setObject:image forKey:@"Image"]; 

     dispatch_async(dispatch_get_main_queue(), ^{ 
      cell.imageView.image = [imageDictionary objectForKey:@"Image"]; 
      [cell setNeedsDisplay]; 
     }); 
    }); 
    } 
    return cell; 
} 

Każda pomoc zostanie bardzo doceniona. Z góry dziękuję.

+2

Wygląda na to, że ponownie pobierasz obraz za każdym razem. Widok kolekcji nie buforuje obrazów dla Ciebie. Musisz utworzyć własny słownik pamięci podręcznej. Zajrzyj do NSCache. – yuf

+0

@yuf zrobię - dzięki! – BrianS

Odpowiedz

15

@yuf - jeszcze raz dziękuję za wskazany kierunek. Wygląda na to, że NSCache przynosi mi rezultaty, których szukałem. Oto kod, który działa poprawnie. Jeśli ktoś ma podobny problem, możesz porównać to z moim pierwotnym pytaniem.

- (UICollectionViewCell *)collectionView:(UICollectionView *)collectionView cellForItemAtIndexPath:(NSIndexPath *)indexPath;{ 
    CollectionViewCell *cell = [collectionView dequeueReusableCellWithReuseIdentifier:@"MY_CELL" forIndexPath:indexPath]; 

    NSString *imageName = [[self.photos objectAtIndex:indexPath.row] objectForKey:@"fullimage"]; 
    UIImage *image = [imageCache objectForKey:imageName]; 

    if(image){ 

     cell.imageView.image = image; 
    } 

    else{ 

    cell.imageView.image = nil; 

    dispatch_queue_t downloadQueue = dispatch_queue_create("image downloader", NULL); 
    dispatch_async(downloadQueue, ^{ 

     NSData *data = [NSData dataWithContentsOfURL:[NSURL URLWithString:[[self.photos objectAtIndex:indexPath.row] objectForKey:@"fullimage"]]]; 
     UIImage *image = [UIImage imageWithData:data]; 

     dispatch_async(dispatch_get_main_queue(), ^{ 

      cell.imageView.image = image; 

     }); 

     [imageCache setObject:image forKey:imageName]; 
    }); 
    } 

    return cell; 
} 
+5

To nie jest całkowicie poprawne, ponieważ jeśli komórki są ponownie używane poprawnie, możliwe jest, że do czasu załadowania obrazu i zaktualizowania komórki, to wystąpienie komórki jest używane do reprezentowania innego rekordu w modelu. Lepiej zaktualizować pamięć podręczną, a następnie wywołać widok kolekcji reloadItemsAtIndexPaths: i uzyskać cellForItemAtIndexPath, aby ustawić obraz z pamięci podręcznej (jeśli jest widoczny). – Ants

+0

Ach, ma sens. W rzeczywistości wpadłem na ten problem (tylko raz, ale rozumiem, co masz na myśli). Dzięki! – BrianS

+0

@Ants, jeśli możesz dodać kod, wiele osób skorzysta (w tym ja). Dzięki. – Satyam

Powiązane problemy