ios - 从数组访问 NSNumber 时崩溃

Question

我遇到了一个奇怪的崩溃，其中一个 NSNumber 的实例似乎被释放了，尽管它仍然存在于数组中。我创建了一个系统来从远程服务器下载多个文件，并有一个 block 来指示进度(实际上是平均进度)。并且进度的计算产生了崩溃。崩溃不一致，并且“通常”发生在任何时候都可能发生。 [_NSProgressFractionTuple floatValue]:无法识别的选择器发送到实例 0x17042ab80 让我相信 NSNumber 以某种方式被释放，但我看不出这是怎么可能的。

给出完整的方法代码:

- (void)downloadFilesFromURLs:(NSArray<NSString *> *)urlPaths withProgress:(void (^)(float progress))progressBlock completion:(void (^)(NSError *error))completionBlock {
    NSMutableArray *toLoad = [[NSMutableArray alloc] init];
    for(NSString *path in urlPaths) {
        if([self fileForURL:path] == nil) {
            [toLoad addObject:path];
        }
    }

    NSInteger itemsToLoad = toLoad.count;

    if(itemsToLoad <= 0) {
        if(completionBlock) {
            completionBlock(nil);
        }
        return;
    }



    // Set progresses to zero
    __block NSMutableArray *progresses = [[NSMutableArray alloc] init];
    for(int i=0; i<itemsToLoad; i++) [progresses addObject:[[NSNumber alloc] initWithFloat:.0f]];

    __block NSInteger requestsOut = itemsToLoad;
    __block NSError *latestError = nil;
    for(int i=0; i<itemsToLoad; i++) {
        NSInteger index = i;
        [self downloadFileFromURL:toLoad[index] named:nil withProgress:^(float progress) {
            progresses[index] = [[NSNumber alloc] initWithFloat:progress];
            if(progressBlock) {
                float overallProgress = .0f;
                for(NSNumber *number in [progresses copy]) {
                    overallProgress += number.floatValue;
                }
                progressBlock(overallProgress/itemsToLoad);
            }
        } completion:^(NSString *filePath, NSError *error) {
            if(error) latestError = error;
            requestsOut -= 1;
            if(requestsOut <= 0) {
                if(completionBlock) {
                    completionBlock(latestError);
                }
            }
        }];
    }
}

代码解释:

所以这个方法接受一个 URL 数组。然后它检查一些文件是否已经下载并创建一个新数组，其中只包含需要下载的 URL。如果所有文件都存在或未提供 URL，则调用完成并中断操作。

接下来我创建一个可变数组并用 NSNumber 实例填充它，所有实例都具有零值。我记得将发出多少请求，并为错误创建一个占位符。我遍历所有 URL 并初始化请求，其中每个请求都将报告进度和完成情况，并且这两者都在主线程上。

所以在进度 block 中，我访问进度数组以通过索引分配新值。然后我计算平均进度并将总体进度报告给输入 block 。

请求完成会减少请求计数器的数量，当该计数器降为零时，将调用输入完成。

情况:

一切按预期工作，整个过程是正确的。给定的值都是有效的，所有文件都在服务器上并且可以访问。当应用程序没有崩溃时，一切正常。

但是当它崩溃时它崩溃了

                for(NSNumber *number in [progresses copy]) {
                    overallProgress += number.floatValue;
                }

并且崩溃是随机的，但无论如何 number.floatValue 似乎正在访问它不应该访问的内存。

我现在有一个解决方案，我将 progresses 数组替换为纯 C 指针 float *progresses = malloc(sizeof(float)*itemsToLoad); 释放完成。它似乎有效但仍然，我在这里错过了什么？带有 NSNumbers 的数组在这里不起作用的原因可能是什么？

一些附加信息:

• 内存还可以，这是直接写入文件，即使没有，整体文件大小也相对较小
• 磁盘空间还可以
• 我使用的是 @(progress) 语法，但将其更改为显式分配，希望能解决这个问题
• progresses不需要__block，我加了它以防万一
• 在调用所有进度之前不会调用完成，即使调用了我也没有理由让应用程序崩溃

谢谢!

Answer 1

NSMutableArray 不是线程安全的。因此，即使没有明确的内存管理问题，如果 NSMutableArray 被两个不同的线程同时访问，也会发生坏事。我相信在串行队列中分派(dispatch) withProgress block 可以解决问题。