objective-c multithreading

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • objective-c multithreading

    Hallo Leute,
    ich habe mal wieder eine Frage zu multithreading.
    Ich benutzte das Asset Library um mir die Photos zu holen, das in dieser Zeit nicht die UI Blockiert ist benutzte ich ein Operation Block.
    So weit Funktioniert alles nur jetzt ist es extrem langsam und eigentlich funktioniert das Asset Library eh asynchron ich verstehe nicht warum die UI blockiert ist.
    Ich habe wahrscheinlich irgendwas Falsch verstanden ^^ vielleicht könnt ihr mich aufklären.

    Hier der Code.
    Photo Engine:

    Quellcode

    1. #import "LCPhotoEngine.h"
    2. #import "NSDate+LCTimeShiftDate.h"
    5. @implementation LCPhotoEngine
    8. - (void)getAllPhotos
    9. {
    11. NSMutableArray *photos = [NSMutableArray new];
    12. ALAssetsLibrary *assetLibrary = [ALAssetsLibrary new];
    14. [assetLibrary enumerateGroupsWithTypes:ALAssetsGroupSavedPhotos usingBlock:^(ALAssetsGroup *group, BOOL *stop) {
    15. @autoreleasepool {
    17. if (group) { // #1
    19. [group setAssetsFilter:[ALAssetsFilter allPhotos]];
    21. [group enumerateAssetsUsingBlock: ^(ALAsset *asset, NSUInteger index, BOOL *stop) {
    23. if (!asset && (index == NSNotFound)) {
    24. // NotFound
    25. } else {
    27. [photos addObject:asset];
    29. if ([self.delegate respondsToSelector:@selector(photoEngine:didReturnWithAssetRepresentation:)]) {
    30. [self.delegate photoEngine:self didReturnWithAssetRepresentation:asset];
    31. }
    33. }
    34. }];
    36. } else { // #1
    38. if ([self.delegate respondsToSelector:@selector(photoEngine:didFinishWithResults:)]) {
    39. [self.delegate photoEngine:self didFinishWithResults:(NSArray*)photos];
    40. }
    42. } // #1 else
    44. } // autoreleasepool
    47. }
    53. // failureBlock
    54. failureBlock:^(NSError *error) {
    55. NSLog(@"error enumerating AssetLibrary groups %@\n", error);
    56. }];
    58. }
    61. - (NSArray*)filterValidPhotosInAssets:(NSArray*)assets byDateBlocks:(NSArray*)dates
    62. {
    63. __block NSMutableArray *validPhotos = [NSMutableArray new];
    65. [assets enumerateObjectsUsingBlock:^(ALAsset *asset, NSUInteger idx, BOOL *stop) {
    66. if([self isValidAsset:asset byDates:dates]) {
    67. [validPhotos addObject:[UIImage imageWithCGImage:asset.defaultRepresentation.fullScreenImage]];
    68. }
    69. }];
    71. if(!validPhotos.count) {
    72. [validPhotos addObject:[UIImage imageNamed:@"dev_current_record_image@2x.png"]];
    73. }
    75. return validPhotos;
    76. }
    79. - (BOOL)isValidAsset:(ALAsset*)asset byDates:(NSArray*)dates
    80. {
    81. __block NSDate *creationDate = [NSDate dateTimeShiftWithDate:[asset valueForProperty:ALAssetPropertyDate]];
    82. __block BOOL isValid = NO;
    84. [dates enumerateObjectsUsingBlock:^(NSArray *periodOfTime, NSUInteger idx, BOOL *stop) {
    86. NSDate *startDate = periodOfTime.firstObject;
    87. NSDate *enddate = periodOfTime.lastObject;
    89. if([startDate compare:creationDate] == NSOrderedAscending &&
    90. [enddate compare:creationDate] == NSOrderedDescending
    91. ) {
    92. isValid = YES;
    93. }
    95. }];
    97. return isValid;
    98. }
    101. @end
    Alles anzeigen
    Methoden Aufruf:

    Quellcode

    1. NSBlockOperation *photoOperation = [NSBlockOperation blockOperationWithBlock:^{ [self.photoEngine getAllPhotos]; }];
    12. [self.tableDataOperationQueue addOperation:photoOperation];
    Alles anzeigen



    ich bedanke mich schon einmal im voraus für die Hilfe.

  • Du musst normalerweise NSOperation subclasen und isConcurrent überschreiben, damit asynchron gearbeitet wird.

    Mit NSBlockOperation habe ich noch nie gearbeitet. Aber wenn man da erst ne Subclass erstellen muss um dann einen async. Block losschicken zu können, der einem eigentlich das Subclassen spart, dann werde ich das wohl auch nicht tun.

    Gruß
    Seminare, Artikel, Code. ObjectiveCeeds - alles für die Apfelzucht.

  • ja ich finde es auch cool das es so einfach geht ^^.
    hier habe ich noch einmal den text aus der documentation

    Quellcode

    1. The NSBlockOperation class is a concrete subclass of NSOperation that manages the concurrent execution of one or more blocks

    aber ich finde es komisch warum es jetzt so lange dauert um die Bilder aus zu lesen

  • Lustig das kapiere ich nicht :(
    also meine delegate Methoden werden sofort aufgerufen es gibt keine Zeitverzögerung.
    Hier einmal die Methoden:

    didReturnWithAssetRepresentation

    Quellcode

    1. - (void)photoEngine:(LCPhotoEngine*)photoEngine didReturnWithAssetRepresentation:(ALAsset*)asset{ if(!self.wasFoundFirstValidPhoto) { if([self.photoEngine isValidAsset:asset byDates:[LCPersistanceEngine datesByDatesBlockWithActivityName:self.activityName]]) { [self setActivityImageViewWithRandomImage:[UIImage imageWithCGImage:asset.defaultRepresentation.fullScreenImage]]; self.foundFirstValidPhoto = YES; } }

    setActivityImageViewWithRandomImage:

    Quellcode

    1. - (void)setActivityImageViewWithRandomImage:(UIImage*)randomImage{ self.activityImageView.image = randomImage; [self.activityIndicatorView removeFromSuperview];
    2. }


    die zwei Methoden werden sofort aufgerufen nur nur das setzten des Bildes in den ImageView dauert jetzt extrem lange.
    Eigentlich dauert jetzt alles was ich mit der UI mach sehr lange.
    Woran kann das liegen ?
    Kann es daran liegen weil ich in einem anderen OperationQueue arbeite ?

  • Dann wird zu 99.9% dein setImage: ... bluBlab nicht aus dem Hauptthread aufgerufen. Nachrichten an UI Elemente mussen immer im Hauptthread abgeschickt werden, sonst gibts ein undefiniertes Verhalten - meistens ein paar Sekunden Verzögerung.

    Nimm z.B. eine der performSelectorInMainThread Methoden. Oder über die c API:

    Quellcode

    1. dispatch_async(dispatch_get_main_queue(), ^{
    2. // code....
    3. });
    Seminare, Artikel, Code. ObjectiveCeeds - alles für die Apfelzucht.

  • Wenn du Code async. ausführen willst, brauchst du einfach nur:

    Quellcode

    1. dispatch_queue_t queue = dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0);
    3. dispatch_async(queue, ^{
    5. // code
    7. });


    Einen Autoreleasepool musst du nicht erzeugen (wenn ich das noch richtig in Erinnerung habe, brauchst du den bei der NSOperation API auch nicht, der wird hier auch von System angelegt)
    Seminare, Artikel, Code. ObjectiveCeeds - alles für die Apfelzucht.

  • Setzt du das Bild etwa im Backgroundthread? Das darfst du natürlich nicht. Du musst dem MsinThread eine Nachricht schicken, dass das Bild da ist und der muss dieses dann in das ImageView setzen.

    Gruß

    Claus
    2 Stunden Try & Error erspart 10 Minuten Handbuchlesen.

    Pre-Kaffee-Posts sind mit Vorsicht zu geniessen :)