Partition::getBounds() - Code Metrics - Inspection of "Start work on branch 1.x" - drupol/phpartition - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#2)

by Pol

created 2019-05-10 20:42 UTC

Partition::getBounds() A

↳ Parent: Partition

Complexity

Conditions	5
Paths	9

Size

Total Lines	24
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	13
nc	9
nop	0
dl	0
loc	24
rs	9.5222
c	0
b	0
f	0

<?php

declare(strict_types = 1);

namespace drupol\phpartition\Partition;

use drupol\phpartition\Contract\Valuable;
use drupol\phpartition\Contract\PartitionItem as PartitionItemInterface;
use drupol\phpartition\Contract\Partition as PartitionInterface;

/**
 * Class Partition.
 */
class Partition extends \ArrayObject implements PartitionInterface
{
    /**
     * @return array
     */
    public function exportArrayCopy()
    {
        return \array_map(
            static function (Valuable $item) {
                return $item->getValue();
            },
            $this->getArrayCopy()
        );
    }

    /**
     * {@inheritdoc}
     */
    public function getBounds(): array
    {
        $max = PHP_INT_MAX;
        $min = PHP_INT_MIN;

        $bounds = [];

        foreach ($this as $index => $item) {
            if ($item instanceof PartitionItemInterface) {
                $item = $item->getWeight();
            }

            if ($item < $max) {
                $max = $item;
                $bounds[0] = $index;
            }

            if ($item > $min) {
                $min = $item;
                $bounds[1] = $index;
            }
        }

        return $bounds;
    }

    /**
     * {@inheritdoc}
     */
    public function getWeight(): float
    {
        $weight = 0;

        foreach ($this as $item) {
            if ($item instanceof PartitionItemInterface) {
                $item = $item->getWeight();
            }

            $weight += $item;
        }

        return $weight;
    }
}


1			<?php
2
3			declare(strict_types = 1);
4
5			namespace drupol\phpartition\Partition;
6
7			use drupol\phpartition\Contract\Valuable;
8			use drupol\phpartition\Contract\PartitionItem as PartitionItemInterface;
9			use drupol\phpartition\Contract\Partition as PartitionInterface;
10
11			/**
12			* Class Partition.
13			*/
14			class Partition extends \ArrayObject implements PartitionInterface
15			{
16			/**
17			* @return array
18			*/
19			public function exportArrayCopy()
20			{
21			return \array_map(
22			static function (Valuable $item) {
23			return $item->getValue();
24			},
25			$this->getArrayCopy()
26			);
27			}
28
29			/**
30			* {@inheritdoc}
31			*/
32			public function getBounds(): array
33			{
34			$max = PHP_INT_MAX;
35			$min = PHP_INT_MIN;
36
37			$bounds = [];
38
39			foreach ($this as $index => $item) {
40			if ($item instanceof PartitionItemInterface) {
41			$item = $item->getWeight();
42			}
43
44			if ($item < $max) {
45			$max = $item;
46			$bounds[0] = $index;
47			}
48
49			if ($item > $min) {
50			$min = $item;
51			$bounds[1] = $index;
52			}
53			}
54
55			return $bounds;
56			}
57
58			/**
59			* {@inheritdoc}
60			*/
61			public function getWeight(): float
62			{
63			$weight = 0;
64
65			foreach ($this as $item) {
66			if ($item instanceof PartitionItemInterface) {
67			$item = $item->getWeight();
68			}
69
70			$weight += $item;
71			}
72
73			return $weight;
74			}
75			}
76

drupol / phpartition

Pull Request — master (#2)

Partition::getBounds() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like