SWIFT - LocationManager looping through multiple times?

Question

I have a locationManager function to grab the users current location and posting the name of the city and state. I have a print statement so I can check in my console if eve

Answer 1

I'd first suggest a few things:

1. Call stopUpdatingLocation before you perform reverseGeocodeLocation.

   You are calling stopUpdatingLocation inside the reverseGeocodeLocation completion handler closure. The problem is that this runs asynchronously, and thus didUpdateLocations may receive additional location updates in the intervening period. And often, when you first start location services, you'll get a number of updates, often with increasing accuracy (e.g. horizontalAccuracy values that are smaller and smaller). If you turn off location services before initiating asynchronous geocode request, you'll minimize this issue.

2. You can also add add a distanceFilter in viewDidLoad, which will minimize redundant calls to the delegate method:

   locationManager.distanceFilter = 1000
   

3. You can use your own state variable that checks to see if the reverse geocode process has been initiated. For example:

   private var didPerformGeocode = false
   
   func locationManager(manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) {
       // if we don't have a valid location, exit
   
       guard let location = locations.first where location.horizontalAccuracy >= 0 else { return }
   
       // or if we have already searched, return
   
       guard !didPerformGeocode else { return }
   
       // otherwise, update state variable, stop location services and start geocode
   
       didPerformGeocode = true
       locationManager.stopUpdatingLocation()
   
       CLGeocoder().reverseGeocodeLocation(location) { placemarks, error in
           let placemark = placemarks?.first
   
           // if there's an error or no placemark, then exit
   
           guard error == nil && placemark != nil else {
               print(error)
               return
           }
   
           let city = placemark?.locality ?? ""
           let state = placemark?.administrativeArea ?? ""
   
           self.navigationBar.title = ("\(city), \(state)")
           self.usersLocation = ("\(city), \(state)")
           print(self.usersLocation)
           self.refreshPosts()
       }
   }