Bonfim\Tpl\VariableTpl

Complexity

Total Complexity

11

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

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

<?php

namespace Bonfim\Tpl;

use Exception;

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

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

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

        $this->variable();
    }

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

    private function variable(): void
    {
        if (preg_match_all($this->pattern, $this->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');
                $this->content = str_replace($this->match[0], $this->replace, $this->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] == 'upper') {
            $this->replace = '<?php echo(ucwords('.$this->variable.')); ?>';
        }
    }
}