AFNetworking für Image-Downloads, nicht reagierende Benutzeroberfläche

Question

Ich verwende AFNetworking, um Bilder von einer URL zu ziehen, die Größe zu ändern, auf Festplatte zu speichern und den Pfad in Core Data zu protokollieren, dann in eine Tabellenansicht zu laden und zu speichern. Wenn der Code ausgeführt wird, friert er meine Benutzeroberfläche ein. Ich bin mir nicht sicher, ob der Download oder die Manipulation meine Probleme verursacht.

Der Code Ich verwende unter

- (void)getPhoto:(NSInteger)type forManagedObject:(MyManagedObject*)object { 

    // download the photo 
    NSURLRequest *request = [NSURLRequest requestWithURL:[NSURL URLWithString:object.photoUrl]]; 
    AFImageRequestOperation *operation = [AFImageRequestOperation imageRequestOperationWithRequest:request success:^(UIImage *image) { 


     // MyManagedObject has a custom setters (setPhoto:,setThumb:) that save the 
     // images to disk and store the file path in the database 
     object.photo = image; 
     object.thumb = [image imageByScalingAndCroppingForSize:CGSizeMake(PhotoBlockCellButtonWidth, PhotoBlockCellButtonHeight)]; 

     NSError *nerror; 
     if (![[DataStore sharedDataStore].managedObjectContext save:&nerror]) { 
      NSLog(@"Whoops, couldn't save: %@", [nerror localizedDescription]); 
      return; 
     } 

     // notify the table view to reload the table 
     [[NSNotificationCenter defaultCenter] postNotificationName:@"ReloadTableView" object:nil]; 

    }]; 
    [operation start]; 
}

Und hier ist ein Beispielcode, die für die Setter von meinem verwalteten Objekt

- (NSString*)uniquePath{ 

    // prepare the directory string 
    NSArray *paths = NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES); 
    NSString *documentsDirectory = [paths objectAtIndex:0]; 

    // acquire a list of all files within the directory and loop creating a unique file name 
    NSFileManager *fileManager = [NSFileManager defaultManager]; 
    NSArray *existingFiles = [fileManager contentsOfDirectoryAtPath:documentsDirectory error:nil]; 
    NSString *uniquePath; 
    do { 
     CFUUIDRef newUniqueId = CFUUIDCreate(kCFAllocatorDefault); 
     CFStringRef newUniqueIdString = CFUUIDCreateString(kCFAllocatorDefault, newUniqueId); 

     uniquePath = [[documentsDirectory stringByAppendingPathComponent:(__bridge NSString *)newUniqueIdString] stringByAppendingPathExtension:@"png"]; 

     CFRelease(newUniqueId); 
     CFRelease(newUniqueIdString); 
    } while ([existingFiles containsObject:uniquePath]); 

    return uniquePath; 
} 

- (NSString*)saveImage:(UIImage*)image{ 
    NSString *path = [self uniquePath]; 
    NSData *data = UIImagePNGRepresentation(image); 
    [data writeToFile:path atomically:YES]; 
    return [NSString stringWithFormat:@"file://%@",path]; 
} 

- (void) setPhoto:(UIImage *)image { 
    self.photoUrl = [self saveImage:image]; 
}

Ich möchte diesen Thread zu einem Hintergrund schieben , aber ich bin mir nicht sicher, was die Auswirkungen von AFNetworking, Core Data und Messaging im Hinblick auf Thread-Sicherheit sind. Jeder Gedanke?

Answer 1

Basierend auf Matts Vorschlag verbesserte ich die Benutzeroberfläche, indem ich meinen Anruf wie folgt umarbeitete.

- (void)getPhoto:(NSInteger)type forManagedObject:(MyManagedObject*)object { 

    // download the photo 
    NSURLRequest *request = [NSURLRequest requestWithURL:[NSURL URLWithString:object.photoUrl]]; 
    AFImageRequestOperation *operation = [AFImageRequestOperation 
     imageRequestOperationWithRequest:request 
     imageProcessingBlock:^UIImage *(UIImage *image) { 
      return [image imageByScalingAndCroppingForSize:CGSizeMake(PhotoBlockCellButtonWidth, PhotoBlockCellButtonHeight)]; 
     } 
     cacheName:nil 
     success:^(NSURLRequest *request, NSHTTPURLResponse *response, UIImage *image) { 

      // MyManagedObject has a custom setters (setPhoto:,setThumb:) that save the 
      // images to disk and store the file path in the database 
      object.photo = image; 
      object.thumb = image; 

      NSError *nerror; 
      if (![[DataStore sharedDataStore].managedObjectContext save:&nerror]) { 
       NSLog(@"Whoops, couldn't save: %@", [nerror localizedDescription]); 
       return; 
      } 

      // notify the table view to reload the table 
      [[NSNotificationCenter defaultCenter] postNotificationName:@"ReloadTableView" object:nil];                 
     } 
     failure:^(NSURLRequest *request, NSHTTPURLResponse *response, NSError *error) { 
      NSLog(@"Error getting photo"); 
     }]; 
    [operation start]; 
}

Answer 2

AFAIK, die Art und Weise Sie Ihre Anfrage in falsch ausgeführt werden:

[operation start]; 

Sie sollten stattdessen die Operation zu einem NSOperationQueue hinzufügen:

NSOperationQueue* operationQueue = [[NSOperationQueue alloc] init]; 
    [operationQueue addOperation:operation]; 

(Sie sollten richtig Speicher-Verwaltung der Warteschlange) .

Dadurch wird Ihre Anfrage asynchron ausgeführt, die Benutzeroberfläche wird nicht blockiert und Sie müssen sich nicht mit Multithreading beschäftigen.