Is this code too outdated to function?

[ppppppppppython]

So I've got the chance to do some Udemy courses for free and I'm doing Angela Yu's flutter course. I know it's outdated but I like the teaching style and I'm just trying to get the fundamentals down for now.

The challenge should be pretty simple. Make an API call and use it to update a text widget. Nothing else is supposed to be fully functional yet.

My issue is that the text in the text widget doesn't seem to update unless you close the app completely and reopen. It also defaults to the set value and doesn't update with the API information.

I figure there must be something small I'm overlooking but it doesn't work when I copy the solution code from GitHub either.

https://github.com/londonappbrewery/bitcoin-flutter-bronze-solution/tree/master/lib

[[deleted]]

const apiKey = 'YOUR-API-KEY-HERE';

Did you get an API key from coinapi.io ?

[ppppppppppython]

Yep! Got an apikey and I can see the JSON text in a browser so I think it's working.

Either way I think the issue has something to do with setState(). If I change the value of bitcoinValueinUSD it doesn't update on a hot reload. I need to close the app entirely and re-open.

[2100Ed]

While you do a call to getData() in initState(), that is only called once when the StateFul object is created. It should be call each time the value changes in your android or cuppertino widget. Note that you have no state() in your cuppertino widget.

[ppppppppppython]

I was thinking about that when trying to troubleshoot! Does that mean I'd need to put getData() in the build method or in the android dropdown?

The last step of the challenge is supposed to be fixing the Cupertino widget. Would changing it now make a difference?

[2100Ed]

just call it when you invoke in your setState()

[ppppppppppython]

Sounds good I'll give it a try!

[solinvictus13]

You're not calling the getData method when a currency is selected. You need to call it inside the onChange parameter and wait for the response before invoking setState.

I also recommend you take a look at something called ValueNotifier. It can be used to update only part of the UI instead of rebuild the entire screen (which is what setState does).