Participant::createFromArray() - Code Metrics - Inspection of "Feature/collection helper methods" - PtrTn/battlerite-sdk - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#3)

by Peter

created 2017-12-16 21:09 UTC

Participant::createFromArray() A

↳ Parent: Participant

Complexity

Conditions	2
Paths	2

Size

Total Lines	59
Code Lines	52

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	50
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	59
ccs	50
cts	51
cp	0.9804
rs	9.597
c	0
b	0
f	0
cc	2
eloc	52
nc	2
nop	1
crap	2

How to fix Long Method

<?php

namespace PtrTn\Battlerite\Dto\Match;

use Webmozart\Assert\Assert;

/**
 * @SuppressWarnings(PHPMD.TooManyFields)
 */
class Participant
{
    /**
     * @var string
     */
    public $type;

    /**
     * @var string
     */
    public $id;

    /**
     * @var string
     */
    public $actor;

    /**
     * @var string
     */
    public $shardId;

    /**
     * @var int
     */
    public $abilityUses;

    /**
     * @var int
     */
    public $attachment;

    /**
     * @var int
     */
    public $damageDone;

    /**
     * @var int
     */
    public $damageReceived;

    /**
     * @var int
     */
    public $deaths;

    /**
     * @var int
     */
    public $disablesDone;

    /**
     * @var int
     */
    public $disablesReceived;

    /**
     * @var int
     */
    public $emote;

    /**
     * @var int
     */
    public $energyGained;

    /**
     * @var int
     */
    public $energyUsed;

    /**
     * @var int
     */
    public $healingDone;

    /**
     * @var int
     */
    public $healingReceived;

    /**
     * @var int
     */
    public $kills;

    /**
     * @var int
     */
    public $mount;

    /**
     * @var int
     */
    public $outfit;

    /**
     * @var int
     */
    public $score;

    /**
     * @var int
     */
    public $side;

    /**
     * @var int
     */
    public $timeAlive;

    /**
     * @var string
     */
    public $userID;

    /**
     * @SuppressWarnings(PHPMD.ExcessiveParameterList)
     */
    private function __construct(
        string $type,
        string $id,
        string $actor,
        string $shardId,
        int $abilityUses,
        int $attachment,
        int $damageDone,
        int $damageReceived,
        int $deaths,
        int $disablesDone,
        int $disablesReceived,
        int $emote,
        int $energyGained,
        int $energyUsed,
        int $healingDone,
        int $healingReceived,
        int $kills,
        int $mount,
        int $outfit,
        int $score,
        int $side,
        int $timeAlive,
        string $userID
    ) {
        $this->type = $type;
        $this->id = $id;
        $this->actor = $actor;
        $this->shardId = $shardId;
        $this->abilityUses = $abilityUses;
        $this->attachment = $attachment;
        $this->damageDone = $damageDone;
        $this->damageReceived = $damageReceived;
        $this->deaths = $deaths;
        $this->disablesDone = $disablesDone;
        $this->disablesReceived = $disablesReceived;
        $this->emote = $emote;
        $this->energyGained = $energyGained;
        $this->energyUsed = $energyUsed;
        $this->healingDone = $healingDone;
        $this->healingReceived = $healingReceived;
        $this->kills = $kills;
        $this->mount = $mount;
        $this->outfit = $outfit;
        $this->score = $score;
        $this->side = $side;
        $this->timeAlive = $timeAlive;
        $this->userID = $userID;
    }

    public static function createFromArray(array $participant): self
    {
        Assert::string($participant['type']);
        Assert::string($participant['id']);
        Assert::string($participant['attributes']['actor']);
        Assert::string($participant['attributes']['shardId']);
        Assert::integer($participant['attributes']['stats']['abilityUses']);
        Assert::integer($participant['attributes']['stats']['attachment']);
        Assert::integer($participant['attributes']['stats']['damageDone']);
        Assert::integer($participant['attributes']['stats']['damageReceived']);
        Assert::integer($participant['attributes']['stats']['deaths']);
        Assert::integer($participant['attributes']['stats']['disablesDone']);
        Assert::integer($participant['attributes']['stats']['disablesReceived']);
        Assert::integer($participant['attributes']['stats']['emote']);
        Assert::integer($participant['attributes']['stats']['energyGained']);
        Assert::integer($participant['attributes']['stats']['energyUsed']);
        Assert::integer($participant['attributes']['stats']['healingDone']);
        Assert::integer($participant['attributes']['stats']['healingReceived']);
        Assert::integer($participant['attributes']['stats']['kills']);
        Assert::integer($participant['attributes']['stats']['mount']);
        Assert::integer($participant['attributes']['stats']['outfit']);
        Assert::integer($participant['attributes']['stats']['score']);
        Assert::integer($participant['attributes']['stats']['side']);
        Assert::integer($participant['attributes']['stats']['timeAlive']);

        if (isset($participant['attributes']['stats']['userID'])) {
            $userId = $participant['attributes']['stats']['userID'];
        }
        else {
            $userId = $participant['relationships']['player']['data']['id'];
        }
        Assert::string($userId);

        return new self(
            $participant['type'],
            $participant['id'],
            $participant['attributes']['actor'],
            $participant['attributes']['shardId'],
            $participant['attributes']['stats']['abilityUses'],
            $participant['attributes']['stats']['attachment'],
            $participant['attributes']['stats']['damageDone'],
            $participant['attributes']['stats']['damageReceived'],
            $participant['attributes']['stats']['deaths'],
            $participant['attributes']['stats']['disablesDone'],
            $participant['attributes']['stats']['disablesReceived'],
            $participant['attributes']['stats']['emote'],
            $participant['attributes']['stats']['energyGained'],
            $participant['attributes']['stats']['energyUsed'],
            $participant['attributes']['stats']['healingDone'],
            $participant['attributes']['stats']['healingReceived'],
            $participant['attributes']['stats']['kills'],
            $participant['attributes']['stats']['mount'],
            $participant['attributes']['stats']['outfit'],
            $participant['attributes']['stats']['score'],
            $participant['attributes']['stats']['side'],
            $participant['attributes']['stats']['timeAlive'],
            $userId
        );
    }
}


1		<?php
2
3		namespace PtrTn\Battlerite\Dto\Match;
4
5		use Webmozart\Assert\Assert;
6
7		/**
8		* @SuppressWarnings(PHPMD.TooManyFields)
9		*/
10		class Participant
11		{
12		/**
13		* @var string
14		*/
15		public $type;
16
17		/**
18		* @var string
19		*/
20		public $id;
21
22		/**
23		* @var string
24		*/
25		public $actor;
26
27		/**
28		* @var string
29		*/
30		public $shardId;
31
32		/**
33		* @var int
34		*/
35		public $abilityUses;
36
37		/**
38		* @var int
39		*/
40		public $attachment;
41
42		/**
43		* @var int
44		*/
45		public $damageDone;
46
47		/**
48		* @var int
49		*/
50		public $damageReceived;
51
52		/**
53		* @var int
54		*/
55		public $deaths;
56
57		/**
58		* @var int
59		*/
60		public $disablesDone;
61
62		/**
63		* @var int
64		*/
65		public $disablesReceived;
66
67		/**
68		* @var int
69		*/
70		public $emote;
71
72		/**
73		* @var int
74		*/
75		public $energyGained;
76
77		/**
78		* @var int
79		*/
80		public $energyUsed;
81
82		/**
83		* @var int
84		*/
85		public $healingDone;
86
87		/**
88		* @var int
89		*/
90		public $healingReceived;
91
92		/**
93		* @var int
94		*/
95		public $kills;
96
97		/**
98		* @var int
99		*/
100		public $mount;
101
102		/**
103		* @var int
104		*/
105		public $outfit;
106
107		/**
108		* @var int
109		*/
110		public $score;
111
112		/**
113		* @var int
114		*/
115		public $side;
116
117		/**
118		* @var int
119		*/
120		public $timeAlive;
121
122		/**
123		* @var string
124		*/
125		public $userID;
126
127		/**
128		* @SuppressWarnings(PHPMD.ExcessiveParameterList)
129		*/
130	4	private function __construct(
131		string $type,
132		string $id,
133		string $actor,
134		string $shardId,
135		int $abilityUses,
136		int $attachment,
137		int $damageDone,
138		int $damageReceived,
139		int $deaths,
140		int $disablesDone,
141		int $disablesReceived,
142		int $emote,
143		int $energyGained,
144		int $energyUsed,
145		int $healingDone,
146		int $healingReceived,
147		int $kills,
148		int $mount,
149		int $outfit,
150		int $score,
151		int $side,
152		int $timeAlive,
153		string $userID
154		) {
155	4	$this->type = $type;
156	4	$this->id = $id;
157	4	$this->actor = $actor;
158	4	$this->shardId = $shardId;
159	4	$this->abilityUses = $abilityUses;
160	4	$this->attachment = $attachment;
161	4	$this->damageDone = $damageDone;
162	4	$this->damageReceived = $damageReceived;
163	4	$this->deaths = $deaths;
164	4	$this->disablesDone = $disablesDone;
165	4	$this->disablesReceived = $disablesReceived;
166	4	$this->emote = $emote;
167	4	$this->energyGained = $energyGained;
168	4	$this->energyUsed = $energyUsed;
169	4	$this->healingDone = $healingDone;
170	4	$this->healingReceived = $healingReceived;
171	4	$this->kills = $kills;
172	4	$this->mount = $mount;
173	4	$this->outfit = $outfit;
174	4	$this->score = $score;
175	4	$this->side = $side;
176	4	$this->timeAlive = $timeAlive;
177	4	$this->userID = $userID;
178	4	}
179
180	4	public static function createFromArray(array $participant): self
181		{
182	4	Assert::string($participant['type']);
183	4	Assert::string($participant['id']);
184	4	Assert::string($participant['attributes']['actor']);
185	4	Assert::string($participant['attributes']['shardId']);
186	4	Assert::integer($participant['attributes']['stats']['abilityUses']);
187	4	Assert::integer($participant['attributes']['stats']['attachment']);
188	4	Assert::integer($participant['attributes']['stats']['damageDone']);
189	4	Assert::integer($participant['attributes']['stats']['damageReceived']);
190	4	Assert::integer($participant['attributes']['stats']['deaths']);
191	4	Assert::integer($participant['attributes']['stats']['disablesDone']);
192	4	Assert::integer($participant['attributes']['stats']['disablesReceived']);
193	4	Assert::integer($participant['attributes']['stats']['emote']);
194	4	Assert::integer($participant['attributes']['stats']['energyGained']);
195	4	Assert::integer($participant['attributes']['stats']['energyUsed']);
196	4	Assert::integer($participant['attributes']['stats']['healingDone']);
197	4	Assert::integer($participant['attributes']['stats']['healingReceived']);
198	4	Assert::integer($participant['attributes']['stats']['kills']);
199	4	Assert::integer($participant['attributes']['stats']['mount']);
200	4	Assert::integer($participant['attributes']['stats']['outfit']);
201	4	Assert::integer($participant['attributes']['stats']['score']);
202	4	Assert::integer($participant['attributes']['stats']['side']);
203	4	Assert::integer($participant['attributes']['stats']['timeAlive']);
204
205	4	if (isset($participant['attributes']['stats']['userID'])) {
206	4	$userId = $participant['attributes']['stats']['userID'];
207		}
208		else {
209		$userId = $participant['relationships']['player']['data']['id'];
210		}
211	4	Assert::string($userId);
212
213	4	return new self(
214	4	$participant['type'],
215	4	$participant['id'],
216	4	$participant['attributes']['actor'],
217	4	$participant['attributes']['shardId'],
218	4	$participant['attributes']['stats']['abilityUses'],
219	4	$participant['attributes']['stats']['attachment'],
220	4	$participant['attributes']['stats']['damageDone'],
221	4	$participant['attributes']['stats']['damageReceived'],
222	4	$participant['attributes']['stats']['deaths'],
223	4	$participant['attributes']['stats']['disablesDone'],
224	4	$participant['attributes']['stats']['disablesReceived'],
225	4	$participant['attributes']['stats']['emote'],
226	4	$participant['attributes']['stats']['energyGained'],
227	4	$participant['attributes']['stats']['energyUsed'],
228	4	$participant['attributes']['stats']['healingDone'],
229	4	$participant['attributes']['stats']['healingReceived'],
230	4	$participant['attributes']['stats']['kills'],
231	4	$participant['attributes']['stats']['mount'],
232	4	$participant['attributes']['stats']['outfit'],
233	4	$participant['attributes']['stats']['score'],
234	4	$participant['attributes']['stats']['side'],
235	4	$participant['attributes']['stats']['timeAlive'],
236	4	$userId
237		);
238		}
239		}
240

PtrTn / battlerite-sdk

Pull Request — master (#3)

Participant::createFromArray() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like