Artist::__construct() - Code Metrics - powered-by-moe/MusicBrainz - Measure and Improve Code Quality continuously with Scrutinizer

Artist::__construct() C
last analyzed 2016-10-21 09:13 UTC

↳ Parent: Artist

Complexity

Conditions	12
Paths	257

Size

Total Lines	18
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	18
rs	5.672
c	0
b	0
f	0
cc	12
eloc	13
nc	257
nop	2

How to fix Complexity

<?php

namespace MusicBrainz;

/**
 * Represents a MusicBrainz artist object
 * @package MusicBrainz
 */
class Artist
{
    /**
     * @var string
     */
    public $id;
    /**
     * @var string
     */
    public $name;
    /**
     * @var MusicBrainz
     */
    protected $brainz;
    /**
     * @var string
     */
    private $type;
    /**
     * @var string
     */
    private $sortName;
    /**
     * @var string
     */
    private $gender;
    /**
     * @var string
     */
    private $country;
    /**
     * @var string
     */
    private $beginDate;
    /**
     * @var string
     */
    private $endDate;
    /**
     * @var array
     */
    private $data;
    /**
     * @var array
     */
    private $releases;

    /**
     * @param array       $artist
     * @param MusicBrainz $brainz
     *
     * @throws Exception
     */
    public function __construct(array $artist, MusicBrainz $brainz)
    {
        if (!isset($artist['id']) || isset($artist['id']) && !$brainz->isValidMBID($artist['id'])) {
            throw new Exception('Can not create artist object. Missing valid MBID');
        }

        $this->data   = $artist;
        $this->brainz = $brainz;

        $this->id        = isset($artist['id']) ? (string)$artist['id'] : '';
        $this->type      = isset($artist['type']) ? (string)$artist['type'] : '';
        $this->name      = isset($artist['name']) ? (string)$artist['name'] : '';
        $this->sortName  = isset($artist['sort-name']) ? (string)$artist['sort-name'] : '';
        $this->gender    = isset($artist['gender']) ? (string)$artist['gender'] : '';
        $this->country   = isset($artist['country']) ? (string)$artist['country'] : '';
        $this->beginDate = isset($artist['life-span']['begin']) ? $artist['life-span']['begin'] : NULL;
        $this->endDate   = isset($artist['life-span']['ended']) ? $artist['life-span']['ended'] : NULL;
    }

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

    /**
     * @return int
     */
    public function getScore()
    {
        return isset($this->data['score']) ? (int)$this->data['score'] : 0;
    }

    /**
     * @return string
     */
    public function getType()
    {
        return $this->type;
    }

    /**
     * @return array
     */
    public function getReleases()
    {
        if (NULL === $this->releases) {
            $this->releases = $this->brainz->browseRelease('artist', $this->getId());
        }

        return $this->releases;
    }

    /**
     * @return string
     */
    public function getId()
    {
        return $this->id;
    }
}



1			<?php
2
3			namespace MusicBrainz;
4
5			/**
6			* Represents a MusicBrainz artist object
7			* @package MusicBrainz
8			*/
9			class Artist
10			{
11			/**
12			* @var string
13			*/
14			public $id;
15			/**
16			* @var string
17			*/
18			public $name;
19			/**
20			* @var MusicBrainz
21			*/
22			protected $brainz;
23			/**
24			* @var string
25			*/
26			private $type;
27			/**
28			* @var string
29			*/
30			private $sortName;
31			/**
32			* @var string
33			*/
34			private $gender;
35			/**
36			* @var string
37			*/
38			private $country;
39			/**
40			* @var string
41			*/
42			private $beginDate;
43			/**
44			* @var string
45			*/
46			private $endDate;
47			/**
48			* @var array
49			*/
50			private $data;
51			/**
52			* @var array
53			*/
54			private $releases;
55
56			/**
57			* @param array $artist
58			* @param MusicBrainz $brainz
59			*
60			* @throws Exception
61			*/
62			public function __construct(array $artist, MusicBrainz $brainz)
63			{
64			if (!isset($artist['id']) \|\| isset($artist['id']) && !$brainz->isValidMBID($artist['id'])) {
65			throw new Exception('Can not create artist object. Missing valid MBID');
66			}
67
68			$this->data = $artist;
69			$this->brainz = $brainz;
70
71			$this->id = isset($artist['id']) ? (string)$artist['id'] : '';
72			$this->type = isset($artist['type']) ? (string)$artist['type'] : '';
73			$this->name = isset($artist['name']) ? (string)$artist['name'] : '';
74			$this->sortName = isset($artist['sort-name']) ? (string)$artist['sort-name'] : '';
75			$this->gender = isset($artist['gender']) ? (string)$artist['gender'] : '';
76			$this->country = isset($artist['country']) ? (string)$artist['country'] : '';
77			$this->beginDate = isset($artist['life-span']['begin']) ? $artist['life-span']['begin'] : NULL;
78			$this->endDate = isset($artist['life-span']['ended']) ? $artist['life-span']['ended'] : NULL;
79			}
80
81			/**
82			* @return mixed
83			*/
84			public function getName()
85			{
86			return $this->name;
87			}
88
89			/**
90			* @return int
91			*/
92			public function getScore()
93			{
94			return isset($this->data['score']) ? (int)$this->data['score'] : 0;
95			}
96
97			/**
98			* @return string
99			*/
100			public function getType()
101			{
102			return $this->type;
103			}
104
105			/**
106			* @return array
107			*/
108			public function getReleases()
109			{
110			if (NULL === $this->releases) {
111			$this->releases = $this->brainz->browseRelease('artist', $this->getId());
112			}
113
114			return $this->releases;
115			}
116
117			/**
118			* @return string
119			*/
120			public function getId()
121			{
122			return $this->id;
123			}
124			}
125
126

powered-by-moe / MusicBrainz

Artist::__construct() C last analyzed 2016-10-21 09:13 UTC

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like

Artist::__construct() C
last analyzed 2016-10-21 09:13 UTC