Я думаю, что вы можете столкнуться с проблемами безопасности потоков, когда лениво добавляете Job
, как в своем обновлении, потому что этот код оценивается из любого потока, который запускает сопрограмму.
Исходный пример, с другой стороны, гарантирует, что задание настроено из основного потока, и это происходит до того, как другие потоки могут быть запущены в обычном действии Android.
Вы можете достичь чего-то похожего на первоначальный пример, создав целое CoroutineContext
в начале области. Это также устраняет необходимость вычисления окончательного контекста для каждой запущенной сопрограммы. Например:
class LifecycleCrScope : CoroutineScope, LifecycleObserver {
private var _ctx: CoroutineContext? = null
override val coroutineContext: CoroutineContext
get() = _ctx!! // throws when not within scope
private fun startScope() {
_ctx = Dispatchers.Main + Job()
}
private fun endScope() {
_ctx!![Job]!!.cancel() // throws only when misused, e.g. no startScope()
_ctx = null
}
@OnLifecycleEvent(ON_RESUME) // <-.
fun resume() { // | Beware:
startScope() // | symmetric but no scope
} // | during onCreate,
// | onStart, onStop, ...
@OnLifecycleEvent(ON_PAUSE) // <-.
fun pause() {
endScope()
}
}
В качестве альтернативы, если вам не нравится бросать
class LifecycleCrScope : CoroutineScope, LifecycleObserver {
// initially cancelled. Jobs started outside scope will not execute silently
private var _ctx: CoroutineContext = Dispatchers.Main + Job().apply { cancel() }
..
private fun endScope() {
_ctx[Job]!!.cancel() // should never throw
}
..