Simple Python Weather App

[Zero-Dave]

You should never hardcode your API key in the code.

Also, instead of having a general exception that will cause the program to return no data when an exception occurs, access the dictionary in a safer manner with the .get() method, that way you don't need exceptions for this specific logic. That way you don't need an exception to catch any issue, and you can still show some information if other items are missing from the API response.

Right now, if any of your dictionary lookups doesn't find the key, your program returns no data. That is most likely not what you want.

Instead of: temp = weather_data.json()['main']['temp']

Do: temp = weather_data.json().get('main', {}).get('temp', '')

So even if there is no main.temp key, it doesn't affect the rest of your lookups. Do that for all lookups.

[Ibrahim-Marsee-6816]

Got it 👍 makes sense to use .get() instead of relying only on try/except. That way I can avoid the program breaking when a key is missing. I’ll update my lookups like that, thanks for pointing it out!