Pluralizer::inflect() - Code Metrics - Inspection of "Doctrine Inflector 2.0" - doctrine/inflector - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Pull Request — master (#90)

by Jonathan

created 2018-06-10 03:28 UTC

Pluralizer::inflect() D

↳ Parent: Pluralizer

Complexity

Conditions	10
Paths	41

Size

Total Lines	39
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	39
rs	4.8196
c	0
b	0
f	0
cc	10
eloc	21
nc	41
nop	1

How to fix Complexity

<?php

declare(strict_types=1);

namespace Doctrine\Inflector;

use Doctrine\Inflector\Rules\Plural;
use Doctrine\Inflector\Rules\Uninflected;
use function array_keys;
use function array_merge;
use function implode;
use function preg_match;
use function preg_replace;
use function strtolower;
use function substr;

class Pluralizer extends InflectorService
{
    /** @var Uninflected */
    private $uninflected;

    public function __construct(Uninflected $uninflected)
    {
        $this->uninflected = $uninflected;

        parent::__construct();
    }

    /**
     * {@inheritdoc}
     */
    public function getRules() : array
    {
        return Plural::RULES;
    }

    /**
     * Returns a word in plural form.
     *
     * @param string $word The word in singular form.
     *
     * @return string The word in plural form.
     */
    public function inflect(string $word) : string
    {
        if (isset($this->cache['pluralize'][$word])) {
            return $this->cache['pluralize'][$word];
        }

        if (! isset($this->rules['merged']['irregular'])) {
            $this->rules['merged']['irregular'] = $this->rules['irregular'];
        }

        if (! isset($this->rules['merged']['uninflected'])) {
            $this->rules['merged']['uninflected'] = array_merge(
                $this->rules['uninflected'],
                $this->uninflected->getUninflected()
            );
        }

        if (! isset($this->rules['cacheUninflected']) || ! isset($this->rules['cacheIrregular'])) {
            $this->rules['cacheUninflected'] = '(?:' . implode('|', $this->rules['merged']['uninflected']) . ')';
            $this->rules['cacheIrregular']   = '(?:' . implode('|', array_keys($this->rules['merged']['irregular'])) . ')';
        }

        if (preg_match('/(.*)\\b(' . $this->rules['cacheIrregular'] . ')$/i', $word, $regs)) {
            $this->cache['pluralize'][$word] = $regs[1] . $word[0] . substr($this->rules['merged']['irregular'][strtolower($regs[2])], 1);

            return $this->cache['pluralize'][$word];
        }

        if (preg_match('/^(' . $this->rules['cacheUninflected'] . ')$/i', $word, $regs)) {
            $this->cache['pluralize'][$word] = $word;

            return $word;
        }

        foreach ($this->rules['rules'] as $rule => $replacement) {
            if (preg_match($rule, $word)) {
                $this->cache['pluralize'][$word] = preg_replace($rule, $replacement, $word);

                return $this->cache['pluralize'][$word];
            }
        }
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

class MyClass implements ReturnsInt {
    public function returnsIntHinted(): int
    {
        if (foo()) {
            return 123;
        }
        // here: null is implicitly returned
    }
}
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Doctrine\Inflector;
6
7			use Doctrine\Inflector\Rules\Plural;
8			use Doctrine\Inflector\Rules\Uninflected;
9			use function array_keys;
10			use function array_merge;
11			use function implode;
12			use function preg_match;
13			use function preg_replace;
14			use function strtolower;
15			use function substr;
16
17			class Pluralizer extends InflectorService
18			{
19			/** @var Uninflected */
20			private $uninflected;
21
22			public function __construct(Uninflected $uninflected)
23			{
24			$this->uninflected = $uninflected;
25
26			parent::__construct();
27			}
28
29			/**
30			* {@inheritdoc}
31			*/
32			public function getRules() : array
33			{
34			return Plural::RULES;
35			}
36
37			/**
38			* Returns a word in plural form.
39			*
40			* @param string $word The word in singular form.
41			*
42			* @return string The word in plural form.
43			*/
44			public function inflect(string $word) : string
45			{
46			if (isset($this->cache['pluralize'][$word])) {
47			return $this->cache['pluralize'][$word];
48			}
49
50			if (! isset($this->rules['merged']['irregular'])) {
51			$this->rules['merged']['irregular'] = $this->rules['irregular'];
52			}
53
54			if (! isset($this->rules['merged']['uninflected'])) {
55			$this->rules['merged']['uninflected'] = array_merge(
56			$this->rules['uninflected'],
57			$this->uninflected->getUninflected()
58			);
59			}
60
61			if (! isset($this->rules['cacheUninflected']) \|\| ! isset($this->rules['cacheIrregular'])) {
62			$this->rules['cacheUninflected'] = '(?:' . implode('\|', $this->rules['merged']['uninflected']) . ')';
63			$this->rules['cacheIrregular'] = '(?:' . implode('\|', array_keys($this->rules['merged']['irregular'])) . ')';
64			}
65
66			if (preg_match('/(.*)\\b(' . $this->rules['cacheIrregular'] . ')$/i', $word, $regs)) {
67			$this->cache['pluralize'][$word] = $regs[1] . $word[0] . substr($this->rules['merged']['irregular'][strtolower($regs[2])], 1);
68
69			return $this->cache['pluralize'][$word];
70			}
71
72			if (preg_match('/^(' . $this->rules['cacheUninflected'] . ')$/i', $word, $regs)) {
73			$this->cache['pluralize'][$word] = $word;
74
75			return $word;
76			}
77
78			foreach ($this->rules['rules'] as $rule => $replacement) {
79			if (preg_match($rule, $word)) {
80			$this->cache['pluralize'][$word] = preg_replace($rule, $replacement, $word);
81
82			return $this->cache['pluralize'][$word];
83			}
84			}
			0 ignored issues – show Bug Best Practice introduced 2018-06-10 03:31 UTC by Report Bug Copy Issue Report In this branch, the function will implicitly return `null` which is incompatible with the type-hinted return `string`. Consider adding a return statement or allowing `null` as return value. For hinted functions/methods where all return statements with the correct type are only reachable via conditions, ?null? gets implicitly returned which may be incompatible with the hinted type. Let?s take a look at an example: interface ReturnsInt { public function returnsIntHinted(): int; } class MyClass implements ReturnsInt { public function returnsIntHinted(): int { if (foo()) { return 123; } // here: null is implicitly returned } } Loading history...
85			}
86			}
87

doctrine / inflector

Pull Request — master (#90)

Pluralizer::inflect() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like