Sketch\Tpl\VariableTag

Complexity

Total Complexity

9

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	filter()	0	3	1
A	handle()	0	9	3
A	getVariable()	0	13	2
A	upper()	0	4	3

<?php

namespace Sketch\Tpl;


class VariableTag extends Tag

The type Sketch\Tpl\Tag was not found. Maybe you did not declare it correctly or list all dependencies?

{
    private $pattern = '/{\s?([\w]+.?[\w]+.?[\w]+)\s?\|?\s?([\w]+)?\s?}/is';
    private $match   = [];
    private $matches = [];
    private $replace = '';
    private $variable = '';

    public function handle(): void
    {
        if (preg_match_all($this->pattern, self::$content, $this->matches, PREG_SET_ORDER)) {
            for ($i = 0; $i < count($this->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 = $this->matches[$i];
                $this->getVariable();
                $this->replace = '<?php echo('.$this->variable.'); ?>';
                $this->filter('upper');
                self::$content = str_replace($this->match[0], $this->replace, self::$content);

The property content does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            };
        }
    }

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

        $variable = $explode[0];

        $variable = '$'.$variable;

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

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.

            $variable .= "->".$explode[$k];
        }

        $this->variable = $variable;
    }

    private function filter(string $name): void
    {
        $this->$name();
    }

    private function upper(): void
    {
        if (isset($this->match[2]) && $this->match[2] == 'capitalize') {
            $this->replace = '<?php echo(ucwords(strtolower('.$this->variable.'))); ?>';
        }
    }
}