Datagen::load() - Code Metrics - Inspection of "Introduce ProcessorInterface & update commands acc..." - shapintv/datagen - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( b28f87...a255d6 )

by Olivier

created 2019-06-18 12:26 UTC

Datagen::load() A

↳ Parent: Datagen

Complexity

Conditions	5
Paths	6

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	5

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	8
cts	8
cp	1
rs	9.4888
c	0
b	0
f	0
cc	5
nc	6
nop	3
crap	5

<?php

declare(strict_types=1);

namespace Shapin\Datagen;

use Shapin\Exception\UnknwonProcessorException;

class Datagen
{
    private $loader;
    private $processors = [];

    public function __construct(Loader $loader, iterable $processors)
    {
        $this->loader = $loader;
        foreach ($processors as $processor) {
            $this->processors[$processor->getName()] = $processor;
        }
    }

    public function load(array $groups = [], array $excludeGroups = [], array $options = []): void
    {
        $fixtures = $this->loader->getFixtures($groups, $excludeGroups);

        foreach ($fixtures as $fixture) {
            $processorOptions = $options[$fixture->getProcessor()] ?: [];

            $this->getProcessor($fixture->getProcessor())->process($fixture, $processorOptions);
        }

        foreach ($this->processors as $processor) {
            $processor->flush($options[$fixture->getProcessor()] ?: []);
foreach ($a as $b) {
}

// $b is defined here only if $a has elements, for example if $a is array()
// then $b would not be defined here. To avoid that, we recommend to set a
// default value for $b.


// Better
$b = 0; // or whatever default makes sense in your context
foreach ($a as $b) {
}

// $b is now guaranteed to be defined here.
        }
    }

    private function getProcessor(string $name): ProcessorInterface
    {
        if (!array_key_exists($name, $this->processors)) {
            throw new UnknwonProcessorException($name);
        }

        return $this->processors[$name];
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Shapin\Datagen;
6
7		use Shapin\Exception\UnknwonProcessorException;
8
9		class Datagen
10		{
11		private $loader;
12		private $processors = [];
13
14	2	public function __construct(Loader $loader, iterable $processors)
15		{
16	2	$this->loader = $loader;
17	2	foreach ($processors as $processor) {
18	2	$this->processors[$processor->getName()] = $processor;
19		}
20	2	}
21
22	2	public function load(array $groups = [], array $excludeGroups = [], array $options = []): void
23		{
24	2	$fixtures = $this->loader->getFixtures($groups, $excludeGroups);
25
26	2	foreach ($fixtures as $fixture) {
27	2	$processorOptions = $options[$fixture->getProcessor()] ?: [];
28
29	2	$this->getProcessor($fixture->getProcessor())->process($fixture, $processorOptions);
30		}
31
32	2	foreach ($this->processors as $processor) {
33	2	$processor->flush($options[$fixture->getProcessor()] ?: []);
		0 ignored issues – show Bug introduced 2019-06-18 12:30 UTC by Report Bug Copy Issue Report The variable `$fixture` seems to be defined by a `foreach` iteration on line `26`. Are you sure the iterator is never empty, otherwise this variable is not defined? It seems like you are relying on a variable being defined by an iteration: foreach ($a as $b) { } // $b is defined here only if $a has elements, for example if $a is array() // then $b would not be defined here. To avoid that, we recommend to set a // default value for $b. // Better $b = 0; // or whatever default makes sense in your context foreach ($a as $b) { } // $b is now guaranteed to be defined here. Loading history...
34		}
35	2	}
36
37	2	private function getProcessor(string $name): ProcessorInterface
38		{
39	2	if (!array_key_exists($name, $this->processors)) {
40		throw new UnknwonProcessorException($name);
41		}
42
43	2	return $this->processors[$name];
44		}
45		}
46

shapintv / datagen

Push — master ( b28f87...a255d6 )

Datagen::load() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like