(Xcode)位置情報取得を許可した時、自動でメッセージを表示させたい。

Question

前提・実現したいこと

こんにちは。
位置情報を使うアプリを製作中です。
位置情報取得の許可伺いをポップアップさせ、「許可」を押したら、自動でメッセージが表示されるようにしたいです。
アドバイス頂けますと、幸いです。

発生している問題・エラーメッセージ

位置情報取得「許可」を押した際、メッセージを表示させる処理が、呼ばれません。

該当のソースコード

import UIKit
import CoreLocation

class ViewController: UIViewController, CLLocationManagerDelegate {

    var locationManager = CLLocationManager()
    var status = CLLocationManager.authorizationStatus()

    @IBAction func button1(_ sender: Any) {
        switch status {
        case .notDetermined:
            locationManager.requestWhenInUseAuthorization()
　　　　　    //位置情報取得許可伺いのポップアップは成功しました。
            break
        default:
            break
        }
    }

    func locationManager(_ manager: CLLocationManager,
                             didChangeAuthorization status: CLAuthorizationStatus) {
        
        var status = CLLocationManager.authorizationStatus()
        
        switch status {
        case .authorizedWhenInUse:
            label1.text = "メッセージ内容1"
            break
        default:
            label1.text = "メッセージ内容2"
            break
        }
    }

    //以下のbutton2は、後で押します。質問の主旨に関係があるかわかりませんが、念のため記載します。
    @IBAction func button2(_ sender: Any) {
        locationManager.delegate = self
        locationManager.distanceFilter = 5
        locationManager.startUpdatingLocation()
    }
}

補足情報（FW/ツールのバージョンなど）

Xcode 10.2.1

Comments

[hoshi-takanori]

CLLocationManager のインスタンスを生成して delegate を設定してますか？

[bread_tabeyo]

hoshi-takanoriさん、コメントありがとうございます。はい、設定しているつもりです。質問のソースコード欄に、関係ないかと思って省略していた部分を追記したので、もし可能でしたら、ご確認いただけますと幸いです。（import、class、別のボタンbutton2）

[hoshi-takanori]

追記されたコードを見ましたが、locationManager はどこで初期化してますか？ また、button2 は「後で押します」ってことは、位置情報取得の許可が得られてから押される想定ということでしょうか？

[bread_tabeyo]

hoshi-takanoriさん、返信ありがとうございます。locationManagerの初期化というのは、

func setupLocationManager() {
locationManager = CLLocationManager()
}

と、記載すること、という理解で合っておりますでしょうか？これは記載していません。この記載の存在を、今ネットで検索して、知りました。

はい、button2は、位置情報取得の許可が得られてから押される想定です。

Answer 1

クラスの中に直接 var locationManager = CLLocationManager() と書いてますが、この一行で locationManager というプロパティを宣言し、CLLocationManager を生成して初期化する、という意味になりますので、func setupLocationManager() は不要です。

「button2は、位置情報取得の許可が得られてから押される」ということですが、button2 で delegate を設定していますので、button2 が押されるまでは locationManager(_:didChangeAuthorization:) が呼ばれることはありませんので、button2 が押されるまで位置情報取得の許可が得られたかどうかも分からないことになります。
たぶんそれでは困るでしょうから、とりあえず locationManager.delegate = self は button1 に移動したらいいと思います。

(なお、ViewController のプロパティとして locationManager を持つということは、その画面の生成と同時に CLLocationManager も生成し、その画面が存在する間だけ保持されることになります。単純なアプリならこれでいいのですが、画面が増えてきたらアプリの設計を見直す必要が生じるかもしれません。)

Comments

[bread_tabeyo]

hoshi-takanoriさん、回答ありがとうございます。
アドバイスの通り、locationManager.delegate = self をbutton1 に移動したら、解決できました！
カッコ内に記載いただいた内容も、勉強になりました。
親切に、またスピーディーにご対応いただき、感謝感謝です。