В чем ошибка функции?

Question

[yerdnaandrey]

День добрый! Сразу хочу извиниться за строчение вопросов на этом форуме. Я так посмотрел, у меня уже 19 тупых вопросов! Итак, перейду к вопросу. В файле functions.php у меня должны были лежать все основные функции сайта. Но уже на функции быстрого подключения к БД что-то пошло не так. Кароче, суть не в этом у меня не может подключить файл (инклудом) config.php лежащий в этой же директории.

functions.php

<?php
mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
include "config.php";
function ConnectDatabase($query){
    $db = new mysqli($hosts, $account, $pswrd, $dbs);
    $connect = $db->query($query);
    return $connect;
}
ConnectDatabase("INSERT INTO notifications (`content`, `to_user`, `dates`, `dates_int`) VALUES ('FHBFG', '12345', 'JANUARY 21, 2020', '12345678')");

?>

config.php

<?php

$dbs = "test_database_off_1";
$hosts = "localhost";
$pswrd = "root";
$account = "root";
$clr = "#6E7CE1";
$nameforum = "name";
$descr = "description";
$dmn = "example.com";
$tag = "1, 2, 3, 4";
?>

Comments

[Ninazu]

Начнем с того что лучше использовать require вместо include, ну чтоб наверняка. Во вторых я бы использовал полный путь к файлу __DIR__. '/config.php';, и в третьих не использовал глобальные переменные а заменил бы их классом

[yerdnaandrey]

Ninazu, спасибо)

Answer 1

[Araik]

Область видимости переменной

Вы используете внутри функции переменные $hosts, $account, $pswrd, $dbs, но не передаете их туда.

Answer 2

[FanatPHP]

Вопросы нормальные, главное чтобы тебе давали осмысленные ответы и ты принимал их к сведению.

Подключение к БД не должно быть функцией. Тем более "быстрое". Код подключения надо написать один раз, без всяких функций.

Не говоря уже о том что у тебя функция не для подключения, а для выполнения запроса, это уже совсем неправильно. НЕЛЬЗЯ соединяться каждый раз, когда ты выполняешь запрос
соединение в скрипте должно быть только одно.

Поэтому функцию сразу на помойку, а код соединения просто положить в файл, который инклюдится во все скрипты.

Comments

[yerdnaandrey]

а каким образом несколько раз подключаться с одного скл линка?

[FanatPHP]

Никак не подключаться
Ты уже подключился один раз, больше не надо
Вот у тебя переменная $db - используй её для всех запросов

[yerdnaandrey]

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

[FanatPHP]

ну вот - через переменную $db

[yerdnaandrey]

FanatPHP, обычным query?

[FanatPHP]

Если в запросе не используются переменные, то обычным query
Но если используются - то только через prepare/bind/execute
вот я писал недавно Как безопасно отправлять пользовательский текст в базу POST запросом?