Hi everyone! Could someone please explain me why I...

# coroutines

[Alexey Pushkarev]

Hi everyone! Could someone please explain me why I can't call one suspend function inside another ?

[dewildte]

That's odd, you could async/await the return of connectService() but that seems wrong too.

[dewildte]

Why exactly are you using suspendKoroutine?

[Alexey Pushkarev]

because I just want to wrap callbacks

[Alexey Pushkarev]

async/await doesn't solve the problem. It tells me the same thing about body.

[dewildte]

Could you make connectService() an extension function of CoroutineScope?

[Alexey Pushkarev]

hm, but why?

[dewildte]

It's easier?

[dewildte]

And you don't get this obscure error?

[dewildte]

Ignore me I am not familiar with the suspendKoroutine API.

[Alexey Pushkarev]

ok, no problem.

[Dico]

suspendCoroutine takes a lambda parameter that accepts the continuation. The continuation must be scheduled to be called inside the lambda.

[Dico]

You're working with low level coroutine api here

[Alexey Pushkarev]

yeah, because I want to hide callback hell with it.

[Dico]

Yeah, just unwrap the suspendCoroutine call

[Dico]

You don't need to schedule the continuation in some special way using the api in your use case

[Alexey Pushkarev]

how to unwrap it?

[Dico]

Take the content of the lambda and make it the function body

[Dico]

Write the code in a single threaded style

[Dico]

The callbacks will get added by the compiler when you call suspend functions, basically

[Alexey Pushkarev]

Probably, you mean that I should remove suspendCoroutine from connectService(), right?

[Dico]

I think I have too little context to answer that

[Dico]

I'm talking about the bottom most function

[Alexey Pushkarev]

ah, sorry you haven't seen the whole method

[Alexey Pushkarev]

the code is messy, but at the first point I am gonna make it work )

[bdawg.io]

Check if the billing client isn’t ready first (and call connectService() if not), then invoke the suspend coroutine

suspend fun doSomethingAsync(...) {
    if (!billingClient.isReady) { connectService() }
    return suspendCoroutine<List<SkuDetails>> { cont -> ... }
}

[Alexey Pushkarev]

@bdawg.io yeah, I have recently tried such way. Probably works! Thanks anyway)

[uli]

I think the responsibility of this method should only be the wrapping of you callback api. I'd factor everything else (connectService,param building) out into a suspende function based on this one