Sketch\Tpl\LoopTag

Complexity

Total Complexity

8

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	setForeachContent()	0	3	1
A	handle()	0	17	3
A	setForeachBlock()	0	3	1
A	setForeachArray()	0	11	2
A	setForeachCallback()	0	3	1

<?php

namespace Sketch\Tpl;

/**
 * Class LoopTag
 * @package Sketch\Tpl
 */
class LoopTag extends Tag
{
    /**
     * @var string
     */
    private $pattern = '/{\s?loop (.*?) as ([\w]+)\s?}(.*?){\s?\/loop\s?}/is';
    /**
     * @var string
     */
    private $block = '';
    /**
     * @var string
     */
    private $array = '';
    /**
     * @var string
     */
    private $callback = '';
    /**
     * @var string
     */
    private $foreachContent = '';
    /**
     * @var
     */
    private $match;

    public function handle(): void
    {
        if (preg_match_all($this->pattern, self::$content, $matches, PREG_SET_ORDER)) {
            for ($i = 0; $i < count($matches); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

                $this->match = $matches[$i];
                $this->setForeachBlock();
                $this->setForeachArray();
                $this->setForeachCallback();
                $this->setForeachContent();

                $content  = "<?php foreach({$this->array} as {$this->callback}): ?>";
                $content .= $this->foreachContent;
                $content .= "<?php endforeach; ?>";

                self::$content = str_replace($this->block, $content, self::$content);
            };
            new LoopTag();
        }
    }

    private function setForeachBlock() : void
    {
        $this->block = $this->match[0];
    }

    private function setForeachArray() : void
    {
        $explode = explode('.', $this->match[1]);

        $variableArray = '$'.$explode[0];

        for ($i = 1; $i < count($explode); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

            $variableArray .= "->".$explode[$i];
        }

        $this->array = $variableArray;
    }

    private function setForeachCallback() : void
    {
        $this->callback = '$'.$this->match[2];
    }

    private function setForeachContent() : void
    {
        $this->foreachContent = trim($this->match[3]);
    }
}