drupol\phpermutations\Iterators\Product

Complexity

Total Complexity

19

Size/Duplication

Total Lines	136
Duplicated Lines	6.62 %

Coupling/Cohesion

Dependencies

1

Test Coverage

Coverage

95.56%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	key()	0	4	1
A	__construct()	0	10	2
A	current()	0	10	2
B	next()	0	17	5
A	valid()	0	12	3
A	rewind()	0	8	2
A	count()	0	10	2
A	toArray()	9	10	2

<?php

namespace drupol\phpermutations\Iterators;

use drupol\phpermutations\Combinatorics;

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

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

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

  /**
   * Product constructor.
   *
   * @param array $datasetArray
   *   The array of dataset.
   */
    public function __construct(array $datasetArray)
    {
        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()

The variable $count_iterators is not named in camelCase.

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

        $this->key++;
    }

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

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

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

        return $isUnlessOneValid;
    }

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

        $this->key = 0;
    }

  /**
   * {@inheritdoc}
   */
    public function count()
    {
        $product = 1;

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

        return $product;
    }

  /**
   * Convert the iterator into an array.
   *
   * @return array
   *   The elements.
   */
    public function toArray()

This method seems to be duplicated in your project.

    {
        $data = array();

        for ($this->rewind(); $this->valid(); $this->next()) {
            $data[] = $this->current();
        }

        return $data;
    }
}