Issues in LastNamesInflection.php - New Issues - Inspection of "Update for late-static binding" - wapmorgan/Morphos - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0d947c...778e4d )

by f

created 2019-01-13 20:52 UTC

src/Russian/LastNamesInflection.php (1 issue)

Labels

Bug 1

Severity

Minor 1

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php
namespace morphos\Russian;

use morphos\S;

/**
 * Rules are from http://gramma.ru/SPR/?id=2.8
 */
class LastNamesInflection extends \morphos\NamesInflection implements Cases
{
    use RussianLanguage, CasesHelper;

    protected static $menPostfixes = ['ов', 'ев' ,'ин' ,'ын', 'ой', 'ий'];
    protected static $womenPostfixes = ['ва', 'на', 'ая', 'яя'];

    /**
     * @param $name
     * @param null $gender
     * @return bool
     */
    public static function isMutable($name, $gender = null)
    {
        $name = S::lower($name);
        if ($gender === null) {
            $gender = static::detectGender($name);
        }
        // составная фамилия - разбить на части и проверить по отдельности
        if (strpos($name, '-') !== false) {
            foreach (explode('-', $name) as $part) {
                if (static::isMutable($part, $gender))
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
                    return true;
            }
            return false;
        }

        if (in_array(S::slice($name, -1), ['а', 'я'], true)) {
            return true;
        }

        if ($gender == static::MALE) {
            // Несклоняемые фамилии (Фоминых, Седых / Стецко, Писаренко)
            if (in_array(S::slice($name, -2), ['ых', 'ко'], true))
                return false;

            // Несклоняемые, образованные из родительного падежа личного или прозвищного имени главы семьи
            // суффиксы: ово, аго
            if (in_array(S::slice($name, -3), ['ово', 'аго'], true))
                return false;

            // Типичные суффикс мужских фамилий
            if (in_array(S::slice($name, -2), ['ов', 'ев', 'ин', 'ын', 'ий', 'ой'], true)) {
                return true;
            }

            // Согласная на конце
            if (static::isConsonant(S::slice($name, -1))) {
                return true;
            }

            // Мягкий знак на конце
            if (S::slice($name, -1) == 'ь') {
                return true;
            }

        } else {
            // Типичные суффиксы женских фамилий
            if (in_array(S::slice($name, -2), ['ва', 'на', 'ая'], true)) {
                return true;
            }
        }

        return false;
    }

    /**
     * @param $name
     * @return null|string
     */
    public static function detectGender($name)
    {
        $name = S::lower($name);
        if (in_array(S::slice($name, -2), static::$menPostfixes, true)) {
            return static::MALE;
        }
        if (in_array(S::slice($name, -2), static::$womenPostfixes, true)) {
            return static::FEMALE;
        }

        return null;
    }

    /**
     * @param $name
     * @param null|string $gender
     * @return array
     */
    public static function getCases($name, $gender = null)
    {
        $name = S::lower($name);
        if ($gender === null) {
            $gender = static::detectGender($name);
        }

        // составная фамилия - разбить на части и склонять по отдельности
        if (strpos($name, '-') !== false) {
            $parts = explode('-', $name);
            $cases = [];
            foreach ($parts as $i => $part) {
                $parts[$i] = static::getCases($part, $gender);
            }

            return static::composeCasesFromWords($parts, '-');
        }

        if (static::isMutable($name, $gender)) {
            if ($gender == static::MALE) {
                if (in_array(S::slice($name, -2), ['ов', 'ев', 'ин', 'ын'], true)) {
                    $prefix = S::name($name);
                    return [
                        static::IMENIT => $prefix,
                        static::RODIT => $prefix.'а',
                        static::DAT => $prefix.'у',
                        static::VINIT => $prefix.'а',
                        static::TVORIT => $prefix.'ым',
                        static::PREDLOJ => $prefix.'е'
                    ];
                } elseif (in_array(S::slice($name, -4), ['ский', 'ской', 'цкий', 'цкой'], true)) {
                    $prefix = S::name(S::slice($name, 0, -2));
                    return [
                        static::IMENIT => S::name($name),
                        static::RODIT => $prefix.'ого',
                        static::DAT => $prefix.'ому',
                        static::VINIT => $prefix.'ого',
                        static::TVORIT => $prefix.'им',
                        static::PREDLOJ => $prefix.'ом'
                    ];
                // Верхний / Убогий / Толстой
                // Верхнего / Убогого / Толстого
                // Верхнему / Убогому / Толстому
                // Верхним / Убогим / Толстым
                // О Верхнем / Об Убогом / О Толстом
                } else if (in_array(S::slice($name, -2), ['ой', 'ый', 'ий'], true)) {
                    $prefix = S::name(S::slice($name, 0, -2));
                    return [
                        static::IMENIT => S::name($name),
                        static::RODIT => $prefix.'ого',
                        static::DAT => $prefix.'ому',
                        static::VINIT => $prefix.'ого',
                        static::TVORIT => $prefix.'ым',
                        static::PREDLOJ => $prefix.'ом'
                    ];
                }

            } else {
                if (in_array(S::slice($name, -3), ['ова', 'ева', 'ина', 'ына'], true)) {
                    $prefix = S::name(S::slice($name, 0, -1));
                    return [
                        static::IMENIT => S::name($name),
                        static::RODIT => $prefix.'ой',
                        static::DAT => $prefix.'ой',
                        static::VINIT => $prefix.'у',
                        static::TVORIT => $prefix.'ой',
                        static::PREDLOJ => $prefix.'ой'
                    ];
                }

                if (in_array(S::slice($name, -2), ['ая'], true)) {
                    $prefix = S::name(S::slice($name, 0, -2));
                    return [
                        static::IMENIT => S::name($name),
                        static::RODIT => $prefix.'ой',
                        static::DAT => $prefix.'ой',
                        static::VINIT => $prefix.'ую',
                        static::TVORIT => $prefix.'ой',
                        static::PREDLOJ => $prefix.'ой'
                    ];
                }
            }

            if (S::slice($name, -1) == 'я') {
                $prefix = S::name(S::slice($name, 0, -1));
                return [
                    static::IMENIT => S::name($name),
                    static::RODIT => $prefix.'и',
                    static::DAT => $prefix.'е',
                    static::VINIT => $prefix.'ю',
                    static::TVORIT => $prefix.'ей',
                    static::PREDLOJ => $prefix.'е'
                ];
            } elseif (S::slice($name, -1) == 'а') {
                $prefix = S::name(S::slice($name, 0, -1));
                return [
                    static::IMENIT => S::name($name),
                    static::RODIT => $prefix.(static::isDeafConsonant(S::slice($name, -2, -1)) ? 'и' : 'ы'),
                    static::DAT => $prefix.'е',
                    static::VINIT => $prefix.'у',
                    static::TVORIT => $prefix.'ой',
                    static::PREDLOJ => $prefix.'е'
                ];
            } elseif (static::isConsonant(S::slice($name, -1)) && S::slice($name, -2) != 'ых') {
                $prefix = S::name($name);
                return [
                    static::IMENIT => S::name($name),
                    static::RODIT => $prefix.'а',
                    static::DAT => $prefix.'у',
                    static::VINIT => $prefix.'а',
                    static::TVORIT => $prefix.'ом',
                    static::PREDLOJ => $prefix.'е'
                ];
            } elseif (S::slice($name, -1) == 'ь' && $gender == static::MALE) {
                $prefix = S::name(S::slice($name, 0, -1));
                return [
                    static::IMENIT => S::name($name),
                    static::RODIT => $prefix.'я',
                    static::DAT => $prefix.'ю',
                    static::VINIT => $prefix.'я',
                    static::TVORIT => $prefix.'ем',
                    static::PREDLOJ => $prefix.'е'
                ];
            }
        }

        $name = S::name($name);
        return array_fill_keys([static::IMENIT, static::RODIT, static::DAT, static::VINIT, static::TVORIT, static::PREDLOJ], $name);
    }

    /**
     * @param $name
     * @param $case
     * @param null $gender
     * @return string
     * @throws \Exception
     */
    public static function getCase($name, $case, $gender = null)
    {
        if (!static::isMutable($name, $gender)) {
            return $name;
        } else {
            $case = static::canonizeCase($case);
            $forms = static::getCases($name, $gender);
            return $forms[$case];
        }
    }
}


1			<?php
2			namespace morphos\Russian;
3
4			use morphos\S;
5
6			/**
7			* Rules are from http://gramma.ru/SPR/?id=2.8
8			*/
9			class LastNamesInflection extends \morphos\NamesInflection implements Cases
10			{
11			use RussianLanguage, CasesHelper;
12
13			protected static $menPostfixes = ['ов', 'ев' ,'ин' ,'ын', 'ой', 'ий'];
14			protected static $womenPostfixes = ['ва', 'на', 'ая', 'яя'];
15
16			/**
17			* @param $name
18			* @param null $gender
19			* @return bool
20			*/
21	60		public static function isMutable($name, $gender = null)
22			{
23	60		$name = S::lower($name);
24	60		if ($gender === null) {
25			$gender = static::detectGender($name);
26			}
27			// составная фамилия - разбить на части и проверить по отдельности
28	60		if (strpos($name, '-') !== false) {
29	3		foreach (explode('-', $name) as $part) {
30	3		if (static::isMutable($part, $gender))
			0 ignored issues – show Bug introduced 2019-01-13 20:54 UTC by Report Bug Copy Issue Report Show Similar Issues like this It seems like `$gender` defined by `static::detectGender($name)` on line `25` can also be of type `string`; however, `morphos\Russian\LastNamesInflection::isMutable()` does only seem to accept `null`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
31	3		return true;
32			}
33			return false;
34			}
35
36	60	View Code Duplication	if (in_array(S::slice($name, -1), ['а', 'я'], true)) {
37	22		return true;
38			}
39
40	39		if ($gender == static::MALE) {
41			// Несклоняемые фамилии (Фоминых, Седых / Стецко, Писаренко)
42	38	View Code Duplication	if (in_array(S::slice($name, -2), ['ых', 'ко'], true))
43	1		return false;
44
45			// Несклоняемые, образованные из родительного падежа личного или прозвищного имени главы семьи
46			// суффиксы: ово, аго
47	37	View Code Duplication	if (in_array(S::slice($name, -3), ['ово', 'аго'], true))
48	1		return false;
49
50			// Типичные суффикс мужских фамилий
51	36		if (in_array(S::slice($name, -2), ['ов', 'ев', 'ин', 'ын', 'ий', 'ой'], true)) {
52	13		return true;
53			}
54
55			// Согласная на конце
56	23		if (static::isConsonant(S::slice($name, -1))) {
57	15		return true;
58			}
59
60			// Мягкий знак на конце
61	8		if (S::slice($name, -1) == 'ь') {
62	8		return true;
63			}
64
65		View Code Duplication	} else {
66			// Типичные суффиксы женских фамилий
67	1		if (in_array(S::slice($name, -2), ['ва', 'на', 'ая'], true)) {
68			return true;
69			}
70			}
71
72	3		return false;
73			}
74
75			/**
76			* @param $name
77			* @return null\|string
78			*/
79	22		public static function detectGender($name)
80			{
81	22		$name = S::lower($name);
82	22		if (in_array(S::slice($name, -2), static::$menPostfixes, true)) {
83	6		return static::MALE;
84			}
85	17		if (in_array(S::slice($name, -2), static::$womenPostfixes, true)) {
86	5		return static::FEMALE;
87			}
88
89	12		return null;
90			}
91
92			/**
93			* @param $name
94			* @param null\|string $gender
95			* @return array
96			*/
97	36		public static function getCases($name, $gender = null)
98			{
99	36		$name = S::lower($name);
100	36		if ($gender === null) {
101			$gender = static::detectGender($name);
102			}
103
104			// составная фамилия - разбить на части и склонять по отдельности
105	36		if (strpos($name, '-') !== false) {
106	3		$parts = explode('-', $name);
107	3		$cases = [];
108	3		foreach ($parts as $i => $part) {
109	3		$parts[$i] = static::getCases($part, $gender);
110			}
111
112	3		return static::composeCasesFromWords($parts, '-');
113			}
114
115	36		if (static::isMutable($name, $gender)) {
116	35		if ($gender == static::MALE) {
117	22		if (in_array(S::slice($name, -2), ['ов', 'ев', 'ин', 'ын'], true)) {
118	5		$prefix = S::name($name);
119			return [
120	5		static::IMENIT => $prefix,
121	5		static::RODIT => $prefix.'а',
122	5		static::DAT => $prefix.'у',
123	5		static::VINIT => $prefix.'а',
124	5		static::TVORIT => $prefix.'ым',
125	5		static::PREDLOJ => $prefix.'е'
126			];
127	17	View Code Duplication	} elseif (in_array(S::slice($name, -4), ['ский', 'ской', 'цкий', 'цкой'], true)) {
128	1		$prefix = S::name(S::slice($name, 0, -2));
129			return [
130	1		static::IMENIT => S::name($name),
131	1		static::RODIT => $prefix.'ого',
132	1		static::DAT => $prefix.'ому',
133	1		static::VINIT => $prefix.'ого',
134	1		static::TVORIT => $prefix.'им',
135	1		static::PREDLOJ => $prefix.'ом'
136			];
137			// Верхний / Убогий / Толстой
138			// Верхнего / Убогого / Толстого
139			// Верхнему / Убогому / Толстому
140			// Верхним / Убогим / Толстым
141			// О Верхнем / Об Убогом / О Толстом
142	16		} else if (in_array(S::slice($name, -2), ['ой', 'ый', 'ий'], true)) {
143	3		$prefix = S::name(S::slice($name, 0, -2));
144			return [
145	3		static::IMENIT => S::name($name),
146	3		static::RODIT => $prefix.'ого',
147	3		static::DAT => $prefix.'ому',
148	3		static::VINIT => $prefix.'ого',
149	3		static::TVORIT => $prefix.'ым',
150	16		static::PREDLOJ => $prefix.'ом'
151			];
152			}
153
154			} else {
155	13	View Code Duplication	if (in_array(S::slice($name, -3), ['ова', 'ева', 'ина', 'ына'], true)) {
156	5		$prefix = S::name(S::slice($name, 0, -1));
157			return [
158	5		static::IMENIT => S::name($name),
159	5		static::RODIT => $prefix.'ой',
160	5		static::DAT => $prefix.'ой',
161	5		static::VINIT => $prefix.'у',
162	5		static::TVORIT => $prefix.'ой',
163	5		static::PREDLOJ => $prefix.'ой'
164			];
165			}
166
167	8		if (in_array(S::slice($name, -2), ['ая'], true)) {
168	3		$prefix = S::name(S::slice($name, 0, -2));
169			return [
170	3		static::IMENIT => S::name($name),
171	3		static::RODIT => $prefix.'ой',
172	3		static::DAT => $prefix.'ой',
173	3		static::VINIT => $prefix.'ую',
174	3		static::TVORIT => $prefix.'ой',
175	3		static::PREDLOJ => $prefix.'ой'
176			];
177			}
178			}
179
180	18		if (S::slice($name, -1) == 'я') {
181	1		$prefix = S::name(S::slice($name, 0, -1));
182			return [
183	1		static::IMENIT => S::name($name),
184	1		static::RODIT => $prefix.'и',
185	1		static::DAT => $prefix.'е',
186	1		static::VINIT => $prefix.'ю',
187	1		static::TVORIT => $prefix.'ей',
188	1		static::PREDLOJ => $prefix.'е'
189			];
190	17	View Code Duplication	} elseif (S::slice($name, -1) == 'а') {
191	5		$prefix = S::name(S::slice($name, 0, -1));
192			return [
193	5		static::IMENIT => S::name($name),
194	5		static::RODIT => $prefix.(static::isDeafConsonant(S::slice($name, -2, -1)) ? 'и' : 'ы'),
195	5		static::DAT => $prefix.'е',
196	5		static::VINIT => $prefix.'у',
197	5		static::TVORIT => $prefix.'ой',
198	5		static::PREDLOJ => $prefix.'е'
199			];
200	12		} elseif (static::isConsonant(S::slice($name, -1)) && S::slice($name, -2) != 'ых') {
201	9		$prefix = S::name($name);
202			return [
203	9		static::IMENIT => S::name($name),
204	9		static::RODIT => $prefix.'а',
205	9		static::DAT => $prefix.'у',
206	9		static::VINIT => $prefix.'а',
207	9		static::TVORIT => $prefix.'ом',
208	9		static::PREDLOJ => $prefix.'е'
209			];
210	3		} elseif (S::slice($name, -1) == 'ь' && $gender == static::MALE) {
211	3		$prefix = S::name(S::slice($name, 0, -1));
212			return [
213	3		static::IMENIT => S::name($name),
214	3		static::RODIT => $prefix.'я',
215	3		static::DAT => $prefix.'ю',
216	3		static::VINIT => $prefix.'я',
217	3		static::TVORIT => $prefix.'ем',
218	3		static::PREDLOJ => $prefix.'е'
219			];
220			}
221			}
222
223	2		$name = S::name($name);
224	2		return array_fill_keys([static::IMENIT, static::RODIT, static::DAT, static::VINIT, static::TVORIT, static::PREDLOJ], $name);
225			}
226
227			/**
228			* @param $name
229			* @param $case
230			* @param null $gender
231			* @return string
232			* @throws \Exception
233			*/
234	6		public static function getCase($name, $case, $gender = null)
235			{
236	6		if (!static::isMutable($name, $gender)) {
237			return $name;
238			} else {
239	6		$case = static::canonizeCase($case);
240	6		$forms = static::getCases($name, $gender);
241	6		return $forms[$case];
242			}
243			}
244			}
245

wapmorgan / Morphos

Push — master ( 0d947c...778e4d )

src/Russian/LastNamesInflection.php (1 issue)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like