chillerlan\PrototypeDOM\Traits\EnumerableTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	each()	0	5	1
A	first()	0	3	1
A	last()	0	3	1
A	map()	0	14	3
A	pluck()	0	3	1
A	reverse()	0	6	1
A	toArray()	0	3	1

<?php
/**
 *
 * @filesource   EnumerableTrait.php
 * @created      11.05.2017
 * @package      chillerlan\PrototypeDOM\Traits
 * @author       Smiley <smiley@chillerlan.net>
 * @copyright    2017 Smiley
 * @license      MIT
 */

namespace chillerlan\PrototypeDOM\Traits;

/**
 * Trait EnumerableTrait
 */
trait EnumerableTrait{

	/**
	 * @link http://api.prototypejs.org/language/Enumerable/prototype/each/
	 *
	 * @param callable $callback
	 *
	 * @return $this
	 */
	public function each($callback){
		$this->map($callback);

		return $this;
	}

	/**
	 * @link http://api.prototypejs.org/language/Array/prototype/first/
	 *
	 * @return \chillerlan\PrototypeDOM\Node\Element|null
	 */
	public function first(){
		return $this->item(0);

It seems like item() must be provided by classes using this trait. How about adding it as abstract method to this trait?

	}

	/**
	 * @link http://api.prototypejs.org/language/Array/prototype/last/
	 *
	 * @return \chillerlan\PrototypeDOM\Node\Element|null
	 */
	public function last(){
		return $this->item($this->count() - 1);

It seems like count() must be provided by classes using this trait. How about adding it as abstract method to this trait?

It seems like item() must be provided by classes using this trait. How about adding it as abstract method to this trait?

	}

	/**
	 * @link http://api.prototypejs.org/language/Enumerable/prototype/collect/
	 * @link http://api.prototypejs.org/language/Enumerable/prototype/map/
	 *
	 * @param $callback
	 *
	 * @return array
	 * @throws \Exception
	 */
	public function map($callback):array {

		if(!is_callable($callback)){
			throw new \Exception('invalid callback'); // @codeCoverageIgnore
		}

		$return = [];

		foreach($this->array as $index => $element){

The property array does not exist. Did you maybe forget to declare it?

			$return[$index] = call_user_func_array($callback, [$element, $index]);
		}

		return $return;
	}

	/**
	 * @link http://api.prototypejs.org/language/Enumerable/prototype/pluck/
	 *
	 * @param string $property
	 *
	 * @return array
	 */
	public function pluck(string $property):array {
		return array_column($this->array, $property);
	}

	/**
	 * @link http://api.prototypejs.org/language/Array/prototype/reverse/
	 *
	 * @return $this
	 */
	public function reverse(){
		$this->array = array_reverse($this->array);
		$this->rewind();

It seems like rewind() must be provided by classes using this trait. How about adding it as abstract method to this trait?

		return $this;
	}

	/**
	 * @link http://api.prototypejs.org/language/Enumerable/prototype/toArray/
	 *
	 * @return array
	 */
	public function toArray():array {
		return $this->array;
	}

}