Почему возникает ошибка подключения к базе?

Question

[Максим Золотой]

Какой то бред. Если указываю логин root в прямо то подключается. А если передаю через переменную то получаю ошибку вот такую Warning: mysqli_connect(): (HY000/1045): Access denied for user ''@'localhost' (using password: NO) in C:\openserver\OSPanel\domains\darkdate\www\modules\config.php on line 9
кто может сказать в чем дело?
так работает

а так ошибку выдает

Answer 1

[Immortal_pony]

У вас переменные объявлены за пределами функции.
Есть три варианта решения:

1. Функция должна аргументы принимать, а при вызове вы будете их передавать (рекомендуется):

function connect($server, $login, $password, $db) {
    return mysqli_connect($server, $login, $password, $db)
}

$connection = connect($server, $login, $password, $db);

2. Функция будет извлекать переменные из глобальной области видимости:

function connect() {
    global $server;
    global $login;        
    global $password;
    global $db;

    return mysqli_connect($server, $login, $password, $db);
}

$connection = connect($server, $login, $password, $db)

3. Функция будет анонимной и извлекать данные из локальной области видимости:

$connect = function() use ($server, $login, $password, $db) {
    return mysqli_connect($server, $login, $password, $db);
};

$connection = $connect();

PS При создании вопросов копируйте код, а не вставляйте скриншоты.

Comments

[Дмитрий Энтелис]

отдельно скажем что за вариант (2) нужно сжигать на костре, да и (3) слегка бессмыслен.

[Максим Золотой]

Спасибо за такой четкий и развернутый ответ)

[Максим Золотой]

Дмитрий Энтелис, а могли бы вы пояснить почему?

[Immortal_pony]

Дмитрий Энтелис, я описал подходы к решению проблемы в прицнипе. В данном случае предпочтительнее первый метод и я это также отметил. Что использовать и в каком конкретном случае - отдельный вопрос.

Максим Золотой, потому что использование глобальных переменных порождает неявные зависимости и затрудняет отладку - любая часть приложения имеет возможность изменить данные в глобальной переменной и это может быть проблемой.

[Максим Золотой]

Immortal_pony, а последний вариант чем плох? я пока его стал использовать

[Дмитрий Энтелис]

Максим Золотой, плохо тем, что это синтаксический мусор не имеющий смысла.
Не нужно писать код ради кода.

Анонимные функции нужны или для замыкания, или для структурирования не структурированного кода, когда у вас там какой-то огромный кусок с кучей переменных которые не нужны в общей области видимости.
понятные примеры в первом ответе Что такое анонимная функция и с чем ее едят?

Если уж говорить про совсем правильный подход - лучше иметь отдельный класс для работы с базой данных, который при инициализации (в идеале через какой-нибудь DI ) получает реквизиты для подключения и дальше уже варится внутри себя.