Anax\Service\WeatherService

Complexity

Total Complexity

17

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setMessage()	0	3	1
B	getWeather()	0	28	8
B	getWeatherThroughMultiCurl()	0	49	8

<?php

namespace Anax\Service;

use Anax\Ipstack\Ipstack;

/**
 * A plain service class with no dependencies.
 */
class WeatherService
{
    private $message = null;



    public function setMessage(string $message) : void
    {
        $this->message = $message;
    }
      
    public function getWeather($cIp, $cLat, $cLon)
    {
        $keyArray = include(__DIR__ . '/../../config/api_keys.php');
        
        $stack = new Ipstack();
        $lat = $cLat;
        $lon = $cLon;
        if (($cLat == null || $cLon == null) && $cIp != null) {
            $response = $stack->getIpInfo($cIp, $keyArray["keys"]["ipstack"]);
            $lat = $response["latitude"];
            $lon = $response["longitude"];
        }

        $query = "https://api.openweathermap.org/data/2.5/forecast?lat=". $lat . "&lon=".  $lon . "&appid=" . $keyArray["keys"]["weather"];

        if ($lat != null && $lon != null) {
            $res = file_get_contents($query) ?? null;
        }
        $info = json_decode($res ?? null, true);

        if ($info == null) {
            $info = ["error" => "Ingen info hittades för angivna ip/koordinater"];
        }
        if (!isset($response)) {
            $response = null;
        }

        return ["weather" => $info, "ip" => $response];
    }

    public function getWeatherThroughMultiCurl($cIp, $cLat, $cLon)
    {
        $keyArray = include(__DIR__ . '/../../config/api_keys.php');
        $stack = new Ipstack();
        $lat = $cLat;
        $lon = $cLon;
        if (($cLat == null || $cLon == null) && $cIp != null) {
            $ipresponse = $stack->getIpInfo($cIp, $keyArray["keys"]["ipstack"]);
            $lat = $ipresponse["latitude"];
            $lon = $ipresponse["longitude"];
        }

        $dateTime = [time() - 432000, time() - 345600, time() - 259200, time() - 172800, time() - 86400];
        $options = [
            CURLOPT_RETURNTRANSFER => true,
        ];

        // Add all curl handlers and remember them
        // Initiate the multi curl handler
        $mh = curl_multi_init();
        $chAll = [];
        foreach ($dateTime as $dt) {
            $ch = curl_init("https://api.openweathermap.org/data/2.5/onecall/timemachine?lat=". $lat . "&lon=".  $lon . "&dt=" . $dt . "&appid=" . $keyArray["keys"]["weather"]);
            curl_setopt_array($ch, $options);

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

            curl_multi_add_handle($mh, $ch);

It seems like $ch 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?

            $chAll[] = $ch;
        }

        // Execute all queries simultaneously,
        // and continue when all are complete
        $running = null;
        do {
            curl_multi_exec($mh, $running);
        } while ($running);

        // Close the handles
        foreach ($chAll as $ch) {
            curl_multi_remove_handle($mh, $ch);
        }
        curl_multi_close($mh);

        // All of our requests are done, we can now access the results
        $response = [];
        foreach ($chAll as $ch) {
            $data = curl_multi_getcontent($ch);
            $response[] = json_decode($data, true);
        }

        return ["weather" => $response ?? null, "ip" => $ipresponse ?? null];
    }
}