StateKeys::get() - Code Metrics - appserver-io/messaging - Measure and Improve Code Quality continuously with Scrutinizer

StateKeys::get() B
last analyzed 2018-07-18 09:55 UTC

↳ Parent: StateKeys

Complexity

Conditions	8
Paths	8

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	72

Importance

Changes

Metric	Value
dl	0
loc	27
ccs	0
cts	27
cp	0
rs	8.4444
c	0
b	0
f	0
cc	8
nc	8
nop	1
crap	72

<?php

/**
 * AppserverIo\Messaging\Utils\StateKeys
 *
 * NOTICE OF LICENSE
 *
 * This source file is subject to the Open Software License (OSL 3.0)
 * that is available through the world-wide-web at this URL:
 * http://opensource.org/licenses/osl-3.0.php
 *
 * PHP version 5
 *
 * @author    Tim Wagner <[email protected]>
 * @copyright 2015 TechDivision GmbH <[email protected]>
 * @license   http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
 * @link      https://github.com/appserver-io/messaging
 * @link      http://www.appserver.io
 */

namespace AppserverIo\Messaging\Utils;

/**
 * This class holds the priority keys used as message state.
 *
 * @author    Tim Wagner <[email protected]>
 * @copyright 2015 TechDivision GmbH <[email protected]>
 * @license   http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
 * @link      https://github.com/appserver-io/messaging
 * @link      http://www.appserver.io
 */
class StateKeys
{

    /**
     * Private constructor for marking the class as utility.
     */
    final protected function __construct()
    {
        /* Class is a utility class */
    }

    /**
     * Returns the initialized state key for the passed priority key.
     *
     * @param integer $key The state key to return the instance for
     *
     * @return \AppserverIo\Psr\Pms\StateKeyInterface The instance
     *
     * @throws \Exception
     */
    public static function get($key)
    {
        switch($key) { // check the passed key and return the requested state key instance

            case 1:
                return StateActive::get();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 2:
                return StatePaused::get();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 3:
                return StateToProcess::get();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 4:
                return StateInProgress::get();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 5:
                return StateProcessed::get();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 6:
                return StateFailed::get();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 7:
            default:
                return StateUnknown::get();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
        }
    }
}


1			<?php
2
3			/**
4			* AppserverIo\Messaging\Utils\StateKeys
5			*
6			* NOTICE OF LICENSE
7			*
8			* This source file is subject to the Open Software License (OSL 3.0)
9			* that is available through the world-wide-web at this URL:
10			* http://opensource.org/licenses/osl-3.0.php
11			*
12			* PHP version 5
13			*
14			* @author Tim Wagner <[email protected]>
15			* @copyright 2015 TechDivision GmbH <[email protected]>
16			* @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
17			* @link https://github.com/appserver-io/messaging
18			* @link http://www.appserver.io
19			*/
20
21			namespace AppserverIo\Messaging\Utils;
22
23			/**
24			* This class holds the priority keys used as message state.
25			*
26			* @author Tim Wagner <[email protected]>
27			* @copyright 2015 TechDivision GmbH <[email protected]>
28			* @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
29			* @link https://github.com/appserver-io/messaging
30			* @link http://www.appserver.io
31			*/
32			class StateKeys
33			{
34
35			/**
36			* Private constructor for marking the class as utility.
37			*/
38			final protected function __construct()
39			{
40			/* Class is a utility class */
41			}
42
43			/**
44			* Returns the initialized state key for the passed priority key.
45			*
46			* @param integer $key The state key to return the instance for
47			*
48			* @return \AppserverIo\Psr\Pms\StateKeyInterface The instance
49			*
50			* @throws \Exception
51			*/
52			public static function get($key)
53			{
54			switch($key) { // check the passed key and return the requested state key instance
			0 ignored issues – show Coding Style introduced 2016-03-14 13:18 UTC by Report Bug Copy Issue Report Expected 1 space after SWITCH keyword; 0 found Loading history...
55			case 1:
56			return StateActive::get();
57			break;
			0 ignored issues – show Unused Code introduced 2016-03-14 13:18 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...
58			case 2:
59			return StatePaused::get();
60			break;
			0 ignored issues – show Unused Code introduced 2016-03-14 13:18 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...
61			case 3:
62			return StateToProcess::get();
63			break;
			0 ignored issues – show Unused Code introduced 2016-03-14 13:18 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...
64			case 4:
65			return StateInProgress::get();
66			break;
			0 ignored issues – show Unused Code introduced 2016-03-14 13:18 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...
67			case 5:
68			return StateProcessed::get();
69			break;
			0 ignored issues – show Unused Code introduced 2016-03-14 13:18 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...
70			case 6:
71			return StateFailed::get();
72			break;
			0 ignored issues – show Unused Code introduced 2016-03-14 13:18 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...
73			case 7:
74			default:
75			return StateUnknown::get();
76			break;
			0 ignored issues – show Unused Code introduced 2016-03-14 13:18 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...
77			}
78			}
79			}
80

appserver-io / messaging

StateKeys::get() B last analyzed 2018-07-18 09:55 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

StateKeys::get() B
last analyzed 2018-07-18 09:55 UTC