Circular::convert() - Code Metrics - Inspection of "coverage" - alecrabbit/php-accessories - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( e241fd...061b3c )

by Alec

created 2019-02-12 12:50 UTC

Circular::convert() A

↳ Parent: Circular

Complexity

Conditions	4
Paths	4

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	4

Importance

Changes

Metric	Value
cc	4
eloc	11
nc	4
nop	1
dl	0
loc	18
ccs	10
cts	10
cp	1
crap	4
rs	9.9
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace AlecRabbit\Accessories;

use function AlecRabbit\typeOf;

/**
 * Class Circular
 */
class Circular implements \Iterator
{
    /** @var Rewindable */
    protected $data;

    /**
     * Circular constructor.
     * @param array|callable|Rewindable $data
     */
    public function __construct($data)
    {
        $this->data = $this->convert($data);
    }

    /**
     * @param mixed $data
     * @return Rewindable
     */
    private function convert(&$data): Rewindable
    {
        if (\is_array($data)) {
            return
                new Rewindable(
                    function () use (&$data): \Generator {
                        yield from $data;
                    }
                );
        }
        if (\is_callable($data)) {
            return
                new Rewindable($data);
        }
        if ($data instanceof Rewindable) {
            return $data;
        }
        throw new \InvalidArgumentException('Unexpected argument type: ' . typeOf($data) . ' for ' . Caller::get());
    }

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

    /**
     * @return mixed
     */
    public function value()
    {
        if (null === $value = $this->current()) {
            $this->rewind();
            $value = $this->current();
        }
        $this->next();
        return $value;
//        if (!$this->valid()) {
//            $this->rewind();
//        } else {
//            $this->next();
//        }
//        return $this->current();
    }

    /**
     * {@inheritdoc}
     */
    public function current()
    {
        return $this->data->current();
    }

    /**
     * {@inheritdoc}
     */
    public function rewind(): void
    {
        $this->data->rewind();
    }

    /**
     * {@inheritdoc}
     */
    public function next(): void
    {
        $this->data->next();
    }

    /**
     * {@inheritdoc}
     */
    public function valid(): bool
    {
        return
            $this->data->valid();
    }

    /**
     * {@inheritdoc}
     */
    public function key()
    {
        return $this->data->key();
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace AlecRabbit\Accessories;
6
7		use function AlecRabbit\typeOf;
8
9		/**
10		* Class Circular
11		*/
12		class Circular implements \Iterator
13		{
14		/** @var Rewindable */
15		protected $data;
16
17		/**
18		* Circular constructor.
19		* @param array\|callable\|Rewindable $data
20		*/
21	8	public function __construct($data)
22		{
23	8	$this->data = $this->convert($data);
24	6	}
25
26		/**
27		* @param mixed $data
28		* @return Rewindable
29		*/
30	8	private function convert(&$data): Rewindable
31		{
32	8	if (\is_array($data)) {
33		return
34	4	new Rewindable(
35		function () use (&$data): \Generator {
36	4	yield from $data;
37	4	}
38		);
39		}
40	4	if (\is_callable($data)) {
41		return
42	2	new Rewindable($data);
43		}
44	2	if ($data instanceof Rewindable) {
45	1	return $data;
46		}
47	1	throw new \InvalidArgumentException('Unexpected argument type: ' . typeOf($data) . ' for ' . Caller::get());
48		}
49
50		/**
51		* @return mixed
52		*/
53	1	public function __invoke()
54		{
55	1	return $this->value();
56		}
57
58		/**
59		* @return mixed
60		*/
61	2	public function value()
62		{
63	2	if (null === $value = $this->current()) {
64	2	$this->rewind();
65	2	$value = $this->current();
66		}
67	2	$this->next();
68	2	return $value;
69		// if (!$this->valid()) {
70		// $this->rewind();
71		// } else {
72		// $this->next();
73		// }
74		// return $this->current();
75		}
76
77		/**
78		* {@inheritdoc}
79		*/
80	6	public function current()
81		{
82	6	return $this->data->current();
83		}
84
85		/**
86		* {@inheritdoc}
87		*/
88	6	public function rewind(): void
89		{
90	6	$this->data->rewind();
91	6	}
92
93		/**
94		* {@inheritdoc}
95		*/
96	6	public function next(): void
97		{
98	6	$this->data->next();
99	6	}
100
101		/**
102		* {@inheritdoc}
103		*/
104	4	public function valid(): bool
105		{
106		return
107	4	$this->data->valid();
108		}
109
110		/**
111		* {@inheritdoc}
112		*/
113	4	public function key()
114		{
115	4	return $this->data->key();
116		}
117		}
118

alecrabbit / php-accessories

Push — master ( e241fd...061b3c )

Circular::convert() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like