ResolverTrait::resolveByContainer() - Code Metrics - Inspection of "simplify method" - n1215/jugoya - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 84dd72...3d3d30 )

by n

created 2017-06-03 07:47 UTC

ResolverTrait::resolveByContainer() A

↳ Parent: ResolverTrait

Complexity

Conditions	3
Paths	3

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	8
cts	8
cp	1
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	9
nc	3
nop	3
crap	3

<?php

namespace N1215\Jugoya\Resolver;

use Psr\Container\ContainerInterface;
use Psr\Container\NotFoundExceptionInterface;
use Psr\Container\ContainerExceptionInterface;

trait ResolverTrait
{

    /**
     * @param string $ref
     * @param ContainerInterface $container
     * @param string $expectedClass
     * @return mixed
     * @throws UnresolvedException
     */
    protected function resolveByContainer($ref, ContainerInterface $container, $expectedClass)
    {
        try {
            $entry = $container->get($ref);
        } catch (NotFoundExceptionInterface $e) {
            throw new UnresolvedException('Could not found an entry from the container.', 0, $e);
        } catch (ContainerExceptionInterface $e) {
            throw new UnresolvedException('Something wrong with the container.', 0, $e);
        }

        $this->assertInstanceOf($expectedClass, $entry);

        return $entry;
    }

    /**
     * @param string $expectedClass
     * @param mixed $entry
     * @return void
     * @throws UnresolvedException
     */
    private function assertInstanceOf($expectedClass, $entry)
    {
        if ($entry instanceof $expectedClass) {
            return;
        }
        $type = is_object($entry) ? get_class($entry) : gettype($entry);
        throw new UnresolvedException("Expected container returns an instance of {$expectedClass}, {$type} given.");
    }
}


1		<?php
2
3		namespace N1215\Jugoya\Resolver;
4
5		use Psr\Container\ContainerInterface;
6		use Psr\Container\NotFoundExceptionInterface;
7		use Psr\Container\ContainerExceptionInterface;
8
9		trait ResolverTrait
10		{
11
12		/**
13		* @param string $ref
14		* @param ContainerInterface $container
15		* @param string $expectedClass
16		* @return mixed
17		* @throws UnresolvedException
18		*/
19	11	protected function resolveByContainer($ref, ContainerInterface $container, $expectedClass)
20		{
21		try {
22	11	$entry = $container->get($ref);
23	11	} catch (NotFoundExceptionInterface $e) {
24	2	throw new UnresolvedException('Could not found an entry from the container.', 0, $e);
25	2	} catch (ContainerExceptionInterface $e) {
26	2	throw new UnresolvedException('Something wrong with the container.', 0, $e);
27		}
28
29	7	$this->assertInstanceOf($expectedClass, $entry);
30
31	5	return $entry;
32		}
33
34		/**
35		* @param string $expectedClass
36		* @param mixed $entry
37		* @return void
38		* @throws UnresolvedException
39		*/
40	7	private function assertInstanceOf($expectedClass, $entry)
41		{
42	7	if ($entry instanceof $expectedClass) {
43	5	return;
44		}
45	2	$type = is_object($entry) ? get_class($entry) : gettype($entry);
46	2	throw new UnresolvedException("Expected container returns an instance of {$expectedClass}, {$type} given.");
47		}
48		}
49

n1215 / jugoya

Push — master ( 84dd72...3d3d30 )

ResolverTrait::resolveByContainer() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like