15

所以我在主线程上创建我的下载

NSURLRequest *request = [NSURLRequest requestWithURL:download.URL];
NSURLSessionDownloadTask *downloadTask = [self.downloadSession downloadTaskWithRequest:request];
[downloadTask resume];

并将与下载关联的 NSManagedContextID 添加到 NSMutableDictionary,以便稍后在委托回调中检索它

[self.downloads setObject:[download objectID] forKey:[NSNumber numberWithInteger:downloadTask.taskIdentifier]];

我的self.downloadSession上面是这样配置的

- (NSURLSession *)backgroundSession
{
static NSURLSession *session = nil;
static dispatch_once_t onceToken;
dispatch_once(&onceToken, ^{
    NSURLSessionConfiguration *configuration = [NSURLSessionConfiguration backgroundSessionConfiguration:@"com.test.backgroundSession"];
    configuration.discretionary = YES;
    session = [NSURLSession sessionWithConfiguration:configuration delegate:self delegateQueue:nil];
});
return session;
}

我的问题是委托回调似乎是在不同的线程上调用的

- (void)URLSession:(NSURLSession *)session downloadTask:(NSURLSessionDownloadTask *)downloadTask didWriteData:(int64_t)bytesWritten totalBytesWritten:(int64_t)totalBytesWritten totalBytesExpectedToWrite:(int64_t)totalBytesExpectedToWrite
{   
    NSManagedObjectID *downloadID = [self.downloads objectForKey:[NSNumber numberWithInteger:downloadTask.taskIdentifier]];

    double progress = (double)totalBytesWritten / (double)totalBytesExpectedToWrite;

    NSDictionary *userInfo = [NSDictionary dictionaryWithObjectsAndKeys:downloadID,@"download",[NSNumber numberWithDouble:progress],@"progress", nil];

    [[NSNotificationCenter defaultCenter] postNotificationName:@"DownloadProgress" object:nil userInfo:userInfo];

} 

因此,当我访问 self.downloads 以获取正确的 objectID 时,我实际上是从与创建它的线程不同的线程访问 NSMutableDictionary,并且我相信 NSMutableDictionary 不是线程安全的。那么最好的解决方案是什么,我可以使用这样的东西

session = [NSURLSession sessionWithConfiguration:configuration delegate:self delegateQueue:[NSOperationQueue mainQueue]];

声明会话时,将委托队列设置为 mainQueue,这会导致在主线程上调用所有委托,但如果可能,我想将所有回调保留在后台线程上

4

2 回答 2

10

在您的示例中,这不是问题,因为您的字典已移交给通知系统,并且不再被操作队列线程使用。只有当一个对象可能同时从多个线程访问时,线程安全才是一个问题。

如果您的 dict 是 iVar,您应该这样做:

像这样创建自己的队列

myQueue = [[NSOperationQueue alloc] init];
// This creates basically a serial queue, since there is just on operation running at any time.
[myQueue setMaxConcurrentOperationCount:1];

然后在此队列上安排每次访问您的字典,例如:

[myQueue addOperationWithBlock:^
{
    // Access your dictionary 
}];

当然,为您的 URLSesson 委托使用这个队列:

session = [NSURLSession sessionWithConfiguration:configuration delegate:self delegateQueue:myQueue];

由于这个队列被设置为一个串行队列,所以总是只有一个线程在后台访问字典。

使用 dict 信息计算某些内容时要小心。您也必须在该队列上执行此操作。但是,您可以将计算结果放在任何其他队列/线程上,例如在主线程上更新 UI。

[myQueue addOperationWithBlock:^
{
    // Calculate with your dictionary
    // Maybe the progress calcualtion
    NSString* progress = [self calculateProgress: iVarDict];
    dispatch_async(dispatch_get_main_queue(), ^
    {
       // use progress to update UI 
    });
}];

我认为发布通知不必使用该模式,因为系统会正确处理线程。但为了保存,你应该检查一下。

于 2013-09-22T11:01:31.447 回答
0

您可以使用 GCD 串行队列来确保只有一个委托同时执行。

您可以将队列声明为类的实例变量并在 init 方法中对其进行初始化,如下所示:

dispatch_queue_t delegateQueue;

...

delegateQueue = dispatch_queue_create("com.yourcompany.mydelegatequeue", 0x0);

在您的委托方法中,只需使其在此队列中执行:

- (void)URLSession:(NSURLSession *)session downloadTask:(NSURLSessionDownloadTask *)downloadTask didWriteData:(int64_t)bytesWritten totalBytesWritten:(int64_t)totalBytesWritten totalBytesExpectedToWrite:(int64_t)totalBytesExpectedToWrite
{   
    dispatch_sync(delegateQueue, ^{
    NSManagedObjectID *downloadID = [self.downloads objectForKey:[NSNumber numberWithInteger:downloadTask.taskIdentifier]];

    double progress = (double)totalBytesWritten / (double)totalBytesExpectedToWrite;

    NSDictionary *userInfo = [NSDictionary dictionaryWithObjectsAndKeys:downloadID,@"download",[NSNumber numberWithDouble:progress],@"progress", nil];

    [[NSNotificationCenter defaultCenter] postNotificationName:@"DownloadProgress" object:nil userInfo:userInfo];
});

} 

这样,虽然每个委托都在其线程中被调用,但一次只有他们访问 self.downloads,您可以将它们保存在单独的线程中。

于 2013-09-22T17:54:01.763 回答