Sequences::skippedSequences() - Code Metrics - Inspection of "Change namespaces" - cycle/proxy-factory - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( d3a94c...76e109 )

by Valentin

created 2019-03-04 13:40 UTC

Sequences::skippedSequences() A

↳ Parent: Sequences

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	11
rs	9.9
c	0
b	0
f	0
cc	3
nc	3
nop	1

<?php
declare(strict_types=1);

namespace Cycle\ORM\Promise\ConflictResolver;

class Sequences
{
    /**
     * Examples:
     * [], <any> => 0
     *
     *
     * @param array $sequences
     * @param int   $originSequence
     *
     * @return int
     */
    public function find(array $sequences, int $originSequence): int
    {
        if (empty($sequences) || $originSequence > max($sequences)) {
            return $originSequence;
        }

        $gaps = $this->skippedSequences($sequences);

        if (isset($gaps[$originSequence])) {
            return $originSequence;
        }

        //we do not add "1" as postfix: $var, $var2, $var3, etc
        unset($gaps[1]);
        if (empty($gaps)) {
            $max = max($sequences);
            if ($max === 0) {
                return 2;
            }

            return $max + 1;
        }

        return min($gaps);
    }

    private function skippedSequences(array $sequences): array
    {
        $skipped = [];
        for ($i = 0; $i < max($sequences); $i++) {
            if (!in_array($i, $sequences)) {
                $skipped[$i] = $i;
            }
        }

        return $skipped;
    }
}

1			<?php
2			declare(strict_types=1);
3
4			namespace Cycle\ORM\Promise\ConflictResolver;
5
6			class Sequences
7			{
8			/**
9			* Examples:
10			* [], <any> => 0
11			*
12			*
13			* @param array $sequences
14			* @param int $originSequence
15			*
16			* @return int
17			*/
18			public function find(array $sequences, int $originSequence): int
19			{
20			if (empty($sequences) \|\| $originSequence > max($sequences)) {
21			return $originSequence;
22			}
23
24			$gaps = $this->skippedSequences($sequences);
25
26			if (isset($gaps[$originSequence])) {
27			return $originSequence;
28			}
29
30			//we do not add "1" as postfix: $var, $var2, $var3, etc
31			unset($gaps[1]);
32			if (empty($gaps)) {
33			$max = max($sequences);
34			if ($max === 0) {
35			return 2;
36			}
37
38			return $max + 1;
39			}
40
41			return min($gaps);
42			}
43
44			private function skippedSequences(array $sequences): array
45			{
46			$skipped = [];
47			for ($i = 0; $i < max($sequences); $i++) {
48			if (!in_array($i, $sequences)) {
49			$skipped[$i] = $i;
50			}
51			}
52
53			return $skipped;
54			}
55			}

cycle / proxy-factory

Push — master ( d3a94c...76e109 )

Sequences::skippedSequences() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like