Coordinates::getCoordinates() - Code Metrics - Lyco18/weather-module - Measure and Improve Code Quality continuously with Scrutinizer

Coordinates::getCoordinates() A
last analyzed 2020-02-03 08:14 UTC

↳ Parent: Coordinates

Complexity

Conditions	3
Paths	3

Size

Total Lines	20
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	3.072

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	16
c	1
b	0
f	0
dl	0
loc	20
ccs	12
cts	15
cp	0.8
rs	9.7333
cc	3
nc	3
nop	1
crap	3.072

<?php
namespace Anax\Model;

class Coordinates
{
    private $config;
    /**
     * Constructor, allow for $di to be injected.
     *
     * @param Array $config for api key
     */
    public function __construct($config)
    {
        $this->config = $config;
    }

    public function setConfig($config)
    {
        $this->config = $config;
    }

    public function getCoordinates(String $search) : array
    {
        $key = $this->config->config;

        if (is_string($search)) {

            $details = json_decode(file_get_contents("https://api.opencagedata.com/geocode/v1/json?q=$search&key={$key}"));
            $results = $details->results;
            if (isset($results[0]->geometry)) {
                $valid = "Valid";
                $lat = $results[0]->geometry->lat;
                $long = $results[0]->geometry->lng;
            } else {
                $valid = "Not valid, showing Gold Coast instead";
                $lat = -28.016666;
                $long = 153.399994;
            }
            return [
                "valid" => $valid,
                "lat" => $lat,
                "long" => $long
            ];
        }
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

class MyClass implements ReturnsInt {
    public function returnsIntHinted(): int
    {
        if (foo()) {
            return 123;
        }
        // here: null is implicitly returned
    }
}
    }
}


1		<?php
2		namespace Anax\Model;
3
4		class Coordinates
5		{
6		private $config;
7		/**
8		* Constructor, allow for $di to be injected.
9		*
10		* @param Array $config for api key
11		*/
12	1	public function __construct($config)
13		{
14	1	$this->config = $config;
15	1	}
16
17	1	public function setConfig($config)
18		{
19	1	$this->config = $config;
20	1	}
21
22	1	public function getCoordinates(String $search) : array
23		{
24	1	$key = $this->config->config;
25
26	1	if (is_string($search)) {
		0 ignored issues – show introduced 2020-01-04 10:24 UTC by Report Bug Copy Issue Report The condition `is_string($search)` is always `true`. Loading history...
27	1	$details = json_decode(file_get_contents("https://api.opencagedata.com/geocode/v1/json?q=$search&key={$key}"));
28	1	$results = $details->results;
29	1	if (isset($results[0]->geometry)) {
30	1	$valid = "Valid";
31	1	$lat = $results[0]->geometry->lat;
32	1	$long = $results[0]->geometry->lng;
33		} else {
34		$valid = "Not valid, showing Gold Coast instead";
35		$lat = -28.016666;
36		$long = 153.399994;
37		}
38		return [
39	1	"valid" => $valid,
40	1	"lat" => $lat,
41	1	"long" => $long
42		];
43		}
		0 ignored issues – show Bug Best Practice introduced 2020-01-02 11:42 UTC by Report Bug Copy Issue Report The function implicitly returns `null` when the `if` condition on line 26 is `false`. This is incompatible with the type-hinted return `array`. Consider adding a return statement or allowing `null` as return value. For hinted functions/methods where all return statements with the correct type are only reachable via conditions, ?null? gets implicitly returned which may be incompatible with the hinted type. Let?s take a look at an example: interface ReturnsInt { public function returnsIntHinted(): int; } class MyClass implements ReturnsInt { public function returnsIntHinted(): int { if (foo()) { return 123; } // here: null is implicitly returned } } Loading history...
44		}
45		}
46

Lyco18 / weather-module

Coordinates::getCoordinates() A last analyzed 2020-02-03 08:14 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

Coordinates::getCoordinates() A
last analyzed 2020-02-03 08:14 UTC