CLLocationManagerDelegate methods do not run

Question

I am trying to create an application with only a map view that has a "Show user location" button placed on it, which I implement with the help of MapKit. With a press of the button I would also like to get user coordinates and for that I employ CLLocationManagerDelegate methods that I have trouble running. I did set "Privacy - Location When In Use Usage Description" string in plist.

Here is the code of my MapViewController:

import UIKit
import MapKit
import CoreLocation

class MapViewController: UIViewController, CLLocationManagerDelegate {
    // MARK: - Properties
    @IBOutlet weak var mapView: MKMapView!
    let locationManager = CLLocationManager()
    
    var fetchedData = ""
    
    // MARK: - Lifecycle
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
    }
    
    // MARK: - Actions
    // this puts user location on map
    @IBAction func showUserLocation() {
        // check for location tracing permissions first
        let authStatus = CLLocationManager.authorizationStatus()
        
        if authStatus == .notDetermined {
            locationManager.requestWhenInUseAuthorization()
            return
        } else if authStatus == .denied || authStatus == .restricted {
            showLocationServicesDeniedAlert()
            return
        }
        
        let region = MKCoordinateRegion(center: mapView.userLocation.coordinate, latitudinalMeters: 1000, longitudinalMeters: 1000)

        mapView.setRegion(mapView.regionThatFits(region), animated: true)
        
        getCoordinates()
    }
    
    // MARK: - Helper methods
    // handle permission errors
    func showLocationServicesDeniedAlert() {
        let alert = UIAlertController(title: "Paikannuspalvelut pois päältä", message: "Ole hyvä ja salli paikannuspalvelut tälle äpille.", preferredStyle: .alert)
        
        let okAction = UIAlertAction(title: "OK", style: .default, handler: nil)
        alert.addAction(okAction)
        
        present(alert, animated: true, completion: nil)
    }
    
    // get user coordinates
    func getCoordinates() {
        locationManager.delegate = self
        locationManager.desiredAccuracy = kCLLocationAccuracyNearestTenMeters
        locationManager.startUpdatingLocation()
    }
    
}

// MARK: - MKMapViewDelegate
extension MapViewController: MKMapViewDelegate {
    
}

// MARK: - CLLocationManagerDelegate
func locationManager(_ manager: CLLocationManager, didFailWithError error: Error) {
    print("*** Location manager failed with error: \(error.localizedDescription)")
}

func locationManager(_ manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) {
    let newLocation = locations.last!
    print("*** didUpdateLocations \(newLocation)")
}

unfortunately, I dont have anything newer than iphone5. Anyway, I believe locationManager(_:, didUpdateLocations) should be able to print the newLocation in case of a successful run. I just cannot figure out the problem here. — qlep, Jul 24 '20 at 20:04
On the simulator, have you selected the option to mimic a location? — Magnas, Jul 25 '20 at 04:47
Ok. You have iPhone 5, so you are able to develop for iOS 10 and lower. Location without "simulation of location" not working on simulator probably because of lack of GPS-module. Try to "Debug -> Simulate location" on Xcode's menu. — Ilya Muradymov, Jul 25 '20 at 06:08
@Magnas, yes, I do simulate location while running code in simulator yet I cannot see the print from locationManager(_: didUpdateLocation) — qlep, Jul 25 '20 at 13:15

score 0 · Answer 1 · answered Jul 25 '20 at 06:43

0

when app runs make sure Location is not set to "None".

you can find these options on status bar when the simulator is active.

answered Jul 25 '20 at 06:43

Govani Dhruv Vijaykumar

735
7
11

yup, doing that and not seeing the print from location manager – qlep Jul 25 '20 at 13:18

score 0 · Accepted Answer · answered Jul 25 '20 at 17:04

0

Thanks everyone for the input! As soon as I moved CLLocationManagerDelegate methods INSIDE view controller class everything started to work properly.

answered Jul 25 '20 at 17:04

qlep

15
5

score 0 · Answer 3 · answered Jul 25 '20 at 19:21

I gather that you successfully discovered that you need to put these methods inside the view controller. Excellent.

But I might suggest adding the protocol conformance in an extension of your class, not in the main implementation. This keeps your code nicely organized. For example:

import UIKit
import MapKit
import CoreLocation

class MapViewController: UIViewController {
    // MARK: - Properties

    ...
    
    // MARK: - Lifecycle

    override func viewDidLoad() { ... }
    
    // MARK: - Actions

    // this puts user location on map
    @IBAction func showUserLocation() { ... }
    
    // MARK: - Helper methods

    // handle permission errors
    func showLocationServicesDeniedAlert() { ... }
    
    // get user coordinates
    func getCoordinates() { ... } 
}

// MARK: - MKMapViewDelegate

extension MapViewController: MKMapViewDelegate { ... }

// MARK: - CLLocationManagerDelegate

extension MapViewController: CLLocationManagerDelegate {
    func locationManager(_ manager: CLLocationManager, didFailWithError error: Error) {
        print("*** Location manager failed with error: \(error.localizedDescription)")
    }
    
    func locationManager(_ manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) {
        let newLocation = locations.last!
        print("*** didUpdateLocations \(newLocation)")
    }
}

Thank you for your suggestion, @Rob! I'll do that. It's funny how the hint for this solution was right before my eyes in form of that `MKMapViewDelegate` extension stub. — qlep, Jul 26 '20 at 11:37

CLLocationManagerDelegate methods do not run

3 Answers3