Anax\Weather\Weather

Complexity

Total Complexity

5

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setApiKey()	0	4	1
A	getWeather()	0	35	4

<?php

namespace Anax\Weather;

class Weather
{
    private $key;

    public function setApiKey($key)
    {
        $this->key = $key["config"]["key"];
        return $this;
    }


    public function getWeather($lat, $lon)
    {

        $muH = curl_multi_init();
        $res = [];
        $handles = [];
        $handles[0] = curl_init();
        curl_setopt($handles[0], CURLOPT_URL, "https://api.openweathermap.org/data/2.5/onecall?lat=${lat}&lon=${lon}&exclude=hourly,minutely&appid={$this->key}&units=metric&lang=sv");

It seems like $handles[0] can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

        curl_setopt($handles[0], CURLOPT_RETURNTRANSFER, true);
        curl_setopt($handles[0], CURLOPT_HEADER, false);
        curl_multi_add_handle($muH, $handles[0]);

It seems like $handles[0] can also be of type false; however, parameter $ch of curl_multi_add_handle() does only seem to accept resource, maybe add an additional type check?

        for ($i = 1; $i < 6; $i++) {
            $handles[$i] = curl_init();
            $days = "-${i} days";
            $time = date(strtotime($days));
            curl_setopt($handles[$i], CURLOPT_URL, "https://api.openweathermap.org/data/2.5/onecall/timemachine?lat=${lat}&lon=${lon}&dt=${time}&exclude=hourly,minutely&appid={$this->key}&units=metric&lang=sv");
            curl_setopt($handles[$i], CURLOPT_RETURNTRANSFER, true);
            curl_setopt($handles[$i], CURLOPT_HEADER, false);
            curl_multi_add_handle($muH, $handles[$i]);
        }

        $running = 0;
        do {
            \curl_multi_exec($muH, $running);
        } while ($running > 0);

        foreach ($handles as $i => $handle) {
            $res[$i] = \json_decode(\curl_multi_getcontent($handle), true);
            \curl_multi_remove_handle($muH, $handle);
        }

        \curl_multi_close($muH);

        return $res;
    }
}