Как обойти global в данной ситуации?

Question

[Twitt]

есть класс Database.php , и есть файл config.php.
Собственно, когда в Database.php делаю require(__DIR__ . '/config.php');
и хочу в construct заюзать переменную с config.php, до тех пор пока я не введу global $db_name; я не смогу заюзать их.
Т.к. я знаю что глобалки это зло, и хочу изначально держать код в чистоте, нужно найти обход данного кейса.

Так вот, какой пусть обхода в данной ситуации?
Сейчас это выглядит вот так

public function __construct()
    {
        try {
            global $config;
            $this->pdo = new PDO("mysql:host=localhost;dbname=" . $config['db_name'], $config['db_user'], $config['db_password']);
        } catch(Exception $e) {
            echo 'Sorry ' . $e->getMessage();
        }
    }

Comments

[Anton Mashletov]

Жаль, что мой ответ удалили. Но тем не менее, всё остальное будет костыли над костылями.

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

Anton Mashletov, хм, я думал ты удалил, твой ответ не успел прочесть!
Мой ответ в точку же :)

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

Anton Mashletov, мой ответ хорош тем, что в конце видео — приходим к готовому компоненту :) счиатй к фремворку, как ты и завещал :)

[Anton Mashletov]

Максим Федоров, да, кстати, смотрел. Но, мне кажется, автор не будет заморачиваться. Ему надо лишь "обойти global".)

[Twitt]

Anton Mashletov, ох, плохо же про авторов думаешь)

Answer 1

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

Если у вас есть цель, кроме «сделать этот код рабочим» есть еще цели: изучить и делать правильно,
советую посмотреть в сторону внедрения DI контейнера, для конфигов в том числе.

Подробно об этом рассказано в видео Елисеева, ссылка на отрывок:
PSR-7 фреймворк 4/6: Контейнер внедрения зависимостей

Ну или взять уже готовый компонент DI (например от Symfony)

Comments

[Twitt]

Спасибо, как раз думал насчет DI, но думал что для конфигов как то слишком сложно будет
Спасибо)

Answer 2

[Ильдар Сарибжанов]

Передавайте параметры соединения в конструктор при создании экземпляра

class Database
{
	public function __construct($db_name, $db_user, $db_password)
	{
		try {
			$this->pdo = new PDO("mysql:host=localhost;dbname=" . $db_name, $db_user, $db_password);
		} catch (Exception $e) {
			echo 'Sorry ' . $e->getMessage();
		}
	}
}


...........................

$db = new Database($config['db_name'], $config['db_user'], $config['db_password']);

И говорят, что try/catch в данном случае - моветон, но это не точно

Comments

[Станислав Б]

try/catch в данном случае - моветон, но это не точно

это совершенно точно.
что мы получим в случае ексепшена?
сожалеющее ехо и поломанный объект Database, в котором $this->pdo = null

Answer 3

[Sanovskiy]

Паттерн Репозиторий
Глобал не нужен. Лишние параметры не нужны.