Как сделать чтобы при закрытии фрагмента к нему перестали обращаться?

Question

[qqweer]

CoroutineScope(Dispatchers.IO).launch {
            while (true) {
                val msg = Message()
                msg.what = mediaPlayer.currentPosition
                handler.sendMessage(msg)
                delay(10)
            }
        }
    }


    @SuppressLint("HandlerLeak")
    var handler = object : Handler(Looper.getMainLooper()) {


        override fun handleMessage(msg: Message) {
            val currentPosition = msg.what

            // Update positionBar
            positionBar.progress = currentPosition

            // Update Labels
            val elapsedTime = createTimeLabel(currentPosition)
            elapsedTimeLabel.text = elapsedTime

            val remainingTime = createTimeLabel(totalTime - currentPosition)
            remainingTimeLabel.text = "-$remainingTime"
        }
    }

Вылетает с текстом positionBar must not be null, потому что происходит обращение к нему когда закрыт фрагмент уже. Как решить это? Думал дело в том, что я использовал Thread, а не Coroutines, но оказывается не в этом. Как решить? Спасибо.

Answer 1

[Денис Загаевский]

Омагад.
Читай про structured concurrency. Читай про жизненный цикл фрагментов.
Всё происходит из-за того, что ты неправильно обращаешься с ними. Из-за того, что ты поменял один инструмент на другой, неправильное обращение никуда не делось. Ты должен отменить скоуп при выходе с фрагмента(в onDestroyView). И обращаться в мейнтред через MainDispatcher, раз уж ты используешь корутины.

Comments

[qqweer]

Я знаю про это, я просто не знал как завершить корутину. Спасибо, я нашёл функцию cancel() .

[Денис Загаевский]

Michael Lazebny, отменить корутину недостаточно, у тебя в хэндлере в этот момент уже может быть сообщение. Не надо смешивать такие инструменты без нужды.

[qqweer]

Денис Загаевский, как обработать нажатие аппаратной кнопки "Назад" во фрагменте?Когда нажимаю на неё , вылетает. По нажитию кнопки специальной на ui не вылетает.

[Денис Загаевский]

Михаил Лазебный, не надо её отдельно обрабатывать.

[qqweer]

Денис Загаевский, а как быть? Мне же нужно знать когда она будет нажата, чтобы остановить корутину и прочие штуки на вьюхе

[Денис Загаевский]

Михаил Лазебный, нет, не нужно. Фрагменты можно добавлять в backstack, и нужно этим пользоваться. Останавливать корутины надо из методов жизненного цикла, в частности onDestroyView.

[qqweer]

Денис Загаевский, видимо, у меня какая-то странная ситуация, фрагмент добавлен в backstack, корутину пытался остановить со всех методов, типа: onDestroy(), onDestroyView(), onStop(), безуспешно.Вылетает. Сорян, что мучаю тебе голову, но если есть материал где про это прочитать, сбрось пожалуйста, ибо не понимаю в чём причина.

[Денис Загаевский]

Михаил Лазебный, включай дебаггер и дебажь. Не надо тупо тыкать наугад. Как минимум читай стектрейс, ты до сих пор пишешь "вылетает" вместо конкретного исключения.

[qqweer]

Я вроде писал. У меня positionBar must not be null это seek bar который обновляет корутина

[qqweer]

[Денис Загаевский]

Михаил Лазебный, ты как минимум с июля пишешь чего-то под андроид и до сих пор не знаешь про Logcat?

[qqweer]

Денис Загаевский, знаю, думал тут и так показывает всё. Не суть, я решил это. Сократил весь проект до 1 активити( с песнями и фрагмента который проигрывает. Перенёс обновление positionBar в coroutines, проблема была в том, что у меня когда закрывался фрагмент успевало передаваться сообщение в handler и вылетало. Нужно было просто в Coroutines поставить вместо Dispatchers.IO Dispatchers.Main и засунуть туда функцию которая обновляет positionBar. Теперь можно двигаться дальше, спасибо за помощь.

[Денис Загаевский]

Михаил Лазебный, я тебе говорил про хэндлер 6 часов назад, не надо его использовать в такой ситуации.

[qqweer]

Денис Загаевский, да-да, я помнил, но невнимательность дала свои плоды, думал проканало. Блин, ты крут, даёшь не рыбу, а удочку.