TextFieldでOptional値を扱う場合の改善

前に書いたTextFieldでOptional値を扱う場合にView内の@State属性を付けたメンバー変数に一旦渡して処理する方法を試した。
メンバー変数に値を渡すタイミングとしてonAppear()を使っていた。

onAppear()は描画時に1回呼ばれるだけなので、外部で値が変更された場合に適用されない場合がある。

例としてこの様なモデルを用意した。
timeはOptionalでnilを与えることができる。

import Foundation
import SwiftUI

class Model: ObservableObject {
    init(name: String, time: Double?) {
        self.name = name
        self.time = time
    }
    var name: String
    var time: Double?
}

var models: [Model] = [
    Model(name: "A", time: 1.0),
    Model(name: "B", time: 2.0),
]

Viewとして、ボタンAとBを準備。
それぞれを押すとmodels[0]とmodels[1]の値を切り替えて編集できる。
Aを押してからBを押すと2.0が表示されるはずだがAを押した時に表示された1.0のままになっている。
起動時に1回onAppear()が呼び出されるが、Bを押した時にはonAppear()が呼び出されないので変わらない。

TextFieldでoptionalを扱う場合について · GitHub

Modelに一時変数を用意するのが正しいやり方ではないかと思う。

こんな感じ

class Model: ObservableObject {

.
.

    private var _timeStr: String? = nil
    var timeStr: String {
        get {
            if let _timeStr {
                return _timeStr
            } else {
                if let t = time {
                    _timeStr = String(t)
                } else {
                    _timeStr = ""
                }
            }
            return _timeStr!
        }
        set {
            _timeStr = newValue
        }
    }

ただ、このままだとtimeに反映されないので、編集が終わった段階で反映させる様にしなければならない。
ここではsave()メソッドで行うこととする。

    public func save() {
        if let _timeStr,
           let n = Double(_timeStr) {
            time = n
            debugPrint("save \(name)")
        }
        _timeStr = nil
    }

これで試すが、Bを選んで値を変更してリターンキーを押すが、Aのモデルの方のsave()が呼ばれてしまう。
これではBのモデルを変更できない。なんで？

"save A"
"save A"

TextFieldでoptionalを扱う場合について · GitHub

元に戻って@State属性を付けたメンバー変数に渡す方法でなんとかしてみる。

onAppear()の代わりのタイミングを作るためにTimerを使う。
modelが変化したらタイマーで遅らせて読み直す。

struct ModelView: View {
    @ObservedObject var model: Model
    @State var timeStr: String = ""
    
    @State var timer = Timer.publish(every: 0.1, on: .current, in: .common).autoconnect()

    var body: some View {
        HStack {
            Text(model.name)
            TextField("", text: $timeStr) { editing in
                if editing == false,
                   let t = Double(timeStr) {
                    debugPrint("save \(model.name)")
                    model.time = t
                }
            }
        }
        .onAppear() {
            stopTimer()
            reloadTime()
        }
        .onChange(of: model) { newValue in
            startTimer()
        }
        .onReceive(timer) { timer in
            stopTimer()
            reloadTime()
        }

    }
    
    private func reloadTime() {
        if let t = model.time {
            timeStr = String(t)
        } else {
            timeStr = ""
        }
    }
    
    private func startTimer() {
        timer = self.timer.upstream.autoconnect()
    }
    
    private func stopTimer() {
        timer.upstream.connect().cancel()
    }
}

変化したか比べるためにModelはEquatableをアダプトする必要がある。

class Model: ObservableObject, Equatable {
.
.    
    static func == (lhs: Model, rhs: Model) -> Bool {
        return lhs.name == rhs.name && lhs.time == rhs.time
    }
}

これで試してみる。
あれっ、こちらも変わらないぞ。

"save A"
"save A"

TextFieldでoptionalを扱う場合について · GitHub

原因はTextFieldのonEditingChangedに渡しているブロックだった。
ブロックを作成した時のモデルに紐づくのでいつまでもAのモデルを呼び出してしまう。

            TextField("", text: $timeStr) { editing in
                if editing == false,
                   let t = Double(timeStr) {
                    debugPrint("save \(model.name)")
                    model.time = t
                }
            }

onEditingChangedブロックではフラグを変えるだけにする。

    @State var isEditing = false
    
    var body: some View {
        HStack {
            Text(model.name)
            TextField("", text: $timeStr) { editing in
                isEditing = editing
            }
        }

フラグの変化をみて処理する様にする。

        .onChange(of: isEditing, perform: { editing in
            if editing == false,
               let t = Double(timeStr) {
                debugPrint("save \(model.name)")
                model.time = t
            }
        })

こんどはBのモデルが呼び出されている。

"save A"
"save B"

TextFieldでoptionalを扱う場合について · GitHub

Modelで処理する方も同じはず。

struct ModelView: View {
    @ObservedObject var model: Model
    
    @State var isEditing = false

    var body: some View {
        HStack {
            Text(model.name)
            TextField("", text: $model.timeStr) { editing in
                isEditing = editing
            }
        }
        .onChange(of: isEditing, perform: { editing in
            if editing == false {
               model.save()
            }
        })
    }
}

こっちもうまくいった。

"save A"
"save B"

TextFieldでoptionalを扱う場合について · GitHub

まとめ

モデルで処理するのが多分正しい方法。
- 個人的には _timeStr という感じで変数を使うのが気に入らないのだが…(といいつつobjcでは散々使っていた)　　
onEditingChanged ブロックはフラグ処理だけにする。
ViewでもTimer使えばなんとかなる。
- Viewが複雑(設定項目が増える)になってくるるとこっちの方がいいかなというのが現在の感想。