Sketch\Tpl\VariableTag

Complexity

Total Complexity

6

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	handle()	0	19	3
A	getVar()	0	13	2

<?php

namespace Sketch\Tpl;

/**
 * Class VariableTag
 * @package Sketch\Tpl
 */
class VariableTag extends Tag
{
    public function __construct()
    {
        parent::__construct('/{{(\s?)+([\w\.]+)(\s?)+\|?(\s?)+([\w|]+)?(\s?)+}}/is');
    }

    public function handle(array $match): string
    {
        $filters = explode('|', $match[5]);

        $res = "<?= ";
        
        foreach ($filters as $filter) {
            $res .= "$filter(";
        }

        $res .= $this->getVar();

        foreach ($filters as $filter) {
            $res .= ")";
        }

        $res .= " ?>";

        return $res;
    }

    private function getVar(): string
    {
        $explode = explode('.', $this->match[2]);

        $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];
        }

        return $variable;
    }
}