I have this (ugly) code that works : ```lifecycleScope.launch { viewModel.state.collect { update...

# codereview

[jean]

I have this (ugly) code that works :

lifecycleScope.launch {
    viewModel.state.collect { updateState(it, binding) }
}
lifecycleScope.launch {
    viewModel.load()
}

I cannot have viewModel.load() inside the first block after collect since it never ends and if I put it before I miss a bunch of events. Is there a way to write this in a better way?

[Giorgos Neokleous]

How about launchIn ?

lifecycleScope.launch {
    viewModel.state.onEach { updateState(it, binding) }.launchIn(lifecycleScope)

    viewModel.load()
}

[Giorgos Neokleous]

You can even take out the whole observation from the launch block with launchIn

[jean]

that’s a bit better indeed, thanks 🙂