drupol\phpartition\PartitionItem

Complexity

Total Complexity

6

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Dependencies

0

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getValue()	0	12	4
A	getItem()	0	4	1

<?php

namespace drupol\phpartition;

/**
 * Class PartitionItem.
 *
 * @package drupol\phpartition
 */
class PartitionItem implements PartitionItemInterface
{

    protected $item;
    protected $valueOrCallable;

  /**
   * PartitionItem constructor.
   *
   * @param mixed $item
   *   The item.
   * @param float|callable $valueOrCallable

Should the type for parameter $valueOrCallable not be double|callable|null? Also, consider making the array more specific, something like array<String>, or String[].

   *   A callable that will get it's value or a value.
   */
    public function __construct($item, $valueOrCallable = null)
    {
        $this->item = $item;
        $this->valueOrCallable = $valueOrCallable;
    }

  /**
   * {@inheritdoc}
   */
    public function getValue()
    {
        if (is_callable($this->valueOrCallable)) {
            return call_user_func($this->valueOrCallable, $this->item);
        }

        if (!is_null($this->valueOrCallable)) {
            return $this->valueOrCallable;
        }

        return is_numeric($this->item) ? $this->item : 0;
    }

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