Caxy\HtmlDiff\Table\TableRow

Complexity

Total Complexity

13

Size/Duplication

Total Lines	98
Duplicated Lines	8.16 %

Coupling/Cohesion

Components	2
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes	5
Bugs	0	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getTable()	0	4	1
A	setTable()	0	10	2
A	getCells()	0	4	1
A	addCell()	0	10	2
A	removeCell()	0	11	3
A	getCell()	0	4	2
A	insertCells()	8	8	2

<?php

namespace Caxy\HtmlDiff\Table;

/**
 * Class TableRow
 * @package Caxy\HtmlDiff\Table
 */
class TableRow extends AbstractTableElement
{
    /**
     * @var Table
     */
    protected $table;

    /**
     * @var array
     */
    protected $cells = array();

    /**
     * @return Table
     */
    public function getTable()
    {
        return $this->table;
    }

    /**
     * @param Table|null $table
     *
     * @return $this
     */
    public function setTable(Table $table = null)
    {
        $this->table = $table;

        if (!in_array($this, $table->getRows())) {
            $table->addRow($this);

It seems like $table is not always an object, but can also be of type null. Maybe add an additional type check?

        }

        return $this;
    }

    /**
     * @return array
     */
    public function getCells()
    {
        return $this->cells;
    }

    /**
     * @param TableCell $cell
     *
     * @return $this
     */
    public function addCell(TableCell $cell)
    {
        $this->cells[] = $cell;

        if (!$cell->getRow()) {
            $cell->setRow($this);
        }

        return $this;
    }

    /**
     * @param TableCell $cell
     */
    public function removeCell(TableCell $cell)
    {
        $key = array_search($cell, $this->cells, true);

        if ($key !== false) {
            unset($this->cells[$key]);
            if ($cell->getRow()) {
                $cell->setRow(null);
            }
        }
    }

    /**
     * @param int $index
     *
     * @return TableCell|null
     */
    public function getCell($index)
    {
        return isset($this->cells[$index]) ? $this->cells[$index] : null;
    }

    /**
     * @param array    $cells
     * @param null|int $position
     */
    public function insertCells($cells, $position = null)

This method seems to be duplicated in your project.

    {
        if ($position === null) {
            $this->cells = array_merge($this->cells, $cells);
        } else {
            array_splice($this->cells, $position, 0, $cells);
        }
    }
}