avoid timer is added many times

[ooyamatakehisa]

Related Issue

• 226

What

• 226 を治そうと色々やってみたのですが、simulatorでは再現できず、すこし怪しいけどおそらく関係ないところに関するPRです

Memo

• 226 に関して、ミニプレイヤーで一時停止ボタンを押して、ボタン表示がちゃんと変わるということは、RecordingControllerのstopActionは正しく実行されている。stopActionの実態は親ViewのRecognitionPaneで定義されているpauseRecordingで、この中でタイマーの停止等もされているはず。なのにおそらく、波形を更新するタイマーと時間を更新するタイマーが止まっていない。

  つまり、以下の isPaused = true までは実行されてるから、ボタン表示はちゃんと切り替わってる。なのにそれ以降の行が実行されていない。

    func pauseRecording() {
        isPaused = true
        audioRecorder!.pause()
  
        updateRecordingTimeTimer?.invalidate()
        updateWaveformTimer?.invalidate()
        streamingRecognitionTimer?.invalidate()
    }

  Timerのinvalidateメソッドのドキュメントを読むと、タイマーが設定された場所と同じスレッドでinvalidateを実行しないといけないと書いてるが、これもすべてメインスレッドで行われている。別の可能性として、タイマーが余分に何個も作成されてRunLoopに追加されていて、@Stateで持ってるタイマーの参照以外の余分なタイマーがinvalidateされていないのが原因かとも思ったが、余分にタイマーが追加されているようにも見えない。

• 根本的な問題として、@Stateには値型のみを保存して、参照型には@StateObjectを使え、と公式Docにも書いてある。Timerは参照型でもしかするとそれに起因する問題かもしれない。 #226 の予期せぬところで波形が初期化される問題も、isAmpsリストが初期化されてしまうのが問題で、本来リストは@StateObjectとかで持たせるべきなので、これも同様の原因かもしれない。
  • https://developer.apple.com/tutorials/swiftui/handling-user-input

  • 277

[shibukazu]

https://github.com/Stateで持ってるタイマーの参照以外の余分なタイマーがinvalidateされていないのが原因かとも思ったが、余分にタイマーが追加されているようにも見えない。

個人的にはこれが原因としてありそうな気がしました.... このコードでも以下のような条件分岐を行なってisValidの場合は念の為invalidateするような実装にすれば重複してRunLoopに入ることは防げそうな気がしました。

if let recognizedResultsScrollTimer {
  if recognizedResultsScrollTimer.isValid { return }
}

[ooyamatakehisa]

@shibukazu 確かに書いといて損はないし、一応下のやつ足しとくわ！

if let recognizedResultsScrollTimer {
  if recognizedResultsScrollTimer.isValid { return }
}