drupol\phpermutations\Product

Complexity

Total Complexity

19

Size/Duplication

Total Lines	123
Duplicated Lines	7.32 %

Coupling/Cohesion

Dependencies

1

Test Coverage

Coverage

96.36%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	2
A	current()	0	9	2
B	next()	0	18	5
A	key()	0	3	1
A	valid()	0	11	3
A	rewind()	0	7	2
A	count()	0	9	2
A	toArray()	9	9	2

<?php

namespace drupol\phpermutations;

/**
 * Class Product.
 *
 * @package drupol\phpermutations
 */
class Product extends Combinatorics implements \Iterator {

  /**
   * @var int
   */
  protected $key;

  /**
   * @var \Iterator[]
   */
  protected $iterators = array();

  /**
   * Product constructor.
   *
   * @param array $datasetArray
   * @param int|null $length
   */
  public function __construct(array $datasetArray, $length = NULL) {

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

    parent::__construct($datasetArray);

    foreach ($datasetArray as $dataset) {
      $this->iterators[] = new \ArrayIterator($dataset);
    }

    $this->key = 0;
  }

  /**
   * @inheritdoc
   */
  public function current() {
    $tuple = array();

    foreach ($this->iterators as $iterator) {
      $tuple[] = $iterator->current();
    }

    return $tuple;
  }

  /**
   * @inheritdoc
   */
  public function next() {
    /** @var $reverseIterators \Iterator[] */
    $reverseIterators = array_reverse($this->iterators);

    foreach ($reverseIterators as $key => $iterator) {
      $iterator->next();
      if ($iterator->valid()) {
        foreach ($this->iterators as $key2 => $iterator2) {
          if ($key >= count($this->iterators) - $key2) {
            $iterator2->rewind();
          }
        }
        break;
      }
    }

    $this->key++;
  }

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

  /**
   * @inheritdoc
   */
  public function valid() {
    $isUnlessOneValid = FALSE;

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

    foreach ($this->iterators as $iterator) {
      if ($iterator->valid()) {
        $isUnlessOneValid = TRUE;

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

      }
    }

    return $isUnlessOneValid;
  }

  /**
   * @inheritdoc
   */
  public function rewind() {
    foreach ($this->iterators as $iterator) {
      $iterator->rewind();
    }

    $this->key = 0;
  }

  /**
   * @return int
   */
  public function count() {
    $product = 1;

    foreach ($this->getDataset() as $dataset) {
      $product *= count($dataset);
    }

    return $product;
  }

  /**
   * @return array
   */
  public function toArray() {

This method seems to be duplicated in your project.

    $data = array();

    for($this->rewind(); $this->valid(); $this->next()) {

Expected 1 space after FOR keyword; 0 found

      $data[] = $this->current();
    }

    return $data;
  }

}