drupol\phpngrams\NGramsTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Dependencies

0

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	ngramsArray()	0	4	1
A	ngramsString()	0	6	2
B	doNgrams()	0	16	5

<?php

namespace drupol\phpngrams;

trait NGramsTrait
{
    /**
     * @param array $data
     * @param int $n
     * @param bool $cyclic
     *
     * @return \Generator
     */
    public function ngramsArray(array $data, $n = 1, $cyclic = true)

Avoid variables with short names like $n. Configured minimum length is 3.

    {
        return $this->doNgrams($data, $n, $cyclic);
    }

    /**
     * @param $data
     * @param int $n
     * @param bool $cyclic
     *
     * @return \Generator
     */
    public function ngramsString($data, $n = 1, $cyclic = true)

Avoid variables with short names like $n. Configured minimum length is 3.

    {
        foreach ($this->doNgrams(str_split($data), $n, $cyclic) as $data) {
            yield implode('', $data);
        }
    }

    /**
     * @param $data
     * @param $n
     * @param $cyclic
     *
     * @return \Generator
     */
    private function doNgrams($data, $n = 1, $cyclic = true)

Avoid variables with short names like $n. Configured minimum length is 3.

    {
        $dataLength = count($data);

        $n = $n > $dataLength ? $dataLength : $n;

Consider using a different name than the parameter $n. This often makes code more readable.

        $length = (false === $cyclic ? $dataLength - $n + 1 : $dataLength);

        for ($j = 0; $j < $length; $j++) {
            $ngrams = [];
            for ($i = $j; $i < $n + $j; $i++) {
                $ngrams[] = $data[$i%$dataLength];
            }
            yield $ngrams;
        }
    }
}