Не выполняйте блок, если задача отменена

Question

У меня есть MainViewController ->ThumbnailViewController ->ImageFullScreenViewController. Как следует из их названия, у меня есть главный экран, из которого я иду на экран, который показывает коллекцию изображений и при выборе изображения, я открываю изображение в полноэкранном режиме.

В ThumbnailViewController, загрузить изображения следующим образом

private func getImages() { 
     self.galleryImages.removeAll() 
     for url in urls { 
      let task = NSURLSession.sharedSession().dataTaskWithURL(url) { (data, response, error) in 

       // errors are handled 

       if let image = UIImage(data: data) { 
        self.galleryImages.append(image!) 
       } 
      } 
      task.resume() 
      requests.append(task) 
     } 
    } 

В viewDidLoad() я называю getImages(). В viewWillDisappear() из ThumbnailViewController я отменяю текущие задачи.

override func viewWillDisappear(animated: Bool) { 
    super.viewWillDisappear(animated) 
    if isMovingFromParentViewController() { 
     if requests.count > 0 { 
      for request in requests { 
       if request is NSURLSessionDataTask { 
        let task = request as! NSURLSessionDataTask 
        task.cancel() 
       } 
      } 
     } 
    } 
} 

Проблема заключается в том, когда я открываю ThumbnailViewController и немедленно вернуться, и если я открываю ThumbnailViewController сразу, я могу видеть две копии одного и того же изображения в некоторых случаях (редко, но воспроизводимые).

В ходе расследования я обнаружил, что отмена NSURLSessionDataTask в viewWillDisappear отменяет только задание, но не блок завершения (который является поведением). В некоторых редких случаях блок завершения выполняется для предыдущего NSURLSessionDataTask, тем самым заканчивая смешивание с ответом нового NSURLSessionDataTask.

Как я мог справиться с этим?

Примечание: galleryImages одноэлементно свойство, которое я повторно использовать в ImageFullScreenViewController (UIPageViewController)

Answer 1

Во-первых, вы должны позаботиться о выпуске ThumbnailViewController объекта. Объявите список захвата с weak self, и он, по дополнительным, удалить вашу проблему в 99% случаев (если не удерживать self в некоторых других местах), хотя ваша модель еще не совершенны с этим singletone

private func getImages() { 
    self.galleryImages.removeAll() 
    for url in urls { 
     let task = NSURLSession.sharedSession().dataTaskWithURL(url) { [weak self] (data, response, error) in 

      // errors are handled 
      // replace self. with self?. 

      if let image = UIImage(data: data) { 
       self?.galleryImages.append(image!) 
      } 
     } 
     task.resume() 
     requests.append(task) 
    } 
}