InflectionOrdinalizeTrait::doOrdinalize() - Code Metrics - Inspection of "Merge branch 'develop'" - bytic/inflector - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 03087a...f49b8e )

by Gabriel

created 2018-02-09 20:40 UTC

InflectionOrdinalizeTrait::doOrdinalize() B

↳ Parent: InflectionOrdinalizeTrait

Complexity

Conditions	5
Paths	5

Size

Total Lines	20
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes

Metric	Value
cc	5
eloc	16
nc	5
nop	1
dl	0
loc	20
ccs	0
cts	11
cp	0
crap	30
rs	8.8571
c	0
b	0
f	0

<?php

namespace Nip\Inflector\Traits;

/**
 * Trait InflectionOrdinalizeTrait
 * @package Nip\Inflector\Traits
 *
 * @method ordinalize($string)
 */
trait InflectionOrdinalizeTrait
{


    /**
     * @param $number
     * @return string
     */
    protected function doOrdinalize($number)
    {
        if (in_array(($number % 100), range(11, 13))) {
            return $number . 'th';
        } else {
            switch (($number % 10)) {
                case 1:
                    return $number . 'st';
                    break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
                case 2:
                    return $number . 'nd';
                    break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
                case 3:
                    return $number . 'rd';
                default:
                    return $number . 'th';
                    break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            }
        }
    }
}


1			<?php
2
3			namespace Nip\Inflector\Traits;
4
5			/**
6			* Trait InflectionOrdinalizeTrait
7			* @package Nip\Inflector\Traits
8			*
9			* @method ordinalize($string)
10			*/
11			trait InflectionOrdinalizeTrait
12			{
13
14
15			/**
16			* @param $number
17			* @return string
18			*/
19			protected function doOrdinalize($number)
20			{
21			if (in_array(($number % 100), range(11, 13))) {
22			return $number . 'th';
23			} else {
24			switch (($number % 10)) {
25			case 1:
26			return $number . 'st';
27			break;
			0 ignored issues – show Unused Code introduced 2018-02-02 00:15 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
28			case 2:
29			return $number . 'nd';
30			break;
			0 ignored issues – show Unused Code introduced 2018-02-02 00:15 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
31			case 3:
32			return $number . 'rd';
33			default:
34			return $number . 'th';
35			break;
			0 ignored issues – show Unused Code introduced 2018-02-02 00:15 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
36			}
37			}
38			}
39			}
40

bytic / inflector

Push — master ( 03087a...f49b8e )

InflectionOrdinalizeTrait::doOrdinalize() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like