Ho scritto due modi per caricare in modo asincrono le immagini nella mia cella UITableView. In entrambi i casi l'immagine verrà caricata correttamente, ma quando scorro la tabella le immagini cambieranno alcune volte fino a quando lo scorrimento terminerà e l'immagine tornerà all'immagine giusta. Non ho idea del perché questo stia accadendo.
#define kBgQueue dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0)
- (void)viewDidLoad
{
[super viewDidLoad];
dispatch_async(kBgQueue, ^{
NSData* data = [NSData dataWithContentsOfURL: [NSURL URLWithString:
@"http://myurl.com/getMovies.php"]];
[self performSelectorOnMainThread:@selector(fetchedData:)
withObject:data waitUntilDone:YES];
});
}
-(void)fetchedData:(NSData *)data
{
NSError* error;
myJson = [NSJSONSerialization
JSONObjectWithData:data
options:kNilOptions
error:&error];
[_myTableView reloadData];
}
- (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView {
// Return the number of sections.
return 1;
}
- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section{
// Return the number of rows in the section.
// Usually the number of items in your array (the one that holds your list)
NSLog(@"myJson count: %d",[myJson count]);
return [myJson count];
}
- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath{
myCell *cell = [tableView dequeueReusableCellWithIdentifier:@"cell"];
if (cell == nil) {
cell = [[myCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:@"cell"];
}
dispatch_async(kBgQueue, ^{
NSData *imgData = [NSData dataWithContentsOfURL:[NSURL URLWithString:[NSString stringWithFormat:@"http://myurl.com/%@.jpg",[[myJson objectAtIndex:indexPath.row] objectForKey:@"movieId"]]]];
dispatch_async(dispatch_get_main_queue(), ^{
cell.poster.image = [UIImage imageWithData:imgData];
});
});
return cell;
}...
- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath{
myCell *cell = [tableView dequeueReusableCellWithIdentifier:@"cell"];
if (cell == nil) {
cell = [[myCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:@"cell"];
}
NSURL* url = [NSURL URLWithString:[NSString stringWithFormat:@"http://myurl.com/%@.jpg",[[myJson objectAtIndex:indexPath.row] objectForKey:@"movieId"]]];
NSURLRequest* request = [NSURLRequest requestWithURL:url];
[NSURLConnection sendAsynchronousRequest:request
queue:[NSOperationQueue mainQueue]
completionHandler:^(NSURLResponse * response,
NSData * data,
NSError * error) {
if (!error){
cell.poster.image = [UIImage imageWithData:data];
// do whatever you want with image
}
}];
return cell;
}
5
Stai tentando di archiviare informazioni nelle celle effettive. Questo è male, molto male. Dovresti archiviare le informazioni in n array (o qualcosa di simile) e quindi visualizzarle nelle celle. Le informazioni in questo caso sono il vero UIImage. Sì, caricarlo in modo asincrono ma caricarlo in un array.
—
Fogmeister,
@Fogmeister Ti riferisci a
—
Rob,
poster? È presumibilmente una visualizzazione di immagini nella sua cella personalizzata, quindi ciò che EXEC_BAD_ACCESS sta facendo è perfettamente giusto. Hai ragione a dire che non dovresti usare la cella come repository per i dati del modello, ma non credo sia quello che sta facendo. Sta solo dando alla cella personalizzata ciò di cui ha bisogno per presentarsi. Inoltre, e questo è un problema più sottile, sarei diffidente riguardo alla memorizzazione di un'immagine, stessa, nel tuo array di modelli che supporta la vista da tavolo. È meglio utilizzare un meccanismo di memorizzazione nella cache delle immagini e l'oggetto modello deve recuperare da quella cache.
Sì, esattamente il mio punto. Osservando la richiesta (che viene mostrata per intero) sta scaricando l'immagine in modo asincrono e inserendola direttamente in imageView nella cella. (Usando così la cella per memorizzare i dati, cioè l'immagine). Quello che dovrebbe fare è fare riferimento a un oggetto e richiedere l'immagine da quell'oggetto (contenuto in un array o da qualche parte). Se l'oggetto non ha ancora l'immagine, dovrebbe restituire un segnaposto e scaricare l'immagine. Quindi quando l'immagine viene scaricata e pronta per essere visualizzata, informa la tabella in modo che possa aggiornare la cella (se è visibile).
—
Fogmeister,
Quello che sta facendo forzerà il download ogni volta che scorre fino a quella cella nella tabella. Se le immagini vengono archiviate in modo persistente dipende da lui, ma almeno memorizzarle per la durata della visualizzazione della tabella.
—
Fogmeister,
Esatto: D In questo modo devi solo recuperare l'immagine dall'URL una volta. Lo vedrai su cose come Facebook Friend Picker. All'inizio tutti gli avatar sono segnaposto grigi. Quindi, mentre scorri, si riempiono tutti mentre si muove. Ma poi quando si scorre indietro a una cella mostrata in precedenza, mostrerà immediatamente l'immagine già scaricata.
—
Fogmeister,