Coordinates::getCoordinates() - Code Metrics - Inspection of "test config file edits" - Lyco18/weather-module - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( 860cfe...01dec1 )

by Lydia

created 2020-01-04 10:21 UTC

Coordinates::getCoordinates() A

↳ Parent: Coordinates

Complexity

Conditions	3
Paths	3

Size

Total Lines	20
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	16
c	1
b	0
f	0
dl	0
loc	20
rs	9.7333
cc	3
nc	3
nop	1

<?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;

        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			public function __construct($config)
13			{
14			$this->config = $config;
15			}
16
17			public function setConfig($config)
18			{
19			$this->config = $config;
20			}
21
22			public function getCoordinates(String $search) : array
23			{
24			$key = $this->config;
25
26			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			$details = json_decode(file_get_contents("https://api.opencagedata.com/geocode/v1/json?q=$search&key={$key}"));
28			$results = $details->results;
29			if (isset($results[0]->geometry)) {
30			$valid = "Valid";
31			$lat = $results[0]->geometry->lat;
32			$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			"valid" => $valid,
40			"lat" => $lat,
41			"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

Push — master ( 860cfe...01dec1 )

Coordinates::getCoordinates() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like