Times::on() - Code Metrics - Inspection of "Minor optimizations and rephrases." - loophp/collection - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 4e0120...c40ba3 )

by Pol

created 2020-01-04 10:39 UTC

Times::on() A

↳ Parent: Times

Complexity

Conditions	4
Paths	1

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	10
nc	1
nop	1
dl	0
loc	18
rs	9.9332
c	1
b	0
f	0

<?php

declare(strict_types=1);

namespace loophp\collection\Operation;

use Closure;
use Generator;
use loophp\collection\Contract\Operation;

/**
 * Class Times.
 */
final class Times implements Operation
{
    /**
     * @var callable|null
     */
    private $callback;

    /**
     * @var int
     */
    private $number;

    /**
     * Times constructor.
     *
     * @param int $number
     * @param callable|null $callback
     */
    public function __construct(int $number, ?callable $callback = null)
    {
        $this->number = $number;
        $this->callback = $callback;
    }

    /**
     * {@inheritdoc}
     */
    public function on(iterable $collection): Closure
    {
        $number = $this->number;
        $callback = $this->callback;

        return static function () use ($number, $callback): Generator {
            if (1 > $number) {
                return;
            }

            if (null === $callback) {
                $callback = static function ($value) {
                    return $value;
                };
            }

            for ($current = 1; $current <= $number; ++$current) {
                yield $callback($current);
            }
        };
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace loophp\collection\Operation;
6
7			use Closure;
8			use Generator;
9			use loophp\collection\Contract\Operation;
10
11			/**
12			* Class Times.
13			*/
14			final class Times implements Operation
15			{
16			/**
17			* @var callable\|null
18			*/
19			private $callback;
20
21			/**
22			* @var int
23			*/
24			private $number;
25
26			/**
27			* Times constructor.
28			*
29			* @param int $number
30			* @param callable\|null $callback
31			*/
32			public function __construct(int $number, ?callable $callback = null)
33			{
34			$this->number = $number;
35			$this->callback = $callback;
36			}
37
38			/**
39			* {@inheritdoc}
40			*/
41			public function on(iterable $collection): Closure
42			{
43			$number = $this->number;
44			$callback = $this->callback;
45
46			return static function () use ($number, $callback): Generator {
47			if (1 > $number) {
48			return;
49			}
50
51			if (null === $callback) {
52			$callback = static function ($value) {
53			return $value;
54			};
55			}
56
57			for ($current = 1; $current <= $number; ++$current) {
58			yield $callback($current);
59			}
60			};
61			}
62			}
63

loophp / collection

Push — master ( 4e0120...c40ba3 )

Times::on() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like