ReliqArts\DirectTranslator\Translation\Replacer\PatternReplacer

Complexity

Total Complexity

3

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	replace()	0	10	1
A	buildPatternsAndReplacementsFromMap()	0	16	2

<?php

declare(strict_types=1);

namespace ReliqArts\DirectTranslator\Translation\Replacer;

use ReliqArts\DirectTranslator\Translation\Replacer;
use ReliqArts\DirectTranslator\Vocabulary;

final class PatternReplacer implements Replacer
{
    /**
     * {@inheritdoc}
     *
     * @return string
     */
    public function replace(
        string $inputText,
        Vocabulary $vocabulary,
        string $direction = self::DIRECTION_LTR
    ): string {
        $map = array_replace($vocabulary->getPhrases(), $vocabulary->getWords());

        list($patterns, $replacements) = $this->buildPatternsAndReplacementsFromMap($map, $direction);

        return preg_replace($patterns, $replacements, $inputText);
    }

    /**
     * @param array  $map
     * @param string $direction
     *
     * @return string[]
     */
    private function buildPatternsAndReplacementsFromMap(array $map, string $direction): array
    {
        $finalMap = $map;
        if ($direction === self::DIRECTION_RTL) {
            $finalMap = array_flip($map);
        }

        $patterns = array_map(
            function (string $wordOrPhrase): string {
                return sprintf('/\b%s\b/ui', $wordOrPhrase);
            },
            array_keys($finalMap)

It seems like $finalMap can also be of type null; however, parameter $input of array_keys() does only seem to accept array, maybe add an additional type check?

        );
        $replacements = array_values($finalMap);

It seems like $finalMap can also be of type null; however, parameter $input of array_values() does only seem to accept array, maybe add an additional type check?

        return [$patterns, $replacements];

The expression return array($patterns, $replacements) returns the type array<integer,array> which is incompatible with the documented return type string[].

    }
}