【Swift3】observeValueメソッドのchange: [NSKeyValueChangeKey : Any]? のアンラップの方法

Question

AVAudioSession の outputVolume について addObserver で変更の通知を受け取るような実装を行いました。

    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)

        setSystemVoiceObserver()
    }
    
    func setSystemVoiceObserver() {
        let audioSession: AVAudioSession = AVAudioSession.sharedInstance()
        try! audioSession.setCategory(AVAudioSessionCategoryAmbient)
        try! audioSession.setActive(true)
        
        audioSession.addObserver(self, forKeyPath: "outputVolume", options: .new, context: nil)
    }
    
    override func viewDidDisappear(_ animated: Bool) {
        super.viewDidDisappear(animated)
        
        let audioSession: AVAudioSession = AVAudioSession.sharedInstance()
        audioSession.removeObserver(self, forKeyPath: "outputVolume", context: nil)
    }

    override func observeValue(forKeyPath keyPath: String?, of object: Any?, change: [NSKeyValueChangeKey : Any]?, context: UnsafeMutableRawPointer?) {
        if keyPath == "outputVolume" {
            let i: UInt32 = UInt32(change![.newKey]! as! Float * 100)
            setVolumeLabelText(i) // 音量の変更を受け取ったあとに行う処理
        }
    }

上記コードのchangeパラメータの中身を取得するのに強制的アンラップを１行で３回も行ってしまいました。swiftlintにも怒られてしまい、とても罪悪感があるのですが、一方で「受け取るものの型も存在もわかりきってるのだからいいんじゃないか」とも思っています。

(1)このような通知を受け取るメソッドで実行時にエラーになる可能性はどれくらいあるのでしょうか？
(2)このような通知のoptional型をいい感じに取り出す方法はないでしょうか？if letとoptional chainingでいけるか試してみましたが、

            if let i: UInt32 = UInt32(change?[.newKey]? as? Float * 100) {
                setVolumeLabelText(i) // 音量の変更を受け取ったあとに行う処理
            }

上記のコードではas?をas!や(change?[.newKey] as? Float)!で置き換えるようにコンパイラが警告してきて、結局どこかで強制アンラップをしなければならないような状態です。できれば１行でnil合体演算子も使いたくはないのですが……。

Answer 1

普通に書いてみたつもり。

if keyPath == "outputVolume", let change = change, let newKey = change[.newKey] as? Float {
    let i = UInt32(newKey * 100)
    setVolumeLabelText(i) // 音量の変更を受け取ったあとに行う処理
}

Answer 2

可読性がちょっとあれですが

if let i = change?[.newKey].flatMap({ $0 as? Float}).map({ UInt32($0 * 100) }) {
    setVolumeLabelText(i)
}

こうですかね？

Optional.map / Optional.flatMapを使用しています。

Comments

[idonotknow]

関数型プログラミングな感じでかっこいいです。
なぜflatMap?と思ったのですが、調べてみたらOptional型をアンラップしてくれるのですね。勉強になりました、ありがとうございます。