順番にCountDownTimerを呼び出したいが、同時に動いてしまう。

Question

前提・実現したいこと

入力した時間分カウントダウンするタイマーアプリを作っているのですが、
同じCountDown関数を用いて「3.2.1」と、開始前のカウントダウンを表示した後に、
メインのカウントダウンを動かしたいです。

しかし現状は開始前のカウントダウンと、
メインのカウントダウンが同時に動いてしまうため困っています。

該当のソースコード

package com.example.TapMeditation

import androidx.appcompat.app.AppCompatActivity
import android.os.Bundle
import android.os.CountDownTimer
import android.view.View
import kotlinx.android.synthetic.main.activity_meditation.*

class Meditation : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState,)
        setContentView(R.layout.activity_meditation,)

        //問題の箇所、1行目と2行目が同時に動いてしまう。
        //理想の動作としては1行目の終了後に、2行目を呼び出したい
        CountDown(3000)
        CountDown(intent.getLongExtra("TIME",0))　//他のアクティビティーから時間を受け取る
        }

    //カウントダウン開始関数
    private lateinit var countDownTimer: CountDownTimer
        fun CountDown(time:Long) {

            countDownTimer = object : CountDownTimer(time, 500) {
                override fun onTick(time: Long) {
                    val elapsedMinutes = time / 60000
                    val elapsedSeconds =(time-elapsedMinutes*60000)/1000

                    //カウントダウンをtxt_timeleftにて表示
                    txt_timeleft.text = String.format("%1$02d",elapsedMinutes)+":"+String.format("%1$02d",elapsedSeconds)
                }

                override fun onFinish() {
                }

            }.start()
        }
}

補足

・1行目、2行目を入れ替えると、
表示されるカウントダウンは変わるものの、同時にタイマーが動いている。

・エミュレーター上では、2行目のカウントダウン終了後に、
1行目のカウントダウンが残っていればそちらが表示される。

Comments

[mikan_s4n]

callback処理は試されましたか？

Answer 1

普通にcall backを受け取る方法でやってみました。

package com.example.TapMeditation

import androidx.appcompat.app.AppCompatActivity
import android.os.Bundle
import android.os.CountDownTimer
import android.view.View
import kotlinx.android.synthetic.main.activity_meditation.*

class Meditation : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState,)
        setContentView(R.layout.activity_meditation,)

        CountDown(3000) {
            CountDown(intent.getLongExtra("TIME", 0))
        }
    }

    //カウントダウン開始関数
    private lateinit var countDownTimer: CountDownTimer
    
    private fun CountDown(time:Long, onFinish : (() -> Unit)? = null) {

        countDownTimer = object : CountDownTimer(time, 500) {
            override fun onTick(time: Long) {
                val elapsedMinutes = time / 60000
                val elapsedSeconds =(time-elapsedMinutes*60000)/1000

                //カウントダウンをtxt_timeleftにて表示
                txt_timeleft.text = String.format("%1$02d",elapsedMinutes)+":"+String.format("%1$02d",elapsedSeconds)
            }

            override fun onFinish() {
                onFinish()
            }

        }.start()
    }
}

Answer 2

onCreate()に関数を並べただけでは、そのまま流れてしまうのですぐに後者が実行されてしまいます。前者が終了してから後者が実行されるような工夫が必要になります。

いろいろやり方はあるでしょうが、CountDown()を「カウントが終わるまで待たせる関数」に改良して、KotlinのCoroutineという非同期処理を用いて並べた2つのCountDown()関数を順番に実行してみてはどうでしょうか。

まずはCoroutineを用いるために、build.gradle(:app)のdependenciesブロックに1行追加してください。

dependencies {

    implementation 'org.jetbrains.kotlinx:kotlinx-coroutines-android:1.3.7'

}

一旦Syncしてから、Kotlinのコードを修正してください。

class Meditation : AppCompatActivity() {
    private val scope = MainScope()

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_meditation)

        //問題の箇所、1行目と2行目が同時に動いてしまう。
        //理想の動作としては1行目の終了後に、2行目を呼び出したい

        // このlaunchのブロックの中がCoroutineとして実行される
        scope.launch {
            CountDown(3000)
            CountDown(intent.getLongExtra("TIME", 0)) //他のアクティビティーから時間を受け取る
        }
    }

    //カウントダウン開始関数
    private lateinit var countDownTimer: CountDownTimer

    // suspendを付加して「待たせる関数」であることを宣言する
    private suspend fun CountDown(time: Long) = suspendCoroutine<Unit> { cont ->

        countDownTimer = object : CountDownTimer(time, 500) {
            override fun onTick(time: Long) {
                val elapsedMinutes = time / 60000
                val elapsedSeconds = (time - elapsedMinutes * 60000) / 1000

                //カウントダウンをtxt_timeleftにて表示
                //txt_timeleft.text = String.format("%1$02d",elapsedMinutes)+":"+String.format("%1$02d",elapsedSeconds)
                val timeLeft = "%02d:%02d".format(elapsedMinutes, elapsedSeconds) // シンプルにしてみた
                txt_timeleft.text = timeLeft
            }

            override fun onFinish() {
                cont.resume(Unit) // suspend状態を解除する
            }
        }.start()

        return@suspendCoroutine // resume()が呼ばれるまで待つ
    }

    override fun onDestroy() {
        super.onDestroy()
        // Activityが破棄されたときの後始末
        if (::countDownTimer.isInitialized) {
            countDownTimer.cancel()
        }
        scope.cancel()
    }
}

元々の構造をできるだけ活かしているので流れ自体は読みやすいのではと思いますが、launchやらsuspendCoroutineやらと見慣れない関数については、それぞれのキーワードで検索すれば情報は豊富にありますので、それを参考にされると良いでしょう。

Comments

[satou_19]

ご返信遅くなり申し訳ありません！
無事に解決致しました！

わざわざコードまで書いて頂き、
ありがとうございます！

コルーチンという処理について知ることができたので、
とても良い学びの機会になりました！

ただ、コールバック処理といった基本的な処理について、
まだまだ理解が浅い部分があるため、
今後はもう少し基礎的な手法を学んでから質問したいと思います！