Ajax\semantic\html\content\table\HtmlTR

Complexity

Total Complexity

10

Size/Duplication

Total Lines	54
Duplicated Lines	22.22 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	setColCount()	0	8	2
A	createItem()	0	6	1
A	setTdTagName()	0	3	1
A	setContainer()	0	4	1
A	setValues()	12	12	3
A	delete()	0	4	1

<?php

namespace Ajax\semantic\html\content\table;

use Ajax\semantic\html\base\HtmlSemCollection;

class HtmlTR extends HtmlSemCollection{

	private $_tdTagName;
	private $_container;
	private $_row;

	public function __construct( $identifier, $colCount){
		parent::__construct( $identifier, "tr", "");
	}

	public function setColCount($colCount){
		$count=$this->count();
		for($i=$count;$i<$colCount;$i++){
			$item=$this->addItem(NULL);
			$item->setTagName($this->_tdTagName);
		}
		return $this;
	}


	protected function createItem($value){
		$count=$this->count();
		$td=new HtmlTD("",$this->_container,$value,$this->_tdTagName);

It seems like $value defined by parameter $value on line 27 can also be of type object<Ajax\common\html\HtmlDoubleElement>; however, Ajax\semantic\html\content\table\HtmlTD::__construct() does only seem to accept string, maybe add an additional type check?

The call to HtmlTD::__construct() has too many arguments starting with $this->_tdTagName.

		$td->setContainer($this->_container, $this->_row, $count);
		return $td;
	}

	public function setTdTagName($tagName="td"){
		$this->_tdTagName=$tagName;
	}

	public function setContainer($container,$row){
		$this->_container=$container;
		$this->_row=$row;
	}

	public function setValues($values=array()){

This method seems to be duplicated in your project.

		$count=$this->count();
		if(\is_array($values)===false){
			$values=\array_fill(0, $count, $values);
		}
		$count=\min(\sizeof($values),$count);

		for ($i=0;$i<$count;$i++){
			$cell=$this->content[$i];
			$cell->setValue($values[$i]);
		}
	}

	public function delete($index){
		$this->removeItem($index);
		return $this;
	}
}