BestServedCold\PhalueObjects\Pattern\Singleton

Complexity

Total Complexity

6

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getInstance()	0	10	3
A	destroy()	0	4	1
A	destroyInstance()	0	4	1

<?php


namespace BestServedCold\PhalueObjects\Pattern;


/**

 * Class Multiton

 *

 * @package   BestServedCold\PhalueObjects\Pattern

 * @author    Adam Lewis <adam.lewis@bestservedcold.com>

 * @copyright Copyright (c) 2015 Best Served Cold Media Limited

 * @license   http://http://opensource.org/licenses/GPL-3.0 GPL License

 * @link      http://bestservedcold.com

 * @since     0.0.1-alpha

 * @version   0.0.2-alpha

 */

class Singleton extends NotConstructable

Since you have declared the constructor as private, maybe you should also declare the class as final.

{

    /**

     * @var array $instances

     */

    protected static $instances = [ ];


    /**

     * Prevent class from being constructed.

     */

    private function __construct()

    {

    }


    /**

     * Get instance from $instances array.

     *

     * If the called class is present in the static $instances array, then return it.

     * Otherwise, create a new copy of the class and store it in the $instances

     * array.

     *

     * @param  string|null $calledClass

     * @return Object

     */

    final public static function getInstance($calledClass = null)

    {

        $calledClass = $calledClass ?: get_called_class();


        if (!isset(self::$instances[ $calledClass ])) {

            self::$instances[ $calledClass ] = new $calledClass();

        }


        return self::$instances[ $calledClass ];

    }


    /**

     * Destroy

     *

     * @return void

     */

    final public static function destroy()

    {

        self::$instances = [ ];

    }


    /**

     * @param $instance

     */

    final public static function destroyInstance($instance)

    {

        unset(self::$instances[ $instance ]);

    }

}