Objective-C crash sur le détruire helper bloc

Il n'y a pas beaucoup d'aller ici, mais je soupçonne que le bloc n'est jamais déplacé vers le tas. Les blocs sont créés sur la pile par défaut. Le compilateur peut souvent trouver le moment de les déplacer en tas, mais la façon dont vous passez celui-ci d'un bloc à l'autre ne le fait probablement jamais.

j'ajouterais un completionCopy = [completion copy] pour le forcer sur le tas. Puis travailler avec completionCopy . Voir réponse de bbum concernant le stockage des blocs dans dictionnaire. Avec ARC, vous n'avez plus besoin d'appeler Block_copy() et Block_release() , mais je pense que vous voulez toujours appeler -copy ici.

hypothèse:

1. doSomethingWithCompletion: crée ExampleBlock.
2. vous démarrez une opération de réseau asynchrone.
3. doSomethingWithCompletion: les retours, et les ExampleBlock est sorti.
4. le fonctionnement asynchrone du réseau se termine, et appelle ExampleBlock .

dans ce cas, le pointeur vers le bloc serait déréférencé après qu'il ait été désalloué. (Peut-être est-ce intermittent si l'on se base sur le fait que le bassin d'autorelease s'est drainé ou si d'autres zones de mémoire à proximité ont été libérées.)

3 solutions possibles:

1. Stocker le bloc dans une propriété

stocker le bloc dans une propriété:

@property (nonatomic, copy) returnType (^exampleBlock)(parameterTypes);

puis en code,

self.exampleBlock = …

Un problème avec cette approche est que vous ne pouvez jamais avoir un exampleBlock .

2. Stocker le bloc dans un tableau

Pour contourner ce problème, vous pouvez stocker les blocs dans une collection (comme NSMutableArray ):

@property (nonatomic, strong) NSMutableArray *blockArray;

puis en code:

self.blockArray = [NSMutableArray array];

// Later on…
[self.blockArray addObject:exampleBlock];

vous pouvez enlever le bloc du tableau quand il est correct de le désallouer.

3. Travailler autour du problème de stockage en passant simplement le bloc autour de

au lieu de gérer le stockage et la destruction de vos blocs, remaniez votre code de façon à ce que exampleBlock soit transmis entre les différentes méthodes jusqu'à la fin de votre opération.

alternativement, vous pouvez utiliser NSBlockOperation pour le code asynchrone, et définir son completionBlock pour la réponse-code fini, et l'ajouter à une Nsopérationqueue.

je pense que le completion est libéré dans votre appel async qui pourrait être à l'origine du crash.

je soupçonne, le problème n'est pas dans votre code mais ailleurs.

un problème possible est celui-ci:

IFF il y a des objets UIKit qui sont capturés dans le bloc completion vous avez probablement un bug subtil quand le bloc est exécuté sur un thread non-principal et ce bloc garde le dernier forte référence à ces objets UIKit:

quand le bloc completion se termine, il est Bloc obtenir désalloué et avec cela, toutes les variables importées sont " détruites "ce qui signifie qu'en cas de pointeurs readable, elles reçoivent un message release . Si c'était la dernière référence forte, l'objet capturé est désallocalisé ce qui se produit dans un thread non-principal - et cela peut être fatal pour les objets UIKit.

Je ne vois rien de mal avec le code posté et je pense que le bug est ailleurs.

aussi blocs de nidification semble inutile et complique la gestion de la mémoire et rend probablement plus difficile de trouver la cause de l'accident.

pourquoi ne pas commencer par déplacer le code de votre ExampleBlock directement dans le bloc completion ?

qu'en est-il de cette solution: si vous appellerez plus tard un bloc qui n'est pas dans la portée courante, alors vous devriez appeler une copie dessus pour déplacer ce bloc dans le tas à partir de la pile

- (void)doSomethingWithCompletion:(void (^)())completion {
    void (^ExampleBlock)(NSString *) = [^{
        NSNotification *notification = [NSNotification notificationWithName:kExampleNotificationName object:nil userInfo:nil];
        [[NSNotificationCenter defaultCenter] postNotification:notification];

        if (completion) {
            completion();
        }
    } copy];

    // Async network call that calls ExampleBlock on either success or failure below...
}