drupol\phpermutations\Iterators\Shift

Complexity

Total Complexity

12

Size/Duplication

Total Lines	117
Duplicated Lines	36.75 %

Coupling/Cohesion

Dependencies

1

Test Coverage

Coverage

52.27%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setLength()	7	7	3
A	key()	0	4	1
A	current()	0	4	1
A	next()	18	18	2
A	rewind()	18	18	2
A	valid()	0	4	1
A	count()	0	4	1

<?php

namespace drupol\phpermutations\Iterators;

use drupol\phpermutations\Combinatorics;

/**
 * Class Shift.
 */
class Shift extends Combinatorics implements \Iterator, \Countable
{
    /**
     * The key.
     *
     * @var int
     */
    protected $key = 0;

    /**
     * A copy of the dataset at a give time.
     *
     * @var array
     */
    protected $current;

    /**
     * Shift constructor.
     *
     * @param array $dataset
     *                       The dataset
     * @param int   $length
     *                       The shift length
     */
    public function __construct(array $dataset = [], $length = 1)
    {
        parent::__construct($dataset, $length);
        $this->current = $this->getDataset();
    }

    /**
     * {@inheritdoc}
     */
    public function setLength($length = null)

This method seems to be duplicated in your project.

    {
        $length = is_null($length) ? $this->datasetCount : $length;

Consider using a different name than the parameter $length. This often makes code more readable.

        $this->length = (abs($length) > $this->datasetCount) ? $this->datasetCount : $length;

        return $this;
    }

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

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

    /**
     * {@inheritdoc}
     */
    public function next()

This method seems to be duplicated in your project.

    {
        $length = $this->getLength();
        $parameters = [];

        if ($length < 0) {
            $parameters[] = ['start' => abs($length), 'end' => null];
            $parameters[] = ['start' => 0, 'end' => abs($length)];
        } else {
            $parameters[] = ['start' => -1 * $length, 'end' => null];
            $parameters[] = ['start' => 0, 'end' => $this->datasetCount + $length * -1];
        }

        $this->current = array_merge(
          array_slice($this->current, $parameters[0]['start'], $parameters[0]['end']),
          array_slice($this->current, $parameters[1]['start'], $parameters[1]['end'])
        );
    }

    /**
     * {@inheritdoc}
     */
    public function rewind()

This method seems to be duplicated in your project.

    {
        $length = -1 * $this->getLength();
        $parameters = [];

        if ($length < 0) {
            $parameters[] = ['start' => abs($length), 'end' => null];
            $parameters[] = ['start' => 0, 'end' => abs($length)];
        } else {
            $parameters[] = ['start' => -1 * $length, 'end' => null];
            $parameters[] = ['start' => 0, 'end' => $this->datasetCount + $length * -1];
        }

        $this->current = array_merge(
          array_slice($this->current, $parameters[0]['start'], $parameters[0]['end']),
          array_slice($this->current, $parameters[1]['start'], $parameters[1]['end'])
      );
    }

    /**
     * {@inheritdoc}
     */
    public function valid()
    {
        return true;
    }

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