FlmBus\Ini\IniSection

Complexity

Total Complexity

31

Size/Duplication

Total Lines	281
Duplicated Lines	7.83 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

87.1%

Importance

Changes

0

19 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setParent()	0	6	1
A	setContents()	0	11	2
A	getContents()	0	4	1
A	hasParent()	0	4	1
A	getParent()	0	4	1
A	getName()	0	4	1
A	getParents()	0	14	2
A	composeContents()	0	12	2
A	get()	0	7	2
A	set()	11	11	2
A	delete()	11	11	3
A	hasItem()	0	6	1
A	toArray()	0	4	1
A	toString()	0	10	2
A	renderName()	0	14	2
A	renderItem()	0	13	2
A	renderStringItem()	0	4	1
A	renderArrayItem()	0	12	3

<?php


namespace FlmBus\Ini;


use FlmBus\Ini\Exceptions\InvalidDataException;

class IniSection
{
    /**
     * @var string
     */
    protected $name;
    /**
     * @var IniSection|null
     */
    protected $parent;
    /**
     * @var array
     */
    protected $contents = [];

    /**
     * Section constructor.
     *
     * @param string $name
     * @param IniSection $parent
     */
    public function __construct($name, IniSection $parent = null)
    {
        $this->name = $name;
        $this->parent = $parent;
    }

    /**
     * @param IniSection $parent
     *
     * @return $this
     */
    public function setParent(IniSection $parent)
    {
        $this->parent = $parent;

        return $this;
    }


    /**
     * @param array $data
     *
     * @return $this
     * @throws InvalidDataException
     */
    public function setContents($data)
    {
        if (!is_array($data))
        {
            throw new InvalidDataException('Invalid section contents! ' .
                                           'Section contents must be an array.');
        }
        $this->contents = IniParser::i()->itemValuetoStringRepresentation($data);

It seems like \FlmBus\Ini\IniParser::i()->itemValuetoStringRepresentation($data) can also be of type string. However, the property $contents is declared as type array. Maybe add an additional type check?

        return $this;
    }

    /**
     * @return array
     */
    protected function getContents()
    {
        return $this->contents;
    }

    /**
     * @return bool
     */
    public function hasParent()
    {
        return ($this->parent instanceof IniSection);
    }

    /**
     * @return IniSection|null
     */
    public function getParent()
    {
        return $this->parent;
    }

    /**
     * @return string
     */
    public function getName()
    {
        return $this->name;
    }


    /**
     * @return IniSection[]
     */
    protected function getParents()
    {
        $parents = [];
        $currentSection = $this;
        while ($currentSection->hasParent())
        {
            $parent = $currentSection->getParent();
            $parents[] = $parent;
            $currentSection = $parent;
        }
        rsort($parents);

        return $parents;
    }

    /**
     * @return array
     */
    protected function composeContents()
    {
        $contents = [];
        $parents = $this->getParents();
        foreach ($parents as $section)
        {
            $contents = array_merge($contents, $section->getContents());
        }
        $contents = array_merge($contents, $this->getContents());

        return $contents;
    }

    /**
     * Get normalized item value
     *
     * @param string $itemName
     * @param mixed $defaultValue
     *
     * @return mixed
     */
    public function get($itemName, $defaultValue = null)
    {
        $contents = $this->composeContents();
        $value = isset($contents[$itemName]) ? $contents[$itemName] : $defaultValue;

        return IniParser::i()->castItemValueToProperType($value);
    }

    /**
     * @param string $itemName
     * @param string|array|bool|null $itemValue
     *
     * @return $this
     * @throws InvalidDataException
     */
    public function set($itemName, $itemValue)

This method seems to be duplicated in your project.

    {
        list($validationResult, $message) = IniParser::i()->validateItemName($itemName);
        if (false === $validationResult)
        {
            throw new InvalidDataException($message);
        }
        $this->contents[$itemName] = IniParser::i()->itemValuetoStringRepresentation($itemValue);

        return $this;
    }

    /**
     * @param string $itemName
     *
     * @return $this
     * @throws InvalidDataException
     */
    public function delete($itemName)

This method seems to be duplicated in your project.

    {
        list($validationResult, $message) = IniParser::i()->validateItemName($itemName);
        if (false === $this->hasItem($itemName) || false === $validationResult)
        {
            throw new InvalidDataException($message);
        }
        unset($this->contents[$itemName]);

        return $this;
    }

    /**
     * @param string $itemName
     *
     * @return bool
     */
    public function hasItem($itemName)
    {
        $value = $this->get($itemName, null);

        return !is_null($value);
    }

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

    /**
     * @return string
     */
    public function toString()
    {
        $lines = $this->renderName();
        foreach ($this->contents as $itemName => $itemValue)
        {
            $lines = array_merge($lines, $this->renderItem($itemName, $itemValue));
        }

        return implode(PHP_EOL, $lines) . PHP_EOL;
    }

    /**
     * @return string[]
     */
    protected function renderName()
    {

        if ($this->hasParent())
        {
            $line = [sprintf('[%s : %s]', $this->getName(), $this->getParent()->getName())];
        }
        else
        {
            $line = [sprintf('[%s]', $this->getName())];
        }

        return $line;
    }

    /**
     * @param string $name
     * @param string|array $value
     *
     * @return array
     */
    protected function renderItem($name, $value)
    {
        if (is_array($value))
        {
            $lines = $this->renderArrayItem($name, $value);
        }
        else
        {
            $lines = $this->renderStringItem($name, $value);
        }

        return $lines;
    }

    /**
     * @param string $name
     * @param string $value
     *
     * @return string[]
     */
    protected function renderStringItem($name, $value)
    {
        return [sprintf('%s = "%s"', $name, $value)];
    }

    /**
     * @param string $name
     * @param array $values
     *
     * @return array
     */
    protected function renderArrayItem($name, array $values)
    {
        $lines = [];
        $isAssocArray = (array_values($values) !== $values);
        foreach ($values as $key => $value)
        {
            $stringKey = $isAssocArray ? $key : '';
            $lines[] = sprintf('%s[%s] = "%s"', $name, $stringKey, $value);
        }

        return $lines;
    }

}