PdoCouponSheetContainer::get() - Code Metrics - GermaniaKG/Coupons - Measure and Improve Code Quality continuously with Scrutinizer

PdoCouponSheetContainer::get() A
last analyzed 2019-02-19 08:42 UTC

↳ Parent: PdoCouponSheetContainer

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	0
cts	7
cp	0
rs	9.8666
c	0
b	0
f	0
cc	3
nc	3
nop	1
crap	12

<?php
namespace Germania\Coupons;

use Psr\Container\ContainerInterface;
use Psr\Container\NotFoundException;

class PdoCouponSheetContainer implements ContainerInterface
{

    public $instances = array();
    /**
     * @var callable
     */
    public $sheet_factory;

    public function __construct( callable $sheet_factory )
    {
        $this->sheet_factory = $sheet_factory;
    }

    /**
     * @param  string|int $id
     */
    public function has( $id )
    {
        if (array_key_exists( $id, $this->instances))
            return !empty( $this->instances[ $id ] );

        $sheet_factory = $this->sheet_factory;

        $this->instances[ $id ] = $sheet_factory( $id );
        return !empty( $this->instances[ $id ] );
    }


    /**
     * @param  string|int $id
     */
    public function get( $id )
    {
        if (!empty( $this->instances[ $id ]))
            return $this->instances[ $id ];

        if (!$instance = $sheet_factory( $id ))

            throw new CouponSheetNotFoundException("Could not find a coupon sheet with ID $id");


        $this->instances[ $id ] = $instance;
        return $instance;
    }

}



1			<?php
2			namespace Germania\Coupons;
3
4			use Psr\Container\ContainerInterface;
5			use Psr\Container\NotFoundException;
6
7			class PdoCouponSheetContainer implements ContainerInterface
8			{
9
10			public $instances = array();
11			/**
12			* @var callable
13			*/
14			public $sheet_factory;
15
16			public function __construct( callable $sheet_factory )
17			{
18			$this->sheet_factory = $sheet_factory;
19			}
20
21			/**
22			* @param string\|int $id
23			*/
24			public function has( $id )
25			{
26			if (array_key_exists( $id, $this->instances))
27			return !empty( $this->instances[ $id ] );
28
29			$sheet_factory = $this->sheet_factory;
30
31			$this->instances[ $id ] = $sheet_factory( $id );
32			return !empty( $this->instances[ $id ] );
33			}
34
35
36			/**
37			* @param string\|int $id
38			*/
39			public function get( $id )
40			{
41			if (!empty( $this->instances[ $id ]))
42			return $this->instances[ $id ];
43
44			if (!$instance = $sheet_factory( $id ))
			0 ignored issues – show Bug introduced 2019-02-06 12:47 UTC by Report Bug Copy Issue Report The variable `$sheet_factory` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
45			throw new CouponSheetNotFoundException("Could not find a coupon sheet with ID $id");
46
47
48			$this->instances[ $id ] = $instance;
49			return $instance;
50			}
51
52			}
53
54

GermaniaKG / Coupons

PdoCouponSheetContainer::get() A last analyzed 2019-02-19 08:42 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

PdoCouponSheetContainer::get() A
last analyzed 2019-02-19 08:42 UTC