ScrobbeBuilder::getQuery() - Code Metrics - Inspection of "[MAJOR] Use Builder when calling LastFm API" - core23/lastfm-php-api - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 981090...e30cbf )

by Christian

created 2019-04-06 00:07 UTC

ScrobbeBuilder::getQuery() A

↳ Parent: ScrobbeBuilder

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	16
rs	9.7333
c	0
b	0
f	0
cc	3
nc	3
nop	0

<?php

/*
 * (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\Builder;

final class ScrobbeBuilder
{
    /**
     * @var array
     */
    private $tracks;

    private function __construct()
    {
    }

    /**
     * @return ScrobbeBuilder
     */
    public function create(): self
    {
        $this->tracks = [];
    }

    /**
     * @param TrackBuilder $builder
     *
     * @return ScrobbeBuilder
     */
    public function addTrack(TrackBuilder $builder): self
    {
        $this->tracks[] = $builder->getData();
    }

    /**
     * @return int
     */
    public function count(): int
    {
        return \count($this->tracks);
    }

    /**
     * @return array
     */
    public function getQuery(): array
    {
        $query = [];

        $tracks = $this->tracks;

        foreach ($tracks as $i => $track) {
            $keys = array_keys($track);

            foreach ($keys as $key) {
                $query[sprintf('%s[%s]', $key, $i)] = $tracks[$i][$key];
            }
        }

        return $query;
    }
}


1			<?php
2
3			/*
4			* (c) Christian Gripp <[email protected]>
5			*
6			* For the full copyright and license information, please view the LICENSE
7			* file that was distributed with this source code.
8			*/
9
10			namespace Core23\LastFm\Builder;
11
12			final class ScrobbeBuilder
13			{
14			/**
15			* @var array
16			*/
17			private $tracks;
18
19			private function __construct()
20			{
21			}
22
23			/**
24			* @return ScrobbeBuilder
25			*/
26			public function create(): self
27			{
28			$this->tracks = [];
29			}
30
31			/**
32			* @param TrackBuilder $builder
33			*
34			* @return ScrobbeBuilder
35			*/
36			public function addTrack(TrackBuilder $builder): self
37			{
38			$this->tracks[] = $builder->getData();
39			}
40
41			/**
42			* @return int
43			*/
44			public function count(): int
45			{
46			return \count($this->tracks);
47			}
48
49			/**
50			* @return array
51			*/
52			public function getQuery(): array
53			{
54			$query = [];
55
56			$tracks = $this->tracks;
57
58			foreach ($tracks as $i => $track) {
59			$keys = array_keys($track);
60
61			foreach ($keys as $key) {
62			$query[sprintf('%s[%s]', $key, $i)] = $tracks[$i][$key];
63			}
64			}
65
66			return $query;
67			}
68			}
69

core23 / lastfm-php-api

GitHub Access Token became invalid

Push — master ( 981090...e30cbf )

ScrobbeBuilder::getQuery() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like