Jmw\Collection\CollectionAbstract

Complexity

Total Complexity

16

Size/Duplication

Total Lines	168
Duplicated Lines	16.67 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	addAll()	13	13	2
A	contains()	0	13	3
A	containsAll()	0	14	3
A	equals()	0	8	2
A	isEmpty()	0	4	1
A	retainAll()	15	15	3
A	toJson()	0	9	2
	add()	0	1	?
	clear()	0	1	?
	iterator()	0	1	?
	remove()	0	1	?
	removeAll()	0	1	?
	size()	0	1	?
	toArray()	0	1	?
	hashCode()	0	1	?

<?php
namespace Jmw\Collection;

/**
 * This class provides a skeletal implementation of the Collection interface, 
 * to minimize the effort required to implement this interface.
 * 
 * To implement an unmodifiable collection, the programmer needs only to extend this
 * class and provide implementations for the iterator and size methods, and use the ImmutableCollectionTrait trait
 * (The iterator returned by the iterator method must implement hasNext and next.)
 * @author john
 *
 */
abstract class CollectionAbstract implements CollectionInterface
{	

The opening class brace should be on a newline by itself.

	/**
	 * Adds all of the elements in the specified collection to this collection (optional operation).
	 * @param CollectionInterface $collection
	 * @return boolean $changed
	*/
	public function addAll(CollectionInterface $collection)

This method seems to be duplicated in your project.

	{
		$iterator = $collection->iterator();
		$changed = false;
		
		while($iterator->hasNext())
		{
			$changed = true;
			$this->add($iterator->next());
		}
		
		return $changed;		
	}
	
	/**
	 * Returns true if this collection contains the specified element.
	 * @return boolean
	*/
	public function contains($element)
	{
		$iterator = $this->iterator();
		
		while($iterator->hasNext())
		{
			if($element === $iterator->next())
			{
				return true;
			}
		}
		return false;
	}
	
	/**
	 * Returns true if this collection contains all of the elements in the specified collection.
	 * @param Collection $collection
	 * @return boolean
	*/
	public function containsAll(CollectionInterface $collection)
	{
		$iterator = $collection->iterator();
		
		while($iterator->hasNext())
		{
			if(!$this->contains($iterator->next()))
			{
				return false;
			}
		}
		
		return true;
	}
	
	/**
	 * Compares the specified object with this collection for equality.
	 * @param unknown $object
	 * @return boolean
	*/
	public function equals($object)
	{
		if(!$object instanceof CollectionInterface)
		{
			return false;
		}
		return $this->hashCode() === $object->hashCode();
	}
	
	/**
	 * Returns true if this collection contains no elements.
	 * @return boolean
	*/
	public function isEmpty()
	{
		return $this->size() === 0;
	}
	
	/**
	 * Retains only the elements in this collection that are contained in the specified collection (optional operation).
	 * @param CollectionInterface $collection
	 * @return boolean
	*/
	public function retainAll(CollectionInterface $collection)

This method seems to be duplicated in your project.

	{
		$iterator = $this->iterator();
		$changed = false;
		
		while($iterator->hasNext())
		{
			if(!$collection->contains($iterator->next()))
			{
				$iterator->remove();
				$changed = true;
			}
		}
		return $changed;
	}

	/**
	 * Returns a json string representing all the elements in this collection
	 * @return string
	 */
	public function toJson()
	{
		$json = json_encode($this->toArray());
		if($error = json_last_error())
		{
			throw new \Exception('Error encoding to json: ' . $error);
		}
		return $json;
	}
	
	/**
	 * Ensures that this collection contains the specified element (optional operation).
	 * @param multitype $element
	 * @return boolean
	 */
	public abstract function add($element);

The abstract declaration must precede the visibility declaration

	
	/**
	 * Removes all of the elements from this collection (optional operation).
	 * @return void
	 */
	public abstract function clear();

The abstract declaration must precede the visibility declaration

	/**
	 * Returns an iterator over the elements in this collection.
	 * @return IteratorInterface
	 */
	public abstract function iterator();

The abstract declaration must precede the visibility declaration

	
	/**
	 * Removes a single instance of the specified element from this collection, if it is present (optional operation).
	 * @param multitype $element
	 * @return boolean
	*/
	public abstract function remove($element);

The abstract declaration must precede the visibility declaration

	
	/**
	 * Removes all of this collection's elements that are also contained in the specified collection (optional operation).
	 * @param CollectionInterface $collection
	 * @return boolean
	*/
	public abstract function removeAll(CollectionInterface $collection);

The abstract declaration must precede the visibility declaration

	
	/**
	 * Returns the number of elements in this collection.
	 * @return int
	*/
	public abstract function size();

The abstract declaration must precede the visibility declaration

	
	/**
	 * Returns an array containing all of the elements in this collection.
	 * @return array
	*/
	public abstract function toArray();

The abstract declaration must precede the visibility declaration

	
	/**
	 * Returns the hash code value for this collection.
	 * @return string
	 */
	public abstract function hashCode();

The abstract declaration must precede the visibility declaration

}