Yiisoft\Strings\CombinedRegexp

Complexity

Total Complexity

9

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Test Coverage

Coverage

68.75%

Importance

Changes	2
Bugs	1	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getCompiledPattern()	0	3	1
A	compilePatterns()	0	13	2
A	matchPatternPosition()	0	14	2
A	matchAny()	0	5	1
A	matchPattern()	0	3	1
A	__construct()	0	7	2

<?php

declare(strict_types=1);

namespace Yiisoft\Strings;

final class CombinedRegexp
{
    private string $freeQuoter;
    private string $quoteReplacer;
    private string $compiledPattern;

    public function __construct(
        private array $patterns,
        private string $regexpQuoter = '/'
    ) {
        $this->freeQuoter = $this->regexpQuoter === '/' ? '~' : '/';
        $this->quoteReplacer = preg_quote($this->regexpQuoter, $this->regexpQuoter);
        $this->compiledPattern = $this->compilePatterns($this->patterns);
    }

    public function getCompiledPattern(): string
    {
        return $this->compiledPattern;
    }

    public function matchAny(string $string, string $flags = 'i'): bool
    {
        $pattern = $this->compiledPattern . $flags;

        return preg_match($pattern, $string) === 1;
    }

    public function matchPattern(string $string)
    {
        return $this->patterns[$this->matchPatternPosition($string)];
    }

    public function matchPatternPosition(string $string): int
    {
        $match = preg_match($this->compiledPattern, $string, $matches);
        if ($match !== 1) {
            throw new \Exception(
                sprintf(
                    'Failed to match pattern "%s" with string "%s".',
                    $this->compiledPattern,
                    $string,
                )
            );
        }

        return count($matches) - 1;
    }

    private function compilePatterns(array $patterns): string
    {
        $quotedPatterns = [];
        for ($i = 0; $i < count($patterns); $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.

            $quotedPatterns[] = preg_replace(
                $this->freeQuoter . $this->regexpQuoter . $this->freeQuoter,
                $this->quoteReplacer,
                $patterns[$i]
            ) . str_repeat('()', $i);
        }
        $combinedRegexps = '(?|' . implode('|', $quotedPatterns) . ')';

        return $this->freeQuoter . $combinedRegexps . $this->freeQuoter;
    }
}