Ajax\common\html\html5\HtmlCollection

Complexity

Total Complexity

11

Size/Duplication

Total Lines	63
Duplicated Lines	12.7 %

Coupling/Cohesion

Components	2
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	addItems()	0	5	2
A	setItems()	0	4	1
A	getItems()	0	3	1
A	addItem()	0	8	2
A	getItem()	8	8	2
A	setItem()	0	4	1
A	count()	0	3	1
	createItem()	0	1	?

<?php

namespace Ajax\common\html\html5;

use Ajax\common\html\HtmlDoubleElement;

abstract class HtmlCollection extends HtmlDoubleElement {
	public function __construct($identifier,$tagName="div"){
		parent::__construct($identifier,$tagName);
		$this->content=array();
	}

	public function addItems($items){
		foreach ($items as $item){
			$this->addItem($item);
		}
	}

	public function setItems($items){
		$this->content=$items;
		return $this;
	}

	public function getItems(){
		return $this->content;
	}

	/**
	 * adds and returns an item
	 * @param HtmlDoubleElement|string $item
	 * @return \Ajax\common\html\HtmlDoubleElement
	 */
	public function addItem($item){
		$itemO=$item;
		if(\is_string($item)){
			$itemO=$this->createItem($item);
		}
		$this->addContent($itemO);
		return $itemO;
	}

	/**
	 * Return the item at index
	 * @param int|string $index the index or the item identifier
	 * @return \Ajax\common\html\HtmlDoubleElement
	 */
	public function getItem($index) {

This method seems to be duplicated in your project.

		if (is_int($index))
			return $this->content[$index];
		else {
			$elm=$this->getElementById($index, $this->content);
			return $elm;
		}
	}

	public function setItem($index, $value) {
		$this->content[$index]=$value;
		return $this;
	}

	public function count(){
		return \sizeof($this->content);
	}
	/**
	 * The item factory
	 * @param string|HtmlDoubleElement $value
	 */
	protected abstract function createItem($value);

The abstract declaration must precede the visibility declaration

}