AbstractFactory::createMember() - Code Metrics - Inspection of "[WIP] Moved generic parts to an abstract factory" - phpDocumentor/Reflection - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#91)

by Jaap

created 2016-01-07 20:21 UTC

AbstractFactory::createMember() A

↳ Parent: AbstractFactory

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	5
ccs	3
cts	3
cp	1
rs	9.4286
nc	1
cc	1
eloc	3
nop	3
crap	1

<?php

namespace phpDocumentor\Reflection\Php\Factory;

use phpDocumentor\Reflection\Element;
use phpDocumentor\Reflection\Php\ProjectFactoryStrategy;
use phpDocumentor\Reflection\Php\StrategyContainer;
use phpDocumentor\Reflection\Types\Context;
use PhpParser\Comment\Doc;
use PhpParser\Node;

abstract class AbstractFactory implements ProjectFactoryStrategy
{
    abstract public function matches($object);

    public final function create($object, StrategyContainer $strategies, Context $context = null)

    {
        if (! $this->matches($object)) {
            throw new \InvalidArgumentException(
                sprintf('%s cannot handle objects with the type %s',
                    __CLASS__,
                    is_object($object) ? get_class($object) : gettype($object)
                )
            );
        }

        return $this->doCreate($object, $strategies, $context);
    }

    abstract protected function doCreate($object, StrategyContainer $strategies, Context $context = null);

    /**
     * @param Node|PropertyIterator|ClassConstantIterator $stmt
     * @param StrategyContainer $strategies
     * @param Context $context
     * @return Element
     */
    protected function createMember($stmt, StrategyContainer $strategies, Context $context = null)
    {
        $strategy = $strategies->findMatching($stmt);
        return $strategy->create($stmt, $strategies, $context);
    }

    /**
     * @param Doc $docBlock
     * @param StrategyContainer $strategies
     * @param Context $context
     * @return null|\phpDocumentor\Reflection\DocBlock
     */
    protected function createDocBlock(Doc $docBlock = null, StrategyContainer $strategies, Context $context = null)
// $a must always be passed; it's default value is never used.
function someFunction($a = 5, $b) { }
    {
        if ($docBlock === null) {
            return null;
        }

        return $this->createMember($docBlock, $strategies, $context);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }
}


1		<?php
2
3		namespace phpDocumentor\Reflection\Php\Factory;
4
5		use phpDocumentor\Reflection\Element;
6		use phpDocumentor\Reflection\Php\ProjectFactoryStrategy;
7		use phpDocumentor\Reflection\Php\StrategyContainer;
8		use phpDocumentor\Reflection\Types\Context;
9		use PhpParser\Comment\Doc;
10		use PhpParser\Node;
11
12		abstract class AbstractFactory implements ProjectFactoryStrategy
13		{
14		abstract public function matches($object);
15
16	42	public final function create($object, StrategyContainer $strategies, Context $context = null)
		0 ignored issues – show Coding Style introduced 2016-01-07 20:06 UTC by Report Bug Copy Issue Report As per PSR2, `final` should precede the visibility keyword. Loading history...
17		{
18	42	if (! $this->matches($object)) {
19	8	throw new \InvalidArgumentException(
20	8	sprintf('%s cannot handle objects with the type %s',
21	8	__CLASS__,
22	8	is_object($object) ? get_class($object) : gettype($object)
23	8	)
24	8	);
25		}
26
27	34	return $this->doCreate($object, $strategies, $context);
28		}
29
30		abstract protected function doCreate($object, StrategyContainer $strategies, Context $context = null);
31
32		/**
33		* @param Node\|PropertyIterator\|ClassConstantIterator $stmt
34		* @param StrategyContainer $strategies
35		* @param Context $context
36		* @return Element
37		*/
38	14	protected function createMember($stmt, StrategyContainer $strategies, Context $context = null)
39		{
40	14	$strategy = $strategies->findMatching($stmt);
41	14	return $strategy->create($stmt, $strategies, $context);
42		}
43
44		/**
45		* @param Doc $docBlock
46		* @param StrategyContainer $strategies
47		* @param Context $context
48		* @return null\|\phpDocumentor\Reflection\DocBlock
49		*/
50	28	protected function createDocBlock(Doc $docBlock = null, StrategyContainer $strategies, Context $context = null)
		0 ignored issues – show Coding Style introduced 2016-01-05 20:45 UTC by Report Bug Copy Issue Report Parameters which have default values should be placed at the end. If you place a parameter with a default value before a parameter with a default value, the default value of the first parameter will never be used as it will always need to be passed anyway: // $a must always be passed; it's default value is never used. function someFunction($a = 5, $b) { } Loading history...
51		{
52	28	if ($docBlock === null) {
53	22	return null;
54		}
55
56	6	return $this->createMember($docBlock, $strategies, $context);
		0 ignored issues – show Documentation introduced 2016-01-05 20:45 UTC by Report Bug Copy Issue Report `$docBlock` is of type `object<PhpParser\Comment\Doc>`, but the function expects a `object<PhpParser\Node>\|o...\ClassConstantIterator>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
57		}
58		}
59

phpDocumentor / Reflection

Pull Request — develop (#91)

AbstractFactory::createMember() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like