RegionBranchStoreUrlJson::findAll() - Code Metrics - TalismanFR/rosselhozbank-api - Measure and Improve Code Quality continuously with Scrutinizer

RegionBranchStoreUrlJson::findAll() A
last analyzed 2020-01-29 13:57 UTC

↳ Parent: RegionBranchStoreUrlJson

Complexity

Conditions	6
Paths	6

Size

Total Lines	28
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	0

Metric	Value
cc	6
eloc	17
c	2
b	1
f	0
nc	6
nop	0
dl	0
loc	28
rs	9.0777

<?php


namespace talismanfr\rosselhozbank\stores;


use talismanfr\rosselhozbank\dto\RegionBranch;
use talismanfr\rosselhozbank\shared\UrlValue;
use talismanfr\rosselhozbank\stores\contracts\RegionBranchStore;

class RegionBranchStoreUrlJson implements RegionBranchStore
{


    private $url;

    /** @var RegionBranch[]|null */
    private $all=null;

    /**
     * RegionBranchStoreUrlJson constructor.
     * @param $url
     */
    public function __construct(?UrlValue $url=null)
    {
        $this->url = $url;
        if(empty($this->url)){
            $this->url=new UrlValue('https://www.rshb.ru/promo/smb/rko-partner/js/region.json');
        }
    }


    /**
     * @inheritDoc
     * @return RegionBranch[]
     */
    public function findAll(): array
    {
        if($this->all){
            return $this->all;
        }


        $resource=file_get_contents($this->url->__toString());
        if(empty($resource)){
            return [];
        }

        try{
            $resource=json_decode($resource,true);
            if(!$resource){
                return [];
            }
        }catch (\Exception $e){
            return [];
        }
        foreach ($resource as $item){
            $this->all[]=new RegionBranch(
                (int)$item['id'],
                $item['origName'],
                $item['newName']
            );
        }
        return $this->all;

    }

    /**
     * @inheritDoc
     */
    public function findByRegion(string $region): ?RegionBranch
    {
        foreach ($this->findAll() as $regionBranch){
            if($regionBranch->getRegionName()==$region){
                return $regionBranch;
            }
        }

        return null;
    }

    public function findById(int $id): ?RegionBranch
    {
        foreach ($this->findAll() as $regionBranch){
            if($regionBranch->getId()==$id){
                return $regionBranch;
            }
        }

        return null;
    }
}

1			<?php
2
3
4			namespace talismanfr\rosselhozbank\stores;
5
6
7			use talismanfr\rosselhozbank\dto\RegionBranch;
8			use talismanfr\rosselhozbank\shared\UrlValue;
9			use talismanfr\rosselhozbank\stores\contracts\RegionBranchStore;
10
11			class RegionBranchStoreUrlJson implements RegionBranchStore
12			{
13
14
15			private $url;
16
17			/** @var RegionBranch[]\|null */
18			private $all=null;
19
20			/**
21			* RegionBranchStoreUrlJson constructor.
22			* @param $url
23			*/
24			public function __construct(?UrlValue $url=null)
25			{
26			$this->url = $url;
27			if(empty($this->url)){
28			$this->url=new UrlValue('https://www.rshb.ru/promo/smb/rko-partner/js/region.json');
29			}
30			}
31
32
33			/**
34			* @inheritDoc
35			* @return RegionBranch[]
36			*/
37			public function findAll(): array
38			{
39			if($this->all){
40			return $this->all;
41			}
42
43
44			$resource=file_get_contents($this->url->__toString());
45			if(empty($resource)){
46			return [];
47			}
48
49			try{
50			$resource=json_decode($resource,true);
51			if(!$resource){
52			return [];
53			}
54			}catch (\Exception $e){
55			return [];
56			}
57			foreach ($resource as $item){
58			$this->all[]=new RegionBranch(
59			(int)$item['id'],
60			$item['origName'],
61			$item['newName']
62			);
63			}
64			return $this->all;
			0 ignored issues – show Bug Best Practice introduced 2020-01-29 08:03 UTC by Report Bug Copy Issue Report The expression `return $this->all` could return the type `null` which is incompatible with the type-hinted return `array`. Consider adding an additional type-check to rule them out. Loading history...
65			}
66
67			/**
68			* @inheritDoc
69			*/
70			public function findByRegion(string $region): ?RegionBranch
71			{
72			foreach ($this->findAll() as $regionBranch){
73			if($regionBranch->getRegionName()==$region){
74			return $regionBranch;
75			}
76			}
77
78			return null;
79			}
80
81			public function findById(int $id): ?RegionBranch
82			{
83			foreach ($this->findAll() as $regionBranch){
84			if($regionBranch->getId()==$id){
85			return $regionBranch;
86			}
87			}
88
89			return null;
90			}
91			}

TalismanFR / rosselhozbank-api

RegionBranchStoreUrlJson::findAll() A last analyzed 2020-01-29 13:57 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

RegionBranchStoreUrlJson::findAll() A
last analyzed 2020-01-29 13:57 UTC