any way to write this in a more idiomatic way? (p...

# coroutines

[Andrew Gazelka]

any way to write this in a more idiomatic way? (particularly the exception throwing to exit collect {})

[streetsofboston]

Not sure if there is one, but maybe the Flow has a takeUntil method or something similar. Then you can just do a plain collect on that resulting Flow…

[streetsofboston]

There is no takeUntil, but there is a takeWhile function for a Flow.

[streetsofboston]

Ah.. I see you already use ‘takeWhile’…. hmmmm

[bezrukov]

I think you can use first here

[bezrukov]

https://kotlin.github.io/kotlinx.coroutines/kotlinx-coroutines-core/kotlinx.coroutines.flow/first.html

[streetsofboston]

Also, I’m not sure where over in the code-snippet comes from….

[Andrew Gazelka]

I think I’ll just use `first {}`… it is good enough

[Andrew Gazelka]

also over does not matter… it is not in the context of the flow

[gildor]

using mutable state is not really idiomatic, why do you need this last?

[Andrew Gazelka]

it needed to be used outside of the try{} catch () statement

[Andrew Gazelka]

I don’t think it really related highly to the problem tho

[Andrew Gazelka]

I think first{} is good enough

[gildor]

first os good enough

[gildor]

I just worry to set it from collect instead of receive it from first() or other terminal operator