Assegnazione di oggetti a variabili esterne a un blocco

Question

Il seguente codice si blocca, poiché il contenuto di sentence scompare quando il blocco finale viene chiuso.

#import <Foundation/Foundation.h>  
int main (int argc, const char * argv[]) { 
    NSAutoreleasePool * pool = [[NSAutoreleasePool alloc] init]; 

    // simple block test - just iterate over some items and 
    // add them to a string 
    NSArray *items = [NSArray arrayWithObjects:@"why ", @"must ", @"this ",nil]; 
    __block NSString *sentence = @""; 
    [items enumerateObjectsUsingBlock:^(id obj, NSUInteger idx, BOOL *stop) 
    { 
     sentence = [sentence stringByAppendingFormat:@"%@",obj]; 
    }]; 
    // crash! 
    NSLog(@"Sentence is %@",sentence); 
    [pool drain]; 
    return 0; 
} 

Qual è il modo corretto/idiomatico per farlo funzionare?

Answer 1

Ok, sono andato via e ho giocato con Xcode per un po ', ed ecco un modello di cosa sta succedendo, che sembra corrispondere a quello che sto vedendo.

Il blocco che ho usato in precedenza non sta facendo nulla di speciale, ma il codice di enumerateObjectsUsingBlock sembra avere una propria NSAutoreleasePool, in modo che sembra essere quello che stava causando dealloc di essere chiamato su oggetti alloc'ed, ma autoreleased all'interno del blocco.

Il seguente codice corrisponde a un comportamento quello che sto vedendo qui sopra:

#import <Foundation/Foundation.h> 
int main (int argc, const char * argv[]) { 
    NSAutoreleasePool * pool = [[NSAutoreleasePool alloc] init]; 

    // simple block test - just iterate over some items and 
    // add them to a string 
    typedef void (^AccArrayBlock)(id obj, int idx, BOOL *stop); 
    // items to 'process' 
    NSArray *items = [NSArray arrayWithObjects:@"why ", @"must ", @"this ",nil]; 
    int idx = 0; 
    BOOL doStop = NO; 
    // make sentence mutable, so we can assign it inside block 
    __block NSString *sentence = @""; 
    // make a similar block to what we'd pass to enumerate... 
    AccArrayBlock myBlock = ^(id obj, int idx, BOOL *stop) 
    { 
     // returns and assigns an autoreleased string object 
     sentence = [sentence stringByAppendingFormat:@"(%d) %@ ",idx,obj]; 
    }; 
    // enumerate items and call block 
    for (NSString *item in items) { 
     // create a pool to clean up any autoreleased objects in loop 
     // remove this line, and the sentence will be valid after loop 
     NSAutoreleasePool *innerPool = [[NSAutoreleasePool alloc] init]; 
     myBlock(item, idx++, &doStop); 
     // drain the pool, autorelease objects from block 
     [innerPool drain]; 
     if (doStop) { 
      break; 
     } 
    } 
    // faults if we drained the pool 
    // Program received signal: “EXC_BAD_ACCESS”. 
    NSLog(@"Sentence is %@",sentence); 
    [pool drain]; 
    return 0; 
} 

Se rimuovo l'oggetto innerPool, quindi il codice funziona come mi aspettavo in origine, e presumibilmente la piscina NSRunLoop finirà per ripulire il vari oggetti NSString.

NOTA: Questo thread è ora il risultato numero 2 Google per 'enumerateObjectsUsingBlock autorelease':

Google 'enumerateObjectsUsingBlock+autorelease'

Il primo risultato conferma questa risposta. Ringrazia tutti.

Answer 2

Ok quindi non sono sicuro al 100% quello che sta succedendo lì, ma nel frattempo funziona se si cambia

NSArray *items = [NSArray arrayWithObjects:@"why ", @"must ", @"this ",nil]; 
NSMutableString *sentence = [[NSMutableString alloc] init]; 
[items enumerateObjectsUsingBlock:^(id obj, NSUInteger idx, BOOL *stop) 
{ 
    [sentence appendFormat:@"%@",obj]; 
}]; 

NSLog(@"Sentence is %@",sentence); 

[sentence release]; sentence = nil; 

aggiornati grazie a @ nacho4d

Answer 3

Come lei ha ricordato, ho il sospetto che questo si arresta in modo anomalo quando viene eseguito il pool autorelease, come probabilmente avviene in enumerateObjectsUsingBlock:. Ciò sarà noioso per aggirare se si dispone di una variabile __block. È possibile usare un NSMutableString, invece, o semplicemente fare questo, che è più pulito in ogni caso:

for (id obj in items) 
{ 
    sentence = [sentence stringByAppendingFormat:@"%@",obj]; 
} 

In alternativa, se si utilizza ARC, il compilatore dovrebbe eliminare il problema per voi.