C#•2d ago

Accessing data for multiple days from weather api

I have the program working where it displays the data for the current day but I not to sure how to get the weather data for the following days. Do I have to first send the json results to an array and cycle through?

6 Replies

[ECH]JamHighlightOP•2d ago

Like where I add adding the icon [weather][0] do I have to use a loop to go through the index

Sebastian•2d ago

You probably need to call a different API endpoint See the docs here https://openweathermap.org/api

Weather API

Explore OpenWeather's vast range of weather APIs including the versatile One Call API 3.0. Ideal for both beginners and professionals, our APIs offer current weather, minute-by-minute forecasts, historical data archives, and future predictions. Access weather data starting from 01-01-1979, global weather maps, solar irradiance predictions, air p...

[ECH]JamHighlightOP•2d ago

api.openweathermap.org/data/2.5/forecast?lat={lat}&lon={lon}&appid={API key} Ye i had to use this one instead

Sebastian•2d ago

Also remove the API key from your message and keep it private

Sebastian•2d ago

Here are some additional pointers: 1. Move the API key out of your code entirely, load it from a local configuration file. Make sure that the configuration file is not tracked by a versioning system (so if you are using git, add it to .gitignore) 2. Don't initialize HttpClient repeatedly. This is especially bad with this type. 3. Always return early if you can (google "c# return early" if you don't know what I mean) 4. You were checking whether the HttpClient was null right after creating it (so it should never be null, the condition is useless) 5. Ideally you should replace JObject with something else 6. You forgot to dispose the object in response variable

private const string ApiKey = "..."; // TODO Load from configuration file instead, keeping this in code is a bad idea

private readonly HttpClient client = new();

async void FetchWeatherForecast()
{
    string city = Citysearch.Text;
    string url = $"https://api.openweathermap.org/data/2.5/forecast?lat={lat}&lon={lon}&appid={ApiKey}";
    
    try
    {
        using HttpResponseMessage response = await client.GetAsync(apiUrl);
        response.EnsureSuccessStatusCode();
        string content = await response.ReadAsStringAsync();
        if (content is null)
        {
            Console.WriteLine("Received no content from wheather API");
            return;
        }

        JObject weatherData = JObject.Parse(content);
        // Handle weather data
    }
    catch (Exception ex)
    {
        Console.WriteLine($"Error while fetching from weather API: {ex.Message}");
    }
}

private const string ApiKey = "..."; // TODO Load from configuration file instead, keeping this in code is a bad idea

private readonly HttpClient client = new();

async void FetchWeatherForecast()
{
    string city = Citysearch.Text;
    string url = $"https://api.openweathermap.org/data/2.5/forecast?lat={lat}&lon={lon}&appid={ApiKey}";
    
    try
    {
        using HttpResponseMessage response = await client.GetAsync(apiUrl);
        response.EnsureSuccessStatusCode();
        string content = await response.ReadAsStringAsync();
        if (content is null)
        {
            Console.WriteLine("Received no content from wheather API");
            return;
        }

        JObject weatherData = JObject.Parse(content);
        // Handle weather data
    }
    catch (Exception ex)
    {
        Console.WriteLine($"Error while fetching from weather API: {ex.Message}");
    }
}

How to deserialize JSON in C# - .NET

Learn how to use the System.Text.Json namespace to deserialize from JSON in .NET. Includes sample code.

Scratch•2d ago

@[ECH]JamHighlight I deleted the message with your code, you leaked your api key. I recommend you rotate it.

Gaming

Programming

Accessing data for multiple days from weather api

Did you find this page helpful?