Sauls\Component\Helper\Operation\StringOperation\CountSentences::execute()

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function execute(string $value): int
    {
        $sentences = \preg_split('/(?<!\w\.\w.)(?<![A-Z][a-z]\.)(?<=\.|\?)\s/um', \trim($value), null, PREG_SPLIT_DELIM_CAPTURE);

        $sentences = array_map(function($value) {
            return trim($value);
        }, $sentences);

It seems like $sentences can also be of type false; however, parameter $arr1 of array_map() does only seem to accept array, maybe add an additional type check?

        return \count($sentences);
    }