Anax\Weather\WeatherController::checkWeatherAction()

Complexity

Conditions	7
Paths	7

Size

Total Lines	53
Code Lines	35

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	56

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function checkWeatherAction()
    {
        $page = $this->di->get("page");
        $title = "Resultat";
        $location = $_GET["location"];


        $type = $this->di->get("request")->getGet("type");

        $weather = $this->di->get("weather");

        if ($type == 'prognos') {
            if (strpos($location, ",") == true) {

It seems like you are loosely comparing strpos($location, ',') of type integer to the boolean true. If you are specifically checking for non-zero, consider using something more explicit like > 0 or !== 0 instead.

                $exploded = explode(",", $location);
                $answer = $weather->checkWeather($exploded[0], $exploded[1]);
            } else {
                $geo = new GetGeo();
                $res = $geo->getGeo($location);
                if ($res["longitude"] !== null) {
                    $answer = $weather->checkWeather($res["latitude"], $res["longitude"]);
                } else {
                    $error = "Felaktig input, prova igen!";
                }
            }
        } elseif ($type == 'history') {
            if (strpos($location, ",") == true) {

It seems like you are loosely comparing strpos($location, ',') of type integer to the boolean true. If you are specifically checking for non-zero, consider using something more explicit like > 0 or !== 0 instead.

                $exploded = explode(",", $location);
                $answer = $weather->checkHistory($exploded[0], $exploded[1]);
            } else {
                $geo = new GetGeo();
                $res = $geo->getGeo($location);
                if ($res["longitude"] !== null) {
                    $answer = $weather->checkHistory($res["latitude"], $res["longitude"]);
                } else {
                    $error = "Felaktig input, prova igen!";
                }
            }
        }



        $data = [
            "forecast" => $answer ?? null,
            "error" => $error ?? null,
            "lat" => $exploded[0] ?? $res["latitude"],

The variable $res does not seem to be defined for all execution paths leading up to this point.

            "lon" => $exploded[1] ?? $res["longitude"]
        ];


        $page->add("weather/resultpage", $data);

        return $page->render([
            "title" => $title,
        ]);