Issues in StatefulGeocoder.php - New Issues - Inspection of "Replace empty() by more strict checks (#1125)" - geocoder-php/php-common - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c4a119...b5db7c )

by Tobias

created 2021-06-21 06:13 UTC

StatefulGeocoder.php (4 issues)

Labels

Duplication 4

Severity

Informational 4

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php

declare(strict_types=1);

/*
 * This file is part of the Geocoder package.
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @license    MIT License
 */

namespace Geocoder;

use Geocoder\Model\Bounds;
use Geocoder\Query\GeocodeQuery;
use Geocoder\Query\ReverseQuery;
use Geocoder\Provider\Provider;

/**
 * @author Tobias Nyholm <[email protected]>
 */
final class StatefulGeocoder implements Geocoder
{
    /**
     * @var string|null
     */
    private $locale;

    /**
     * @var Bounds
     */
    private $bounds;

    /**
     * @var int
     */
    private $limit;

    /**
     * @var Provider
     */
    private $provider;

    /**
     * @param Provider $provider
     * @param string   $locale
     */
    public function __construct(Provider $provider, string $locale = null)
    {
        $this->provider = $provider;
        $this->locale = $locale;
        $this->limit = Geocoder::DEFAULT_RESULT_LIMIT;
    }

    /**
     * {@inheritdoc}
     */
    public function geocode(string $value): Collection
    {
        $query = GeocodeQuery::create($value)
            ->withLimit($this->limit);

        if (null !== $this->locale && '' !== $this->locale) {

            $query = $query->withLocale($this->locale);
        }

        if (!empty($this->bounds)) {
            $query = $query->withBounds($this->bounds);
        }

        return $this->provider->geocodeQuery($query);
    }

    /**
     * {@inheritdoc}
     */
    public function reverse(float $latitude, float $longitude): Collection
    {
        $query = ReverseQuery::fromCoordinates($latitude, $longitude)
            ->withLimit($this->limit);

        if (null !== $this->locale && '' !== $this->locale) {

            $query = $query->withLocale($this->locale);
        }

        return $this->provider->reverseQuery($query);
    }

    /**
     * {@inheritdoc}
     */
    public function geocodeQuery(GeocodeQuery $query): Collection
    {
        $locale = $query->getLocale();
        if ((null === $locale || '' === $locale) && null !== $this->locale) {

            $query = $query->withLocale($this->locale);
        }

        $bounds = $query->getBounds();
        if (empty($bounds) && null !== $this->bounds) {
            $query = $query->withBounds($this->bounds);
        }

        return $this->provider->geocodeQuery($query);
    }

    /**
     * {@inheritdoc}
     */
    public function reverseQuery(ReverseQuery $query): Collection
    {
        $locale = $query->getLocale();
        if ((null === $locale || '' === $locale) && null !== $this->locale) {

            $query = $query->withLocale($this->locale);
        }

        return $this->provider->reverseQuery($query);
    }

    /**
     * @param string $locale
     *
     * @return StatefulGeocoder
     */
    public function setLocale(string $locale): self
    {
        $this->locale = $locale;

        return $this;
    }

    /**
     * @param Bounds $bounds
     *
     * @return StatefulGeocoder
     */
    public function setBounds(Bounds $bounds): self
    {
        $this->bounds = $bounds;

        return $this;
    }

    /**
     * @param int $limit
     *
     * @return StatefulGeocoder
     */
    public function setLimit(int $limit): self
    {
        $this->limit = $limit;

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function getName(): string
    {
        return 'stateful_geocoder';
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		/*
6		* This file is part of the Geocoder package.
7		* For the full copyright and license information, please view the LICENSE
8		* file that was distributed with this source code.
9		*
10		* @license MIT License
11		*/
12
13		namespace Geocoder;
14
15		use Geocoder\Model\Bounds;
16		use Geocoder\Query\GeocodeQuery;
17		use Geocoder\Query\ReverseQuery;
18		use Geocoder\Provider\Provider;
19
20		/**
21		* @author Tobias Nyholm <[email protected]>
22		*/
23		final class StatefulGeocoder implements Geocoder
24		{
25		/**
26		* @var string\|null
27		*/
28		private $locale;
29
30		/**
31		* @var Bounds
32		*/
33		private $bounds;
34
35		/**
36		* @var int
37		*/
38		private $limit;
39
40		/**
41		* @var Provider
42		*/
43		private $provider;
44
45		/**
46		* @param Provider $provider
47		* @param string $locale
48		*/
49		public function __construct(Provider $provider, string $locale = null)
50		{
51		$this->provider = $provider;
52		$this->locale = $locale;
53		$this->limit = Geocoder::DEFAULT_RESULT_LIMIT;
54		}
55
56		/**
57		* {@inheritdoc}
58		*/
59		public function geocode(string $value): Collection
60		{
61		$query = GeocodeQuery::create($value)
62		->withLimit($this->limit);
63
64	View Code Duplication	if (null !== $this->locale && '' !== $this->locale) {
		0 ignored issues – show Duplication introduced 2021-06-21 06:14 UTC by Report Bug Copy Issue Report Show Similar Issues like this This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
65		$query = $query->withLocale($this->locale);
66		}
67
68		if (!empty($this->bounds)) {
69		$query = $query->withBounds($this->bounds);
70		}
71
72		return $this->provider->geocodeQuery($query);
73		}
74
75		/**
76		* {@inheritdoc}
77		*/
78		public function reverse(float $latitude, float $longitude): Collection
79		{
80		$query = ReverseQuery::fromCoordinates($latitude, $longitude)
81		->withLimit($this->limit);
82
83	View Code Duplication	if (null !== $this->locale && '' !== $this->locale) {
		0 ignored issues – show Duplication introduced 2021-06-21 06:14 UTC by Report Bug Copy Issue Report Show Similar Issues like this This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
84		$query = $query->withLocale($this->locale);
85		}
86
87		return $this->provider->reverseQuery($query);
88		}
89
90		/**
91		* {@inheritdoc}
92		*/
93		public function geocodeQuery(GeocodeQuery $query): Collection
94		{
95		$locale = $query->getLocale();
96	View Code Duplication	if ((null === $locale \|\| '' === $locale) && null !== $this->locale) {
		0 ignored issues – show Duplication introduced 2021-06-21 06:14 UTC by Report Bug Copy Issue Report Show Similar Issues like this This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
97		$query = $query->withLocale($this->locale);
98		}
99
100		$bounds = $query->getBounds();
101		if (empty($bounds) && null !== $this->bounds) {
102		$query = $query->withBounds($this->bounds);
103		}
104
105		return $this->provider->geocodeQuery($query);
106		}
107
108		/**
109		* {@inheritdoc}
110		*/
111		public function reverseQuery(ReverseQuery $query): Collection
112		{
113		$locale = $query->getLocale();
114	View Code Duplication	if ((null === $locale \|\| '' === $locale) && null !== $this->locale) {
		0 ignored issues – show Duplication introduced 2021-06-21 06:14 UTC by Report Bug Copy Issue Report Show Similar Issues like this This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
115		$query = $query->withLocale($this->locale);
116		}
117
118		return $this->provider->reverseQuery($query);
119		}
120
121		/**
122		* @param string $locale
123		*
124		* @return StatefulGeocoder
125		*/
126		public function setLocale(string $locale): self
127		{
128		$this->locale = $locale;
129
130		return $this;
131		}
132
133		/**
134		* @param Bounds $bounds
135		*
136		* @return StatefulGeocoder
137		*/
138		public function setBounds(Bounds $bounds): self
139		{
140		$this->bounds = $bounds;
141
142		return $this;
143		}
144
145		/**
146		* @param int $limit
147		*
148		* @return StatefulGeocoder
149		*/
150		public function setLimit(int $limit): self
151		{
152		$this->limit = $limit;
153
154		return $this;
155		}
156
157		/**
158		* {@inheritdoc}
159		*/
160		public function getName(): string
161		{
162		return 'stateful_geocoder';
163		}
164		}
165

geocoder-php / php-common

Push — master ( c4a119...b5db7c )

StatefulGeocoder.php (4 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like