导致UITableView问题的多个NSURLSessions

时间:2015-06-15 00:59:20

标签: ios objective-c uitableview nsurlsession

我在这里碰到了一个奇怪的问题。我的一个NSURLSessions负责获取我存储的餐馆信息的信息(餐馆名称,餐馆的徽标URL等),然后第二个NSURLSession负责使用餐馆的徽标URL检索特定图像并为每个UITableView的单元格设置它。

但问题是,我的UITableView有时根本不加载任何内容,因此单元格为空,但有时我在NSURLSessions中添加了额外的[_tableView reload]。 fetchPosts方法中的完成块,它可以工作,但如果我重新运行它,单元格将再次停止显示任何内容。有些事情肯定是错的。看看下面的代码:

#import "MainViewController.h"
#import "SWRevealViewController.h"
#import "RestaurantNameViewCell.h"
#import "RestaurantList.h"

@interface MainViewController ()

@end

@implementation MainViewController

- (void)viewDidLoad
{
    [super viewDidLoad];


    //List of restaurants needed to load home page
    _restaurantInformationArray = [[NSMutableArray alloc] init];


    self.tableView.dataSource = self;
    self.tableView.delegate = self;


    //setup for sidebar
    SWRevealViewController *revealViewController = self.revealViewController;
    if ( revealViewController )
    {
        [self.sidebarButton setTarget: self.revealViewController];
        [self.sidebarButton setAction: @selector( revealToggle: )];
        [self.view addGestureRecognizer:self.revealViewController.panGestureRecognizer];
    }



    //Get list of restaurants and their image URLs
    [self fetchPosts];

}

- (void)didReceiveMemoryWarning
{
    [super didReceiveMemoryWarning];
    // Dispose of any resources that can be recreated.
}


-(NSInteger)numberOfSectionsInTableView:(UITableView *)tableView {
    return 1;
}

-(NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
    return [_restaurantInformationArray count];
}

-(UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
    RestaurantNameViewCell *cell = (RestaurantNameViewCell *)[_tableView dequeueReusableCellWithIdentifier:@"restaurantName" forIndexPath:indexPath];

    RestaurantList *currentRestaurant = [_restaurantInformationArray objectAtIndex:indexPath.row];


    cell.restaurantName.text = currentRestaurant.name;
    cell.imageAddress = currentRestaurant.imageURL;

    cell.restaurantClicked = [[UITapGestureRecognizer alloc] initWithTarget:self action:@selector(tapDetected:)];
    cell.restaurantClicked.numberOfTapsRequired = 1;
    cell.restaurantLogo.userInteractionEnabled = YES;
    [cell.restaurantLogo addGestureRecognizer:cell.restaurantClicked];
    cell.restaurantLogo.tag = indexPath.row;

    //Add restaurant logo image:
    NSString *URL = [NSString stringWithFormat:@"http://private.com/images/%@.png",cell.imageAddress];
    NSURL *url = [NSURL URLWithString:URL];

    NSURLSessionDownloadTask *downloadLogo = [[NSURLSession sharedSession]downloadTaskWithURL:url completionHandler:^(NSURL *location, NSURLResponse *response, NSError *error) {
        UIImage *downloadedImage = [UIImage imageWithData:[NSData dataWithContentsOfURL:location]];

        cell.restaurantLogo.image = downloadedImage;
    }];
    [downloadLogo resume];

    return cell;
}



-(void)fetchPosts {
    NSString *address = @"http://localhost/xampp/restaurants.php";
    NSURL *url = [NSURL URLWithString:address];

    NSURLSessionDataTask *downloadRestaurants = [[NSURLSession sharedSession]dataTaskWithURL:url completionHandler:^(NSData *data, NSURLResponse *response, NSError *error) {

        NSError *someError;
        NSArray *restaurantInfo = [NSJSONSerialization JSONObjectWithData:data options:kNilOptions error:&someError];


         for(NSDictionary *dict in restaurantInfo) {
             RestaurantList *newRestaurant = [[RestaurantList alloc]init];

             newRestaurant.name = [dict valueForKey:@"name"];
             newRestaurant.imageURL = [dict valueForKey:@"image"];

             [_restaurantInformationArray addObject:newRestaurant];


             //Refresh table view to make sure the cells have info AFTER the above stuff is done
             [_tableView reloadData];
         }
    }];

    [downloadRestaurants resume];
}


@end

这可能是我犯的一个非常愚蠢的错误,但我不确定如何纠正这个问题。我是iOS开发的新手,所以非常感谢一些指导:)

1 个答案:

答案 0 :(得分:3)

除了假设您的网络请求没有错误(如果存在网络错误,您至少应该记录),则存在线程问题。

您的NSURLSession回调可能在后台线程上运行。这使得调用UIKit(aka - [_tableView reloadData])变得不安全。 UIKit不是线程安全的。这意味着从另一个线程调用任何UIKit的API会产生非确定性行为。你想在主线程上运行那段代码:

dispatch_async(dispatch_get_main_queue(), ^{
    [_tableView reloadData];
});

同样用于获取图像。由于表格视图单元格重用可能导致在滚动时显示错误的图像,因此稍微复杂一些。这是因为当用户滚动时,同一个单元实例用于数组中的多个值。当任何这些回调触发时,它将替换该单元格中出现的任何图像。重现这一点的一般步骤如下:

  1. TableView请求5个单元格
  2. MainViewController请求5张图片(每个单元格一张)
  3. 用户向下滚动一个单元格
  4. 第一个单元格被重用为第6个单元格。
  5. MainViewController请求第6个单元格的另一个图像。
  6. 检索第6张图片,触发回调,将第一张图片的图片设置为图片#6。
  7. 检索第一张图片,触发回调,第一张单元格的图片设置为图片#1(不正确)。
  8. 在尝试将图像分配给单元格之前,您需要确保单元格显示正确的单元格。如果您不想在单元格中实现图像提取的逻辑,则可以使用SDWebImage代替。使用SDWebImage的[UIImageView sd_setImageWithURL:]是线程安全的(它将set the image on the main thread)。

    附注:

    • 您只需在_restaurantInformationArray所有更改中重新加载数据,而不是每次都在for循环中。