PlatformFactory - Code Metrics - Inspection of "Development instructions" - LukeCarrier/moodle-componentmgr - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 38c1c8...813522 )

by Luke

created 2016-06-13 12:38 UTC

PlatformFactory A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
wmc	7
c	2
b	0
f	0
lcom	1
cbo	3
dl	0
loc	47
ccs	0
cts	11
cp	0
rs	10

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
B	getPlatform()	0	19	6

<?php

/**
 * Moodle component manager.
 *
 * @author Luke Carrier <[email protected]>
 * @copyright 2016 Luke Carrier
 * @license GPL-3.0+
 */

namespace ComponentManager\Platform;

use ComponentManager\Exception\PlatformException;
use Symfony\Component\DependencyInjection\Definition;
use Symfony\Component\Filesystem\Filesystem;

class PlatformFactory {
    /**
     * Filesystem.
     *
     * @var \Symfony\Component\Filesystem\Filesystem
     */
    protected $filesystem;

    /**
     * Initialiser.
     *
     * @param \Symfony\Component\Filesystem\Filesystem $filesystem
     */
    public function __construct(Filesystem $filesystem) {
        $this->filesystem = $filesystem;
    }

    /**
     * Get the platform with the specified name.
     *
     * @param string $platformName The name of the platform (retrieved from
     *                             PHP_OS).
     *
     * @return \ComponentManager\Platform\Platform

     *
     * @throws \ComponentManager\Exception\PlatformException
     */
    public function getPlatform($platformName) {
        switch ($platformName) {
            case 'Darwin':
            case 'FreeBSD':
            case 'Linux':
                return new LinuxPlatform($this->filesystem);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

            case 'WINNT':
            case 'Windows':
                return new WindowsPlatform($this->filesystem);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

            default:
                throw new PlatformException(
                        sprintf('Unsupported platform %s', PHP_OS),
                        PlatformException::CODE_UNKNOWN_PLATFORM);
        }
    }
}


1			<?php
2
3			/**
4			* Moodle component manager.
5			*
6			* @author Luke Carrier <[email protected]>
7			* @copyright 2016 Luke Carrier
8			* @license GPL-3.0+
9			*/
10
11			namespace ComponentManager\Platform;
12
13			use ComponentManager\Exception\PlatformException;
14			use Symfony\Component\DependencyInjection\Definition;
15			use Symfony\Component\Filesystem\Filesystem;
16
17			class PlatformFactory {
18			/**
19			* Filesystem.
20			*
21			* @var \Symfony\Component\Filesystem\Filesystem
22			*/
23			protected $filesystem;
24
25			/**
26			* Initialiser.
27			*
28			* @param \Symfony\Component\Filesystem\Filesystem $filesystem
29			*/
30			public function __construct(Filesystem $filesystem) {
31			$this->filesystem = $filesystem;
32			}
33
34			/**
35			* Get the platform with the specified name.
36			*
37			* @param string $platformName The name of the platform (retrieved from
38			* PHP_OS).
39			*
40			* @return \ComponentManager\Platform\Platform
			0 ignored issues – show Documentation introduced 2016-06-08 11:33 UTC by Report Bug Copy Issue Report Should the return type not be `LinuxPlatform\|null\|WindowsPlatform`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
41			*
42			* @throws \ComponentManager\Exception\PlatformException
43			*/
44			public function getPlatform($platformName) {
45			switch ($platformName) {
46			case 'Darwin':
47			case 'FreeBSD':
48			case 'Linux':
49			return new LinuxPlatform($this->filesystem);
50			break;
			0 ignored issues – show Unused Code introduced 2016-06-08 11:33 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...
51
52			case 'WINNT':
53			case 'Windows':
54			return new WindowsPlatform($this->filesystem);
55			break;
			0 ignored issues – show Unused Code introduced 2016-06-08 11:33 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...
56
57			default:
58			throw new PlatformException(
59			sprintf('Unsupported platform %s', PHP_OS),
60			PlatformException::CODE_UNKNOWN_PLATFORM);
61			}
62			}
63			}
64

LukeCarrier / moodle-componentmgr

Push — develop ( 38c1c8...813522 )

PlatformFactory A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

2 Methods

Duplication Side-by-Side

Filter issues like