N'exécute pas le bloc si la tâche est annulée

Question

J'ai MainViewController ->ThumbnailViewController ->ImageFullScreenViewController. Comme leur nom l'indique, j'ai un écran principal à partir duquel je vais sur un écran qui montre une collection d'images et en sélectionnant une image, j'ouvre l'image en plein écran.

En ThumbnailViewController, je télécharger des images comme suit

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) 
     } 
    } 

En viewDidLoad() j'appelle getImages(). Dans viewWillDisappear() de ThumbnailViewController, j'annule les tâches en cours.

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() 
       } 
      } 
     } 
    } 
} 

Le problème est, quand j'ouvre ThumbnailViewController et immédiatement revenir en arrière et si j'ouvre ThumbnailViewController immédiatement, je peux voir deux exemplaires de même image dans certains cas (rarement, mais reproductible).

Lors de l'enquête, j'ai constaté que l'annulation de NSURLSessionDataTask dans viewWillDisappear annule la tâche uniquement, mais pas le bloc d'achèvement (qui est le comportement). Dans de rares cas, le bloc d'achèvement est exécuté pour le précédent NSURLSessionDataTask, terminant ainsi le mélange avec la réponse du nouveau NSURLSessionDataTask.

Comment puis-je gérer cela?

Note: galleryImages est une propriété singleton que je réutilisez dans ImageFullScreenViewController (UIPageViewController)

Answer 1

Tout d'abord, vous devez prendre soin de libération objet ThumbnailViewController. Déclarer la liste de capture avec weak self, et il sera, par plus, retirez votre question dans le cas de 99% (si vous ne retenez pas self dans d'autres endroits), bien que votre modèle toujours pas parfait avec cette 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) 
    } 
}