GV\Collection

Complexity

Total Complexity

7

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	add()	0	3	1
A	clear()	0	3	2
A	merge()	0	3	1
A	all()	0	3	1
A	last()	0	3	1
A	count()	0	3	1

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 12 and the first side effect is on line 6.

namespace GV;

/** If this file is called directly, abort. */
if ( ! defined( 'GRAVITYVIEW_DIR' ) ) {
	die();
}

/**
 * A generic Collection base class.
 */
class Collection {
	/**
	 * @var array Main storage for objects in this collection.
	 */
	private $storage = array();

	/**
	 * Add an object to this collection.
	 *
	 * @param mixed $value The object to be added.
	 *
	 * @api
	 * @since future
	 * @return void
	 */
	public function add( $value ) {
		$this->storage []= $value;

Expected 1 space before "="; 0 found

	}

	/**
	 * Clear this collection.
	 *
	 * @api
	 * @since future
	 * @return void
	 */
	public function clear() {
		$this->count() && ( $this->storage = array() );
	}

	/**
	 * Merge another collection into here.
	 *
	 * @param \GV\Collection $collection The collection to be merged.
	 *
	 * @api
	 * @since future
	 * @return void
	 */
	public function merge( \GV\Collection $collection ) {
		array_map( array( $this, 'add'), $collection->all() );

No space before closing parenthesis of array is bad style

	}

	/**
	 * Returns all the objects in this collection as an an array.
	 *
	 * @api
	 * @since future
	 * @return array The objects in this collection.
	 */
	public function all() {
		return $this->storage;
	}

	/**
	 * Get the last added object.
	 *
	 * @api
	 * @since future
	 * @return mixed|null The last item in here, or null if there are none.
	 */
	public function last() {
		return end( $this->storage );
	}

	/**
	 * Returns the count of the objects in this collection.
	 *
	 * @api
	 * @since future
	 * @return int The size of this collection.
	 */
	public function count() {
		return count( $this->storage );
	}
}