Profile::getDetails() - Code Metrics - iPublikuj/500px - Measure and Improve Code Quality continuously with Scrutinizer

Profile::getDetails() C
last analyzed 2015-12-06 21:06 UTC

↳ Parent: Profile

Complexity

Conditions	11
Paths	30

Size

Total Lines	30
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	30
rs	5.2653
cc	11
eloc	15
nc	30
nop	1

How to fix Complexity

<?php
/**
 * Profile.php
 *
 * @copyright	More in license.md
 * @license		http://www.ipublikuj.eu
 * @author		Adam Kadlec http://www.ipublikuj.eu
 * @package		iPublikuj:500px!
 * @subpackage	common
 * @since		5.0
 *
 * @date		07.03.15
 */

namespace IPub\FiveHundredPixel;

use Nette;
use Nette\Utils;

use IPub;
use IPub\FiveHundredPixel\Exceptions;

/**
 * 500px's user profile
 *
 * @package		iPublikuj:500px!
 * @subpackage	common
 *
 * @author Adam Kadlec <[email protected]>
 */
class Profile extends Nette\Object
{
	/**
	 * @var Client
	 */
	private $fiveHundredPixel;

	/**
	 * @var string
	 */
	private $profileId;

	/**
	 * @var Utils\ArrayHash
	 */
	private $details;

	/**
	 * @param Client $fiveHundredPixel
	 * @param string $profileId
	 *
	 * @throws Exceptions\InvalidArgumentException
	 */
	public function __construct(Client $fiveHundredPixel, $profileId = NULL)
	{
		$this->fiveHundredPixel = $fiveHundredPixel;

		if (is_numeric($profileId)) {
			throw new Exceptions\InvalidArgumentException("ProfileId must be a username of the account you're trying to read or NULL, which means actually logged in user.");
		}

		$this->profileId = $profileId;
	}

	/**
	 * @return string
	 */
	public function getId()
	{
		if ($this->profileId === NULL) {
			return $this->fiveHundredPixel->getUser();
		}

		return $this->profileId;
	}

	/**
	 * @param string $key
	 *
	 * @return Utils\ArrayHash|NULL
	 */
	public function getDetails($key = NULL)
	{
		if ($this->details === NULL) {
			try {

				if ($this->profileId !== NULL) {
					if (($result = $this->fiveHundredPixel->get('users/show', ['username' => $this->profileId])) && ($result instanceof Utils\ArrayHash)) {
						$this->details = $result->user;
					}

				} else if ($this->fiveHundredPixel->getUser()) {
					if (($result = $this->fiveHundredPixel->get('users')) && ($result instanceof Utils\ArrayHash)) {
						$this->details = $result->user;
					}

				} else {
					$this->details = new Utils\ArrayHash;
				}

			} catch (\Exception $e) {
				// todo: log?
			}
		}

		if ($key !== NULL) {
			return isset($this->details[$key]) ? $this->details[$key] : NULL;
		}

		return $this->details;
	}
}

1			<?php
2			/**
3			* Profile.php
4			*
5			* @copyright More in license.md
6			* @license http://www.ipublikuj.eu
7			* @author Adam Kadlec http://www.ipublikuj.eu
8			* @package iPublikuj:500px!
9			* @subpackage common
10			* @since 5.0
11			*
12			* @date 07.03.15
13			*/
14
15			namespace IPub\FiveHundredPixel;
16
17			use Nette;
18			use Nette\Utils;
19
20			use IPub;
21			use IPub\FiveHundredPixel\Exceptions;
22
23			/**
24			* 500px's user profile
25			*
26			* @package iPublikuj:500px!
27			* @subpackage common
28			*
29			* @author Adam Kadlec <[email protected]>
30			*/
31			class Profile extends Nette\Object
32			{
33			/**
34			* @var Client
35			*/
36			private $fiveHundredPixel;
37
38			/**
39			* @var string
40			*/
41			private $profileId;
42
43			/**
44			* @var Utils\ArrayHash
45			*/
46			private $details;
47
48			/**
49			* @param Client $fiveHundredPixel
50			* @param string $profileId
51			*
52			* @throws Exceptions\InvalidArgumentException
53			*/
54			public function __construct(Client $fiveHundredPixel, $profileId = NULL)
55			{
56			$this->fiveHundredPixel = $fiveHundredPixel;
57
58			if (is_numeric($profileId)) {
59			throw new Exceptions\InvalidArgumentException("ProfileId must be a username of the account you're trying to read or NULL, which means actually logged in user.");
60			}
61
62			$this->profileId = $profileId;
63			}
64
65			/**
66			* @return string
67			*/
68			public function getId()
69			{
70			if ($this->profileId === NULL) {
71			return $this->fiveHundredPixel->getUser();
72			}
73
74			return $this->profileId;
75			}
76
77			/**
78			* @param string $key
79			*
80			* @return Utils\ArrayHash\|NULL
81			*/
82			public function getDetails($key = NULL)
83			{
84			if ($this->details === NULL) {
85			try {
86
87			if ($this->profileId !== NULL) {
88			if (($result = $this->fiveHundredPixel->get('users/show', ['username' => $this->profileId])) && ($result instanceof Utils\ArrayHash)) {
89			$this->details = $result->user;
90			}
91
92			} else if ($this->fiveHundredPixel->getUser()) {
93			if (($result = $this->fiveHundredPixel->get('users')) && ($result instanceof Utils\ArrayHash)) {
94			$this->details = $result->user;
95			}
96
97			} else {
98			$this->details = new Utils\ArrayHash;
99			}
100
101			} catch (\Exception $e) {
102			// todo: log?
103			}
104			}
105
106			if ($key !== NULL) {
107			return isset($this->details[$key]) ? $this->details[$key] : NULL;
108			}
109
110			return $this->details;
111			}
112			}

iPublikuj / 500px

Profile::getDetails() C last analyzed 2015-12-06 21:06 UTC

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like

Profile::getDetails() C
last analyzed 2015-12-06 21:06 UTC