Что можно улучшить/исправить в классе (Singleton)?

Question

[Александр]

Занялся наконец то фундаментальными вещами чтоб подтянуть теоретическую базу.
Сейчас читаю книгу Мэтта Зандстры - PHP: объекты, шаблоны и методики программирования.
Дошел до главы с паттернами и пробую на практике
Написал класс для получения конфигурации используя Синглтон
подскажите пожалуйста косяки какие то есть? Что можно было бы улучшить/исправить

<?php

namespace App\Core;

use Symfony\Component\Dotenv\Dotenv;

/**
 * Class Config
 * @package App\Core
 */
class Config {

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

	/**
	 * @var array
	 */
	private $params = [];

	/**
	 * Config constructor.
	 */
	private function __construct() {
		$dotenv = new Dotenv();
		// здесь немного смущает путь к конфигурационному файлу
		$dotenv->load( __DIR__ . '/../../.env' );
		$this->params = $_ENV;
	}

	/**
	 * @return static
	 */
	public static function getInstance(): self {
		// в этом методе можно использовать и static и self Есть ли какая то разница
		if ( empty( self::$instance ) ) {
			return new Config();
		}

		return self::$instance;
	}

	/**
	 * @return array
	 */
	public function getParams(): array {
		return $this->params;
	}

	/**
	 * @param string $key
	 *
	 * @return string
	 */
	public function getParam( string $key ): string {
		return $this->params[ $key ];
	}
}

Comments

[FanatPHP]

Ты пробовал создать два экземпляра данного класса?

[Александр]

$config = Config::getInstance();
$config2 = Config::getInstance();
var_dump($config2 === $config);
почему-то false

[Александр]

FanatPHP, Да, вижу теперь косяк когда получаю экземпляр

[Дмитрий]

Александр, надо добавить несколько методов заглушек

[Александр]

Дмитрий, можете немножко больше пояснить?

[Дмитрий]

Александр, есть метод магический для клонирования и ещё магии всякой, которая может нарушить этот паттерн

abstract class Singleton {
    protected static $instance = null;
 
    public function __construct(){}
 
    public function __clone(){}
 
    public function __wakeup(){}
 
    public final static function getInstance() {
        if (null === static::$instance) {
            static::$instance = new static();
        }
        return static::$instance;
    }
}

[Виталий Хоменко]

public final static function getInstance() {
if (null === static::$instance) {
static::$instance = new static();
}
return static::$instance;
}

Дмитрий, какой смысл от static в выделенных местах? Статический контекст же един для всех

[Виталий Хоменко]

Дмитрий, или нет? Смотрю на ваш пример и уже не уверен

[Дмитрий]

Виталий Хоменко, это наследование, можно без этого сделать трейт

trait SingletonTrait {
    protected static $instance = null;
 
    public function __construct(){}
 
    public function __clone(){}
 
    public function __wakeup(){}
 
    public final static function getInstance() {
        if (null === self::$instance) {
            self::$instance = new self();
        }
        return self::$instance;
    }
}

Старика это позднее связывание

https://www.php.net/manual/ru/language.oop5.late-s...

[Виталий Хоменко]

Дмитрий, это я понимаю, это нужно для new static(), т.е. определения верхнего класса в цепочке наследования. Но вот в других 3х местах это имеет смысл?

[Дмитрий]

Виталий Хоменко, да я это писал лет 10 назад)

[Александр]

Насколько я понял из вашей беседы
нужно сделать заглушки для магических методов __clone и __wakeup
и метод получения экземпляра сделать финальным - этого будет достаточно чтобы реализовать паттерн

а если вынести это дело в трейт, то можно его использовать для создания нескольких Классов (синглтонов)
правильно я понял?

Answer 1

[Виталий Хоменко]

public static function getInstance(): self {
    if ( empty( self::$instance ) ) {
      self::$instance = new Config();
    }

    return self::$instance;
  }

Comments

[Александр]

Спасибо большое, увидел косяк
теперь наконец то понял суть конструкции

[FanatPHP]

А что мешает мне сделать $config = new Config;?

[Александр]

FanatPHP, мешает приватный конструктор
вы так не сможете сделать, получите ошибку

PHP Fatal error: Uncaught Error: Call to private App\Core\Config::__construct() from invalid context

в этом как раз и суть Singleton

[Максим Федоров]

FanatPHP, приватный конструктор помешает

[FanatPHP]

Максим Федоров, ааа, зевнул. Спасибо

Answer 2

[BATPYIIIKOB]

Также можешь задать классу
final class Config {}
https://refactoring.guru/ru/design-patterns/singleton

Comments

[Александр]

Не пойму, для чего его делать финальным
Все равно не получиться произвести потомка от него
В статье перечитал еще раз и не нашел упоминания о том что класс нужно объявлять финальным
Подскажите причину плиз

[BATPYIIIKOB]

Александр, попробуй - отработает ли clone ?
final можно не добавлять, а вот

private function __clone() {}
    private function __wakeup() {}

я бы прописал