Sketch\Tpl\FuncTpl

Complexity

Total Complexity

7

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	func()	0	9	3
A	setFuncArgs()	0	3	1
A	__toString()	0	3	1
A	setFuncName()	0	3	1
A	__construct()	0	4	1

<?php

namespace Sketch\Tpl;

class FuncTpl
{
    private $pattern = '/{\s?func ([\w]+)\((.*?)\)\s?}/is';
    private $content;
    private $match;
    private $funcName;
    private $funcArgs;

    public function __construct(string $content)
    {
        $this->content = $content;
        $this->func();
    }

    public function __toString(): string
    {
        return $this->content;
    }

    public function func(): void
    {
        if (preg_match_all($this->pattern, $this->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->setFuncName();
                $this->setFuncArgs();
                $content = '<?php '.$this->funcName.'('.$this->funcArgs.'); ?>';
                $this->content = str_replace($matches[$i][0], $content, $this->content);
            };
        }
    }

    private function setFuncName()
    {
        $this->funcName = $this->match[1];
    }

    private function setFuncArgs()
    {
        $this->funcArgs = $this->match[2];
    }
}