User::fromApi() - Code Metrics - Inspection of "Use explicit models instead of anonymous arrays" - core23/lastfm-php-api - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 547017...c67bb4 )

by Christian

created 2018-03-31 15:24 UTC

User::fromApi() A

↳ Parent: User

Complexity

Conditions	3
Paths	1

Size

Total Lines	11
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	11
rs	9.4285
cc	3
eloc	8
nc	1
nop	1

<?php

declare(strict_types=1);

/*
 * (c) Christian Gripp <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Core23\LastFm\Model;

final class User
{
    /**
     * @var string
     */
    private $name;

    /**
     * @var string|null
     */
    private $country;

    /**
     * @var int|null
     */
    private $age;

    /**
     * @var string|null
     */
    private $gender;

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

    /**
     * @var string|null
     */
    private $url;

    /**
     * User constructor.
     *
     * @param string      $name
     * @param null|string $country
     * @param int|null    $age
     * @param null|string $gender
     * @param int         $playcount
     * @param null|string $url
     */
    public function __construct(
        string $name,
        ?string $country,
        ?int $age,
        ?string $gender,
        int $playcount,
        ?string $url
    ) {
        $this->name      = $name;
        $this->country   = $country;
        $this->age       = $age;
        $this->gender    = $gender;
        $this->playcount = $playcount;
        $this->url       = $url;
    }

    /**
     * @return string
     */
    public function getName(): string
    {
        return $this->name;
    }

    /**
     * @return null|string
     */
    public function getCountry(): ?string
    {
        return $this->country;
    }

    /**
     * @return int|null
     */
    public function getAge(): ?int
    {
        return $this->age;
    }

    /**
     * @return null|string
     */
    public function getGender(): ?string
    {
        return $this->gender;
    }

    /**
     * @return int
     */
    public function getPlaycount(): int
    {
        return $this->playcount;
    }

    /**
     * @return null|string
     */
    public function getUrl(): ?string
    {
        return $this->url;
    }

    /**
     * @param array $data
     *
     * @return User
     */
    public static function fromApi(array $data): self
    {
        return new self(
            $data['name'],
            $data['country'] ?? null,
            $data['age'] ? (int) $data['age'] : null,
            $data['gender'] ?? null,
            $data['playcount'] ? (int) $data['playcount'] : null,
            $data['url'] ?? null
        );
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* (c) Christian Gripp <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Core23\LastFm\Model;
13
14			final class User
15			{
16			/**
17			* @var string
18			*/
19			private $name;
20
21			/**
22			* @var string\|null
23			*/
24			private $country;
25
26			/**
27			* @var int\|null
28			*/
29			private $age;
30
31			/**
32			* @var string\|null
33			*/
34			private $gender;
35
36			/**
37			* @var int
38			*/
39			private $playcount;
40
41			/**
42			* @var string\|null
43			*/
44			private $url;
45
46			/**
47			* User constructor.
48			*
49			* @param string $name
50			* @param null\|string $country
51			* @param int\|null $age
52			* @param null\|string $gender
53			* @param int $playcount
54			* @param null\|string $url
55			*/
56			public function __construct(
57			string $name,
58			?string $country,
59			?int $age,
60			?string $gender,
61			int $playcount,
62			?string $url
63			) {
64			$this->name = $name;
65			$this->country = $country;
66			$this->age = $age;
67			$this->gender = $gender;
68			$this->playcount = $playcount;
69			$this->url = $url;
70			}
71
72			/**
73			* @return string
74			*/
75			public function getName(): string
76			{
77			return $this->name;
78			}
79
80			/**
81			* @return null\|string
82			*/
83			public function getCountry(): ?string
84			{
85			return $this->country;
86			}
87
88			/**
89			* @return int\|null
90			*/
91			public function getAge(): ?int
92			{
93			return $this->age;
94			}
95
96			/**
97			* @return null\|string
98			*/
99			public function getGender(): ?string
100			{
101			return $this->gender;
102			}
103
104			/**
105			* @return int
106			*/
107			public function getPlaycount(): int
108			{
109			return $this->playcount;
110			}
111
112			/**
113			* @return null\|string
114			*/
115			public function getUrl(): ?string
116			{
117			return $this->url;
118			}
119
120			/**
121			* @param array $data
122			*
123			* @return User
124			*/
125			public static function fromApi(array $data): self
126			{
127			return new self(
128			$data['name'],
129			$data['country'] ?? null,
130			$data['age'] ? (int) $data['age'] : null,
131			$data['gender'] ?? null,
132			$data['playcount'] ? (int) $data['playcount'] : null,
133			$data['url'] ?? null
134			);
135			}
136			}
137

core23 / lastfm-php-api

GitHub Access Token became invalid

Push — master ( 547017...c67bb4 )

User::fromApi() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like