forwardInvocation中的奇怪“僵尸”:+ getArgument:atIndex方法

时间:2013-04-16 18:51:25

标签: objective-c automatic-ref-counting objective-c-blocks nszombie nsinvocation

以下是我的代码的一部分:

- (void)viewDidLoad
{
    [super viewDidLoad];

    CGRect frame = [[UIScreen mainScreen] bounds];
    _webView = [[UIWebView alloc] initWithFrame:frame];
    [_webView setHidden:NO];
    [self.view addSubview:_webView];

    _vk = [[DPVkontakteCommunicator alloc] initWithWebView:_webView];

    DPVkontakteUserAccount *user;
    NSString *accessToken = [[NSUserDefaults standardUserDefaults]
                                             objectForKey:@"accessToken"];
    NSInteger userId = [[[NSUserDefaults standardUserDefaults]
                                         objectForKey:@"userId"] integerValue];
    user = [[DPVkontakteUserAccount alloc]
                                    initUserAccountWithAccessToken:accessToken
                                                            userId:userId];

    NSLog(@"%@", user);

    [user setSuccessBlock:^(NSDictionary *dictionary)
    {
        NSLog(@"%@", dictionary);
    }];

    NSDictionary *options = @{@"uid":@"1"};
    //    [user usersGetWithCustomOptions:@{@"uid":@"1"}]; // Zombie
    [user usersGetWithCustomOptions:options]; // Not zombie

    //    __block NSDictionary *options = @{};
    //
    //    [_vk startOnCancelBlock:^{
    //        NSLog(@"Cancel");
    //    } onErrorBlock:^(NSError *error) {
    //        NSLog(@"Error: %@", error);
    //    } onSuccessBlock:^(DPVkontakteUserAccount *account) {
    //        NSLog(@"account:%@", account);
    //
    //        [account setSuccessBlock:^(NSDictionary *dictionary)
    //        {
    //            NSLog(@"%@", dictionary);
    //        }];
    //
    //        [account docsGetUploadServerWithCustomOptions:options];
    //    }];
}

以下是处理userGetWithCustomOptions:method:

的部分
- (void)forwardInvocation:(NSInvocation *)anInvocation
{
    NSString *methodName = NSStringFromSelector([anInvocation selector]);
    NSDictionary *options;

    [anInvocation getArgument:&options
                      atIndex:2];

    NSArray *parts = [self parseMethodName:methodName];
    NSString *vkURLMethodSignature = [NSString stringWithFormat:@"%@%@.%@",
                                                                kVKONTAKTE_API_URL,
                                                                parts[0],
                                                                parts[1]];
    // appending params to URL
    NSMutableString *fullRequestURL = [vkURLMethodSignature mutableCopy];

    [fullRequestURL appendString:@"?"];

    [options enumerateKeysAndObjectsUsingBlock:^(id key, id obj, BOOL *stop)
    {
        [fullRequestURL appendFormat:@"%@=%@&", key, [obj encodeURL]];
    }];

    [fullRequestURL appendFormat:@"access_token=%@", _accessToken];

    // performing HTTP GET request to vkURLMethodSignature URL
    NSURL *url = [NSURL URLWithString:fullRequestURL];
    NSURLRequest *urlRequest = [NSURLRequest requestWithURL:url];
    AFJSONRequestOperation *operation;
    operation = [AFJSONRequestOperation
            JSONRequestOperationWithRequest:urlRequest
                                    success:^(NSURLRequest *request,
                                              NSHTTPURLResponse *response,
                                              id JSON)
                                    {
                                        _successBlock(JSON);
                                    }
                                    failure:^(NSURLRequest *request,
                                              NSHTTPURLResponse *response,
                                              NSError *error,
                                              id JSON)
                                    {
                                        _errorBlock(error);
                                    }];

    [operation start];
}

问题是,当我使用“选项”变量时 - 它工作正常,但是当使用直接值 - 它失败时,应用程序崩溃。使用Profile我发现方法调用指向解除分配的对象。

为什么会这样? 没有其他代码可以提供帮助。

ViewController.m代码:https://gist.github.com/AndrewShmig/5398546

DPVkontakteUserAccount.m:https://gist.github.com/AndrewShmig/5398557

1 个答案:

答案 0 :(得分:3)

问题是getArgument:的参数是void *类型。你正在传递&value NSDictionary * __strong *(指向强引用的指针)。强制转换是有效的,因为可以在没有任何警告的情况下为void *分配任何非对象指针。

当你向函数传递一个“强指针”时,这意味着函数应该指向一个“强引用”,当函数退出时,它应该保留指针指向“强”的事实参考”。这意味着如果函数更改引用(由指针指向),它必须首先释放先前的值,然后保留新值。

但是,getArgument:atIndex:void *参数做了什么?它与指向的东西无关,只是将值复制到指向的内存中。因此,它没有做任何保留和释放的东西。基本上,它会在您的value变量中执行一个普通的ARC前非保留赋值。

那为什么会崩溃?发生的事情是value最初是nil,然后在getArgument:atIndex:内,它会将新值分配给它,但它不会保留它。但是,ARC假定它已被保留,因为value是强引用。因此,在范围的最后,ARC发布它。这是一个过度释放,因为它从未保留过。

解决方法是不将“指向强大的指针”传递给getArgument:,因为该方法对“强”一无所知。相反,将“指向unsafe_unretained”或“指向void”的指针传递给它,然后将其转换为强引用:

NSDictionary * __unsafe_unretained temp;
[anInvocation getArgument:&temp atIndex:2];
NSDictionary *options = temp; // or you can just use temp directly if careful

或者:

void *temp;
[anInvocation getArgument:&temp atIndex:2];
NSDictionary *options = (__bridge NSDictionary *)temp;