Check my weather app

1 Like

You should put this in the project feedback category.

It works. A few suggestions:

  1. You call ip-api and geolocation. But ip-api gives you longitude and latitude (under the loc property). I would just call that.

  2. On line 54 you begin a long chain of if/elseif statements. You should try using a switch statement here.

  3. Just as a pointer, I would suggest indenting your code properly. It will make it easier for you to read and much easier for the rest of us to follow. When in doubt, if you select everything and shift-tab, most code editors will auto indent for you.

But it looks good. Have fun with your Wikipedia project.

1 Like

Thank you for your feedback i will change the category, and thank you for all the suggestions i will check that too