LazyServiceFactory::__construct() - Code Metrics - Inspection of "add bad event dispatcher tests" - spiechu/lazy-pimple - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( b8df74...d61374 )

by Dawid

created 2019-04-27 10:40 UTC

LazyServiceFactory::__construct() A

↳ Parent: LazyServiceFactory

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	3
cts	3
cp	1
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1
crap	1

<?php

namespace Spiechu\LazyPimple\Factory;

use ProxyManager\Factory\LazyLoadingValueHolderFactory;
use ProxyManager\Proxy\VirtualProxyInterface;

class LazyServiceFactory
{
    /**
     * @var LazyLoadingValueHolderFactory
     */
    protected $lazyLoadingFactory;

    /**
     * @param LazyLoadingValueHolderFactory $lazyLoadingFactory
     */
    public function __construct(LazyLoadingValueHolderFactory $lazyLoadingFactory)
    {
        $this->lazyLoadingFactory = $lazyLoadingFactory;
    }

    /**
     * @param string   $className
     * @param callable $definition
     *
     * @throws \InvalidArgumentException When $definition callable doesn't return $className instance
     *
     * @return VirtualProxyInterface
     */
    public function getLazyServiceDefinition(string $className, callable $definition): VirtualProxyInterface
    {
        return $this->lazyLoadingFactory->createProxy(
            $className,
            // this fancy method signature is required by lazy loading factory
            static function (&$wrappedObject, $proxy, $method, $parameters, &$initializer) use ($className, $definition) {
                $wrappedObject = $definition();
                $initializer = null;

                // extra defensive programming in action
                if (!$wrappedObject instanceof $className) {

                    throw new \InvalidArgumentException(sprintf(
                        'Object "%s" is not instance of "%s"',
                        \is_object($wrappedObject) ? \get_class($wrappedObject) : \gettype($wrappedObject),
                        $className
                    ));
                }

                return true;
            }
        );
    }
}


1			<?php
2
3			namespace Spiechu\LazyPimple\Factory;
4
5			use ProxyManager\Factory\LazyLoadingValueHolderFactory;
6			use ProxyManager\Proxy\VirtualProxyInterface;
7
8			class LazyServiceFactory
9			{
10			/**
11			* @var LazyLoadingValueHolderFactory
12			*/
13			protected $lazyLoadingFactory;
14
15			/**
16			* @param LazyLoadingValueHolderFactory $lazyLoadingFactory
17			*/
18	4		public function __construct(LazyLoadingValueHolderFactory $lazyLoadingFactory)
19			{
20	4		$this->lazyLoadingFactory = $lazyLoadingFactory;
21	4		}
22
23			/**
24			* @param string $className
25			* @param callable $definition
26			*
27			* @throws \InvalidArgumentException When $definition callable doesn't return $className instance
28			*
29			* @return VirtualProxyInterface
30			*/
31	4		public function getLazyServiceDefinition(string $className, callable $definition): VirtualProxyInterface
32			{
33	4		return $this->lazyLoadingFactory->createProxy(
34	4		$className,
35			// this fancy method signature is required by lazy loading factory
36			static function (&$wrappedObject, $proxy, $method, $parameters, &$initializer) use ($className, $definition) {
37	1		$wrappedObject = $definition();
38	1		$initializer = null;
39
40			// extra defensive programming in action
41	1	View Code Duplication	if (!$wrappedObject instanceof $className) {
			0 ignored issues – show Duplication introduced 2019-04-27 10:18 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
42			throw new \InvalidArgumentException(sprintf(
43			'Object "%s" is not instance of "%s"',
44			\is_object($wrappedObject) ? \get_class($wrappedObject) : \gettype($wrappedObject),
45			$className
46			));
47			}
48
49	1		return true;
50	4		}
51			);
52			}
53			}
54

spiechu / lazy-pimple

Push — master ( b8df74...d61374 )

LazyServiceFactory::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like