3

このブロックを後で実行する非同期メソッドにブロックを渡しています。someMethod:success:failure: に渡す前にブロックをコピーしないと、アプリがクラッシュします。

someMethod:success:failure: に渡す前にブロックをコピーするのではなく、 forwardInvocation: でブロックをコピーする方法はありますか?

フローはsomeMethod:success:failure: -> forwardInvocation: -> httpGet:success:failure です

httpGet:success:failure: HTTP ステータス コードに応じて、成功または失敗のブロックを実行します。

// AppDelegate.h

@interface AppDelegate : UIResponder <UIApplicationDelegate>

@property (strong, nonatomic) id response;
@property (strong, nonatomic) NSError *error;

@end

// AppDelegate.m

@implementation AppDelegate

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
    // The app crashes if the blocks are not copied here!
    [[MyController new] someMethod:[^(NSString *response) {
        self.response = response;
        NSLog(@"response = %@", response);
    } copy] failure:[^(NSError *error) {
        self.error = error;
    } copy]];

    return YES;
}

@end

// MyController.h

@protocol MyControllerProtocol <NSObject>

@optional


- (void)someMethod:(void (^)(NSString *response))success
           failure:(void (^)(NSError *error))failure;

@end

@interface MyController : NSObject <MyControllerProtocol>

@end

// MyController.m

#import "MyController.h"
#import "HTTPClient.h"

@implementation MyController

- (void)forwardInvocation:(NSInvocation *)invocation {
    [invocation retainArguments];

    NSUInteger numberOfArguments = [[invocation methodSignature] numberOfArguments];

    typedef void(^SuccessBlock)(id object);
    typedef void(^FailureBlock)(NSError *error);

    __unsafe_unretained SuccessBlock successBlock1;
    __unsafe_unretained SuccessBlock failureBlock1;
    [invocation getArgument:&successBlock1 atIndex:(numberOfArguments - 2)]; // success block is always the second to last argument (penultimate)
    SuccessBlock successBlock = [successBlock1 copy];
    [invocation getArgument:&failureBlock1 atIndex:(numberOfArguments - 1)]; // failure block is always the last argument
    FailureBlock failureBlock = [failureBlock1 copy];

    NSLog(@"successBlock copy = %@", successBlock);
    NSLog(@"failureBlock copy = %@", failureBlock);

    // Simulates a HTTP request and calls the success block later!
    [HTTPClient httpGet:@"somerequest" success:successBlock failure:failureBlock];
}

- (NSMethodSignature *)methodSignatureForSelector:(SEL)sel {
    NSMethodSignature *methodSignature = [super methodSignatureForSelector:sel];
    return methodSignature;
}

@end


// HTTPClient.h

@interface HTTPClient : NSObject

+ (void)httpGet:(NSString *)path
        success:(void (^)(id object))success
        failure:(void (^)(NSError *error))failure;

@end

// HTTPClient.m

#import "HTTPClient.h"

@implementation HTTPClient

+ (void)httpGet:(NSString *)path
        success:(void (^)(id object))success
        failure:(void (^)(NSError *error))failure
{
    // Invoke the method.
    double delayInSeconds = 2.0;
    dispatch_time_t popTime = dispatch_time(DISPATCH_TIME_NOW, (int64_t)(delayInSeconds * NSEC_PER_SEC));
    dispatch_after(popTime, dispatch_get_main_queue(),
               ^{
                   success(@"foo");
               });
}

@end

完全なソース コードは、https ://github.com/priteshshah1983/BlocksWithNSInvocation にあります。

助けていただけますか？

4

1 に答える 1

3

犯人は行[invocation retainArguments]です。その行をコメントアウトすると、正常に機能します。(呼び出しが保存されたり、非同期で使用されたりすることはないため、この行は必要ありませんでした。)

説明：

何をするかを考えてください-retainArguments。retainオブジェクト ポインター型のすべての引数を呼び出します。そして、呼び出しの割り当てが解除されるとrelease、それらが呼び出されます。

ただし、引数はスタック (コピーされていない) ブロックです。retainそれに影響はありreleaseません（ヒープオブジェクトではないため）。そのため、保持されている場合は何も起こらず、(クラッシュから) ある時点で呼び出しが自動解放されたように見えます (発生するのは完全に正常なことです)。そのため、呼び出しの最終的な解放と割り当て解除は非同期で行われます。呼び出しの割り当てが解除されると、保持されている引数を解放しようとしますが、それまでに有効でなくなったスタック ブロックにメッセージを送ろうとするため、クラッシュが発生します。

PSブロックのコピーforwardInvocation:も不要でした

于 2013-07-13T02:06:56.883 に答える