drupol\phpermutations\Iterators\Rotation

Complexity

Total Complexity

8

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Coupling/Cohesion

Dependencies

1

Test Coverage

Coverage

78.95%

Importance

Changes

0

7 Methods

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

<?php

namespace drupol\phpermutations\Iterators;

use drupol\phpermutations\Combinatorics;

/**
 * Class Rotation.
 *
 * @package drupol\phpermutations\Iterators
 */
class Rotation extends Combinatorics implements \Iterator, \Countable {

  /**
   * The key.
   *
   * @var int
   */
  protected $key = 0;

  /**
   * A copy of the original data.
   *
   * @var mixed[]
   */
  protected $rotation;

  /**
   * Rotation constructor.
   *
   * @param array $dataset
   *   The dataset.
   */
  public function __construct(array $dataset = array()) {
    parent::__construct($dataset, NULL);
    $this->rotation = $this->getDataset();
  }

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

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

  /**
   * Compute the next value of the Iterator.
   *
   * @param int $offset
   *   The offset.
   */
  public function next($offset = 1) {
    $offset = is_null($offset) ? 1 : $offset % $this->count();

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

    $this->rotation = array_merge(array_slice($this->rotation, $offset), array_slice($this->rotation, 0, $offset));
  }

  /**
   * {@inheritdoc}
   */
  public function rewind() {
    $this->rotation = $this->getDataset();
  }

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

TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.

  }

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

}