Eccube\DependencyInjection\Facade\TranslatorFacade

Complexity

Total Complexity

5

Size/Duplication

Total Lines	47
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

3 Methods

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

<?php

namespace Eccube\DependencyInjection\Facade;

use Symfony\Component\DependencyInjection\ContainerInterface;
use Symfony\Component\Translation\TranslatorInterface;

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

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 TranslatorInterface */
    private static $Translator;

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

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

        return self::$instance;
    }

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

        return self::$Translator;
    }
}