Bonfim\Component\View\VariableTpl

Complexity

Total Complexity

8

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	__toString()	0	3	1
B	variable()	0	21	6

<?php

namespace Bonfim\Component\View;

use Exception;

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

    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, $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.

                $explode = explode('.', $matches[$i][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]."']";
                }

                $content = '<?php echo('.$variable.'); ?>';

                if (isset($matches[$i][2]) && $matches[$i][2] == 'upper') {
                    $content = '<?php echo(ucwords('.$variable.')); ?>';
                }

                $this->content = str_replace($matches[$i][0], $content, $this->content);

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

            };
        }
    }
}