FixerFactory::getFixerByIssue() - Code Metrics - pluswerk/typoscript-auto-fixer - Measure and Improve Code Quality continuously with Scrutinizer

FixerFactory::getFixerByIssue() A
last analyzed 2019-12-09 12:29 UTC

↳ Parent: FixerFactory

Complexity

Conditions	5
Paths	5

Size

Total Lines	16
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	13
c	1
b	0
f	0
dl	0
loc	16
rs	9.5222
cc	5
nc	5
nop	1

<?php
declare(strict_types=1);

namespace Pluswerk\TypoScriptAutoFixer\Fixer;

use Pluswerk\TypoScriptAutoFixer\Exception\FixerNotFoundException;
use Pluswerk\TypoScriptAutoFixer\Fixer\EmptySection\EmptySectionFixer;
use Pluswerk\TypoScriptAutoFixer\Fixer\NestingConsistency\NestingConsistencyFixer;
use Pluswerk\TypoScriptAutoFixer\Fixer\Indentation\IndentationFixer;
use Pluswerk\TypoScriptAutoFixer\Fixer\OperatorWhitespace\OperatorWhitespaceFixer;
use Pluswerk\TypoScriptAutoFixer\Issue\AbstractIssue;
use Pluswerk\TypoScriptAutoFixer\Issue\EmptySectionIssue;
use Pluswerk\TypoScriptAutoFixer\Issue\IndentationIssue;
use Pluswerk\TypoScriptAutoFixer\Issue\NestingConsistencyIssue;
use Pluswerk\TypoScriptAutoFixer\Issue\OperatorWhitespaceIssue;

class FixerFactory
{
    /**
     * @param AbstractIssue $issue
     *
     * @return FixerInterface|null
     */
    public function getFixerByIssue(AbstractIssue $issue): ?FixerInterface
    {
        switch (get_class($issue)) {
            case OperatorWhitespaceIssue::class:
                return new OperatorWhitespaceFixer();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case IndentationIssue::class:
                return new IndentationFixer();
                break;
            case EmptySectionIssue::class:
                return new EmptySectionFixer();
                break;
            case NestingConsistencyIssue::class:
                return new NestingConsistencyFixer();
        }
        throw new FixerNotFoundException('Fixer for issue ' . get_class($issue) . ' not found');
    }
}


1			<?php
2			declare(strict_types=1);
3
4			namespace Pluswerk\TypoScriptAutoFixer\Fixer;
5
6			use Pluswerk\TypoScriptAutoFixer\Exception\FixerNotFoundException;
7			use Pluswerk\TypoScriptAutoFixer\Fixer\EmptySection\EmptySectionFixer;
8			use Pluswerk\TypoScriptAutoFixer\Fixer\NestingConsistency\NestingConsistencyFixer;
9			use Pluswerk\TypoScriptAutoFixer\Fixer\Indentation\IndentationFixer;
10			use Pluswerk\TypoScriptAutoFixer\Fixer\OperatorWhitespace\OperatorWhitespaceFixer;
11			use Pluswerk\TypoScriptAutoFixer\Issue\AbstractIssue;
12			use Pluswerk\TypoScriptAutoFixer\Issue\EmptySectionIssue;
13			use Pluswerk\TypoScriptAutoFixer\Issue\IndentationIssue;
14			use Pluswerk\TypoScriptAutoFixer\Issue\NestingConsistencyIssue;
15			use Pluswerk\TypoScriptAutoFixer\Issue\OperatorWhitespaceIssue;
16
17			class FixerFactory
18			{
19			/**
20			* @param AbstractIssue $issue
21			*
22			* @return FixerInterface\|null
23			*/
24			public function getFixerByIssue(AbstractIssue $issue): ?FixerInterface
25			{
26			switch (get_class($issue)) {
27			case OperatorWhitespaceIssue::class:
28			return new OperatorWhitespaceFixer();
29			break;
			0 ignored issues – show Unused Code introduced 2019-12-09 08:29 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...
30			case IndentationIssue::class:
31			return new IndentationFixer();
32			break;
33			case EmptySectionIssue::class:
34			return new EmptySectionFixer();
35			break;
36			case NestingConsistencyIssue::class:
37			return new NestingConsistencyFixer();
38			}
39			throw new FixerNotFoundException('Fixer for issue ' . get_class($issue) . ' not found');
40			}
41			}
42

pluswerk / typoscript-auto-fixer

FixerFactory::getFixerByIssue() A last analyzed 2019-12-09 12:29 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

FixerFactory::getFixerByIssue() A
last analyzed 2019-12-09 12:29 UTC