Bonfim\Tpl\IncludeTpl

Complexity

Total Complexity

5

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	include()	0	7	3
A	__toString()	0	3	1

<?php

namespace Bonfim\Tpl;

class IncludeTpl
{
    private $content;
    private $config;
    private $pattern = '/{\s?include \'?"?(.*?)"?\'?\s?}/is';

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

        $this->include();
    }

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

    public function include()
    {
        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.

                $match = $matches[$i];
                $replace = file_get_contents("{$this->config['template_dir']}/{$match[1]}.html");
                $this->content = str_replace($matches[$i][0], $replace, $this->content);
            };
        }
    }
}