iphone - 为什么在iOS上启动了几个线程后，我的线程似乎失败了？

Question

我在- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath委托(delegate)调用中有以下代码:

dispatch_async( dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{

    AVPlayerItem *playerItem = [AVPlayerItem playerItemWithURL:[webUrls objectAtIndex:indexPath.row]];
    CMTime timeduration = playerItem.duration;
    float seconds = CMTimeGetSeconds(timeduration);
    NSString *duration = [NSString stringWithFormat:@"%f", seconds];

    dispatch_async( dispatch_get_main_queue(), ^{

        UITableViewCell *updatecell = [tblView cellForRowAtIndexPath:indexPath];
        updatecell.detailTextLabel.text = duration;
        [updatecell setNeedsLayout];
    });
});

每个单元在后台缓慢地将seconds加载到该单元上的updatecell.detailTextLabel.text中。问题是我滚动后，加载了大约3或4个单元后，其余的仅在detailTextLabel中迅速显示为0，并且不加载。

有任何想法为什么会这样吗？我没有正确进行线程吗？

Answer 1

有两个想法:

许多服务器对它们从给定客户端接受的并发请求施加了约束。我建议您使用NSOperationQueue将对服务器的并发请求限制为4或5，而不要使用调度队列。

您可能使问题变得更糟，因为如果您向下滚动表格 View 然后又向上备份，那么当您重新显示前几个单元格时，您将重新下载AVPlayerItem并尝试进行其他并发操作您服务器的请求。您确实应该保存以前下载的结果，以消除重复请求相同数据的需要。

您当前未在尝试更新UI之前检查是否仅看到了刚刚下载的单元格。您确实应该检查一下。

因此，我可能建议以下内容:

在 View Controller 的viewDidLoad中，创建我们将用于下载的NSOperationQueue。还指定您的服务器将允许多少个并发操作:

downloadQueue = [[NSOperationQueue alloc] init];
downloadQueue.maxConcurrentOperationCount = 4; // replace this with an appropriate value for your server

之前，您有一个数组webUrls，它是一个NSURL对象的数组。在下面的第4点，我们将讨论退出该数组，并创建一个新的行对象数组。但是在我们可以做到这一点之前，我们应该创建这个新的RowData对象。

每个行对象不仅具有webURL，而且还具有其他内容，例如durationText甚至是AVPlayerItem本身。 (通过保留这些其他对象属性，当单元格滚动回 View 时，我们不需要重新下载数据。)因此，此新类的公共(public)接口(interface)可能类似于:

//
//  RowData.h
//

#import <Foundation/Foundation.h>

@class AVPlayerItem;

@interface RowData : NSObject

@property (nonatomic, strong) NSURL *webURL;
@property (nonatomic, strong) NSString *durationText;
@property (nonatomic, strong) AVPlayerItem *playerItem;
@property (nonatomic, getter = isDownloaded, readonly) BOOL downloaded;
@property (nonatomic, getter = isDownloading, readonly) BOOL downloading;

- (void)downloadInQueue:(NSOperationQueue *)queue completion:(void (^)(BOOL success))block;
- (void)cancelDownload;

@end

顺便说一句，我并不为类名RowData着迷。这有点模棱两可。但是我对您的模型数据的性质了解不足，无法建议一个更好的名称。无论您认为合适如何，都可以随时致电此类(class)。

您的新RowData类可以具有一个名为downloadInQueue的实例方法，该方法执行下载，适本地设置durationText等。通过在此处移动下载逻辑，我们成功地将cellForRowAtIndexPath与下载中涉及的一些细节相隔离。同样重要的是，虽然此downloadInQueue方法不会更新用户界面本身，但它具有completion提供的cellForRowAtIndexPath块(在下面的第5点处演示)，因此，此downloadInQueue方法不必担心UI注意事项。无论如何，downloadInQueue的实现可能看起来像:

//
//  RowData.m
//

#import "RowData.h"
#import <AVFoundation/AVFoundation.h>

@interface RowData ()

@property (nonatomic, getter = isDownloaded) BOOL downloaded;
@property (nonatomic, getter = isDownloading) BOOL downloading;
@property (nonatomic, weak) NSOperation *operation;

@end

@implementation RowData

- (void)downloadInQueue:(NSOperationQueue *)queue completion:(void (^)(BOOL success))completion
{
    if (!self.isDownloading)
    {
        self.downloading = YES;

        NSOperation *currentOperation = [NSBlockOperation blockOperationWithBlock:^{
            BOOL success = NO;

            self.playerItem = [AVPlayerItem playerItemWithURL:self.webURL];
            if (self.playerItem)
            {
                success = YES;
                CMTime timeduration = self.playerItem.duration;
                float seconds = CMTimeGetSeconds(timeduration);
                self.durationText = [NSString stringWithFormat:@"%f", seconds];
            }
            self.downloading = NO;
            self.downloaded = YES;

            [[NSOperationQueue mainQueue] addOperationWithBlock:^{
                completion(success);
            }];
        }];

        [queue addOperation:currentOperation];
        self.operation = currentOperation;
    }
}

- (void)cancelDownload
{
    if ([self isDownloading] && self.operation)
    {
        self.downloading = NO;
        [self.operation cancel];
    }
}

@end

在主 View Controller 中，而不是创建旧的webUrls数组，而是创建这些RowData对象的新数组，例如objects。当然，为每个webURL对象设置RowData属性。 (再次，我对objects的模棱两可的名称并不感到疯狂，但是我对您的应用程序不够了解，无法提出更具体的建议。您可以随意调用此名称。但是下面的代码将使用objects。)

最后，修改您的cellForRowAtIndexPath以使用此新的RowData对象及其downloadInQueue方法。另外，请注意，completion块检查以确保该单元格仍然可见:

RowData *rowData = self.objects[indexPath.row];

if ([rowData isDownloaded])
{
    cell.detailTextLabel.text = rowData.durationText;
}
else
{
    cell.detailTextLabel.text = @"..."; // you really should initialize this so we show something during download or remove anything previously there

    [rowData downloadInQueue:self.downloadQueue completion:^(BOOL success) {
        // note, if you wanted to change your behavior based upon whether the 
        // download was successful or not, just use the `success` variable

        UITableViewCell *updateCell = [tblView cellForRowAtIndexPath:indexPath];

        // by the way, make sure the cell is still on screen

        if (updateCell)
        {
            updateCell.detailTextLabel.text = rowData.durationText;
            [updateCell setNeedsLayout];
        }
    }];
}

如果使用iOS 6，如果要在单元格滚动离开屏幕时取消待处理的下载，则可以使用didEndDisplayingCell协议(protocol)的UITableViewDelegate方法。

- (void)tableView:(UITableView *)tableView didEndDisplayingCell:(UITableViewCell *)cell forRowAtIndexPath:(NSIndexPath *)indexPath
{
    RowData *rowData = self.objects[indexPath.row];

    if ([rowData isDownloading])
        [rowData cancelDownload];
}

如果支持iOS的早期版本，则必须使用UIScrollViewDelegate协议(protocol)方法(例如 scrollViewDidScroll )来手动确定哪些单元格已滚动出屏幕(例如indexPathsForVisibleRows中不包括这些单元格)，但是想法是相同的。

顺便说一句，在上面的示例RowData中，我保存了AVPlayerItem。仅当以后需要AVPlayerItem时，才应该这样做。我们已经保存了duration，它满足了UITableViewCell所需的所有功能，但是我想您以后可能想对AVPlayerItem做些事情，所以我也保存了它。但是，如果以后不再需要该AVPlayerItem，则不要将其保存在RowData对象中。另外，我不知道它们的大小，但是您可能想编写一个didReceiveMemoryWarning，它将迭代您的objects并将每个项目的playerItem对象设置为nil。