Eccube\DependencyInjection\Facade\LoggerFacade

Complexity

Total Complexity

6

Size/Duplication

Total Lines	56
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	5	5	1
A	init()	8	8	2
A	create()	8	8	2
A	getContainer()	4	4	1

<?php

namespace Eccube\DependencyInjection\Facade;

use Eccube\Log\Logger;
use Symfony\Component\DependencyInjection\ContainerInterface;

/**
 * XXX ContainerInterface は不要かも
 */
class LoggerFacade

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

This class seems to be duplicated in your project.

{
    /** @var self|null */
    private static $instance = null;

    /** @var ContainerInterface */
    private static $Container;

    /** @var Logger */
    private static $Logger;

    /**
     * @param ContainerInterface $container
     */
    private function __construct(ContainerInterface $container, Logger $Logger)
    {
        self::$Container = $container;
        self::$Logger = $Logger;
    }

    /**
     * @param ContainerInterface $container
     *
     * @return null|LoggerFacade
     */
    public static function init(ContainerInterface $container, Logger $Logger)
    {
        if (null === self::$instance) {
            self::$instance = new self($container, $Logger);
        }

        return self::$instance;
    }

    /**
     * @return Logger
     * @throws \Exception
     */
    public static function create()
    {
        if (null === self::$instance) {
            throw new \Exception("Facade is not instantiated");
        }

        return self::$Logger;
    }

    /**
     * @deprecated
     * @return ContainerInterface
     */
    public static function getContainer()
    {
        return self::$Container;
    }
}