drupol\phpermutations\Combinations

Complexity

Total Complexity

19

Size/Duplication

Total Lines	113
Duplicated Lines	0 %

Coupling/Cohesion

Dependencies

0

Test Coverage

Coverage

95.83%

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	2
A	key()	0	3	1
A	current()	0	6	2
A	next()	0	6	2
A	rewind()	0	4	1
A	valid()	0	3	1
B	_next()	0	15	5
A	toArray()	0	7	2
A	count()	0	3	1
A	getDataset()	0	3	1
A	getLength()	0	3	1

<?php

namespace drupol\phpermutations;

/**
 * Class Combinations.
 *
 * @package drupol\phpermutations
 */
class Combinations implements \Iterator, \Countable {
  protected $c = NULL;

Avoid variables with short names like $c. Configured minimum length is 3.

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

  protected $dataset = NULL;

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

  protected $count = 0;
  protected $length = 0;
  protected $pos = 0;

  /**
   * Combinations constructor.
   *
   * @param array $dataset
   * @param null $length
   */
  public function __construct(array $dataset = array(), $length = NULL) {

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

    $this->dataset = array_values($dataset);
    $this->count = count($this->dataset);
    $this->length = ($length > $this->count) ? $this->count : $length;
    $this->rewind();
  }

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

  /**
   * @return array
   */
  public function current() {
    $r = array();

Avoid variables with short names like $r. Configured minimum length is 3.

    for($i = 0; $i < $this->length; $i++)

Please always use braces to surround the code block of FOR statements.

Expected 1 space after FOR keyword; 0 found

It is generally a best practice to always use braces with control structures.

      $r[] = $this->dataset[$this->c[$i]];
    return $r;
  }

  /**
   *
   */
  public function next() {
    if ($this->_next())

Please always use braces to surround the code block of IF statements.

It is generally a best practice to always use braces with control structures.

      $this->pos++;
    else

Please always use braces to surround the code block of ELSE statements.

Expected 1 space after ELSE keyword; newline found

It is generally a best practice to always use braces with control structures.

      $this->pos = -1;
  }

  /**
   *
   */
  public function rewind() {
    $this->c = range(0, $this->length);
    $this->pos = 0;
  }

  /**
   * @return bool
   */
  public function valid() {
    return $this->pos >= 0;
  }

  /**
   * @return bool
   */
  protected function _next() {

The method _next is not named in camelCase.

    $i = $this->length - 1;
    while ($i >= 0 && $this->c[$i] == $this->count - $this->length + $i) {
      $i--;
    }
    if($i < 0) {

Expected 1 space after IF keyword; 0 found

      return FALSE;

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

    }
    $this->c[$i]++;
    while($i++ < $this->length - 1) {

Expected 1 space after WHILE keyword; 0 found

      $this->c[$i] = $this->c[$i - 1] + 1;
    }

    return TRUE;

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

  }

  /**
   * @return array
   */
  public function toArray() {
    $data = [];
    for($this->rewind(); $this->valid(); $this->next()) {

Expected 1 space after FOR keyword; 0 found

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

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

  /**
   * @return array|null
   */
  public function getDataset() {
    return $this->dataset;
  }

  /**
   * @return int|null
   */
  public function getLength() {
    return $this->length;
  }
}