If your function is marked suspend then you are ca...

# coroutines

[Tolriq]

If your function is marked suspend then you are calling it from a coroutineScope already. So it's maybe better to do

suspend fun fetchCharacterData(): CharacterGenerator.CharacterData = {
            val apiData = withContext(<http://Dispatchers.IO|Dispatchers.IO>) { URL(CHARACTER_DATA_ENDPOINT).readText()} 
            CharacterGenerator.fromApiData(apiData)
        }

to start on good bases.

[zak.taccardi]

CharacterGenerator.fromApiData(apiData) should be called on the default dispatcher though

[Tolriq]

Maybe my English is realy bad, but that's exactly what I'm trying to say actually 😉

[Mike Palarz]

Ughhh, still having issues. I can't seem to get fetchCharacterData() written correctly. I've tried what @Tolriq recommended, but I'm getting a ton of compiler errors, namely that a suspension function can only be called within a coroutine context...

[Tolriq]

Then you might want to read the coroutine starting guide to start.

[Mike Palarz]

I will do that. Thank you for all of the help so far! 🙂

[Mike Palarz]

I actually think I got this to work! Here's how I've adjusted `fetchCharacterData()`:

suspend fun fetchCharacterData(): CharacterGenerator.CharacterData = withContext(Dispatchers.Default){
    val apiData = withContext(<http://Dispatchers.IO|Dispatchers.IO>) { URL(CHARACTER_DATA_ENDPOINT).readText() }
    CharacterGenerator.fromApiData(apiData)
}

And then I call fetchCharacterData() as follows:

generateButton.setOnClickListener {
            launch {
                characterData = fetchCharacterData()
                displayCharacterData()
            }
        }

Not sure if either of you have any comments on this, but I believe this is along the lines of what both of you were recommending. Thanks again!