RendererFactory - Code Metrics - hogosha/monitor - Measure and Improve Code Quality continuously with Scrutinizer

RendererFactory A
last analyzed 2016-05-11 16:00 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
wmc	3
c	2
b	0
f	1
lcom	0
cbo	3
dl	0
loc	25
rs	10

1 Method

Rating	Name	Duplication	Size	Complexity
A	create()	0	14	3

<?php

/*
 * This file is part of the hogosha-monitor package
 *
 * Copyright (c) 2016 Guillaume Cavana
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * Feel free to edit as you please, and have fun.
 *
 * @author Guillaume Cavana <[email protected]>
 */

namespace Hogosha\Monitor\Renderer;

use Hogosha\Monitor\Monitor;
use Webmozart\Console\Api\IO\IO;

/**
 * @author Guillaume Cavana <[email protected]>
 */
class RendererFactory
{
    /**
     * create.
     *
     * @param string $type
     * @param IO     $io
     *
     * @return RendererInterface
     */
    public static function create($type, IO $io)
    {
        switch ($type) {
            case Monitor::RENDERER_TYPE_TABLE:
                return new TableRenderer($io);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case Monitor::RENDERER_TYPE_CSV:
                return new CsvRenderer($io);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            default:
                return new ListRenderer($io);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
        }
    }
}


1			<?php
2
3			/*
4			* This file is part of the hogosha-monitor package
5			*
6			* Copyright (c) 2016 Guillaume Cavana
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*
11			* Feel free to edit as you please, and have fun.
12			*
13			* @author Guillaume Cavana <[email protected]>
14			*/
15
16			namespace Hogosha\Monitor\Renderer;
17
18			use Hogosha\Monitor\Monitor;
19			use Webmozart\Console\Api\IO\IO;
20
21			/**
22			* @author Guillaume Cavana <[email protected]>
23			*/
24			class RendererFactory
25			{
26			/**
27			* create.
28			*
29			* @param string $type
30			* @param IO $io
31			*
32			* @return RendererInterface
33			*/
34			public static function create($type, IO $io)
35			{
36			switch ($type) {
37			case Monitor::RENDERER_TYPE_TABLE:
38			return new TableRenderer($io);
39			break;
			0 ignored issues – show Unused Code introduced 2016-01-15 16:42 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...
40			case Monitor::RENDERER_TYPE_CSV:
41			return new CsvRenderer($io);
42			break;
			0 ignored issues – show Unused Code introduced 2016-01-15 16:42 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...
43			default:
44			return new ListRenderer($io);
45			break;
			0 ignored issues – show Unused Code introduced 2016-01-15 16:42 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...
46			}
47			}
48			}
49

hogosha / monitor

RendererFactory A last analyzed 2016-05-11 16:00 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

1 Method

Duplication Side-by-Side

Filter issues like

RendererFactory A
last analyzed 2016-05-11 16:00 UTC