swift 3 DispatchGroup leave provoque des plantages quand il est appelé dans la fonction de classe auxiliaire

Question

J'utilise DispatchGroup.enter() et leave() pour traiter la fonction asynchrone reverseG d'une classe auxiliaire. Le problème est clair, j'utilise l'objet mainViewController pour appeler dispatchGroup.leave() de mainViewControllers dans la classe helper! Y a-t-il un moyen de le faire?

Le même code fonctionne lorsque reverseG est déclaré dans le contrôleur de la vue principale.

class Geo { 
    var obj = ViewController() 

    static func reverseG(_ coordinates: CLLocation, _ completion: @escaping (CLPlacemark) ->()) { 
     let geoCoder = CLGeocoder() 
     geoCoder.reverseGeocodeLocation(coordinates) { (placemarks, error) in 
      if let error = error { 
       print("error: \(error.localizedDescription)") 
      } 
      if let placemarks = placemarks, placemarks.count > 0 { 
       let placemark = placemarks.first! 
       completion(placemark) // set ViewController's properties 
      } else { 
       print("no data") 
      } 
      obj.dispatchGroup.leave() // ** ERROR ** 
     } 
    } 


} 

Appel de fonction du contrôleur principal de vue

dispatchGroup.enter() 
Geo.reverseG(coordinates, setValues) // completionHandler: setValues 

dispatchGroup.notify(queue: DispatchQueue.main) { 

    // call another function on completion 

} 

Answer 1

Chaque appel leave doit avoir un appel enter associé. Si vous appelez leave sans avoir d'abord appelé enter, il va planter. Le problème ici est que vous appelez enter sur un groupe, mais reverseG appelle leave sur une autre instance de ViewController. Je suggère de passer le DispatchGroup en tant que paramètre à votre méthode reverseG. Ou, mieux, reverseG ne doit pas quitter le groupe, mais plutôt mettre l'appel leave dans le gestionnaire de fin que les appels reserveG.

dispatchGroup.enter() 
Geo.reverseG(coordinates) { placemark in 
    defer { dispatchGroup.leave() } 

    guard let placemark = placemark else { return } 

    // use placemark here, e.g. call `setValues` or whatever 
} 

dispatchGroup.notify(queue: DispatchQueue.main) { 
    // call another function on completion 
} 

Et

class Geo { 
    // var obj = ViewController() 

    static func reverseG(_ coordinates: CLLocation, completion: @escaping (CLPlacemark?) -> Void) { 
     let geoCoder = CLGeocoder() 
     geoCoder.reverseGeocodeLocation(coordinates) { placemarks, error in 
      if let error = error { 
       print("error: \(error.localizedDescription)") 
      } 
      completion(placemarks?.first) 

      // obj.dispatchGroup.leave() // ** ERROR ** 
     } 
    } 

} 

Cela permet de maintenir la logique DispatchGroup à un niveau de l'application, en gardant vos classes moins étroitement couplées (par exemple, le codeur Geo n'a pas besoin de savoir si le contrôleur de vue utilise l'envoi groupes ou non). Franchement, je ne comprends pas très bien pourquoi vous utilisez un groupe d'expédition s'il n'y a qu'un seul appel. Habituellement, vous placez ce que vous appelez dans le gestionnaire d'achèvement, ce qui simplifie encore le code. Vous n'utilisez généralement des groupes que si vous effectuez toute une série d'appels. (Vous avez peut-être simplement simplifié votre extrait de code alors que vous effectuez de nombreux appels.) Dans ce cas, un groupe de répartition peut avoir du sens, mais vous ne devriez pas faire de requêtes de géocodage simultanées, suggérant un modèle complètement différent. tout à fait.

Answer 2

Passé dispatchGroup comme paramètre avec appel de fonction et cela a fonctionné.

Geo.reverseG(coordinates, dispatchGroup, setValues)